Page MenuHomePhabricator
Differential D15109

Add an edit link on hover for Project profile images
ClosedPublic
Actions

Authored by chad on Jan 24 2016, 8:16 PM.
Tags
None
Referenced Files
F13284489: D15109.diff
Mon, Jun 3, 5:32 AM
F13266322: D15109.diff
Tue, May 28, 12:36 PM
F13259275: D15109.id.diff
Sun, May 26, 6:54 PM
F13250241: D15109.diff
Fri, May 24, 1:27 PM
F13242260: D15109.id36480.diff
Thu, May 23, 2:09 AM
F13240060: D15109.diff
Wed, May 22, 9:58 AM
F13237135: D15109.diff
Tue, May 21, 12:32 PM
F13209177: D15109.id36481.diff
Thu, May 16, 10:22 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rP19f4e8631f00: Add an edit link on hover for Project profile images
Summary

Minor point of polish, but feels really nice. Hover over photo and edit a link to change the picture.

Test Plan

hover hover, clicky clicky

pasted_file (197×311 px, 21 KB)

Diff Detail

Repository
rP Phabricator
Branch
hover-edit-header (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10320
Build 12576: Run Core Tests
Build 12575: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 36480.Jan 24 2016, 8:16 PM
chad retitled this revision from to Add an edit link on hover for Project profile images.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptJan 24 2016, 8:16 PM
chad edited the test plan for this revision. (Show Details)Jan 24 2016, 8:16 PM
chad updated this revision to Diff 36481.Jan 24 2016, 8:20 PM
  • Remove border shadow
epriestley accepted this revision.Jan 24 2016, 9:03 PM
epriestley edited edge metadata.

Maybe only add this if the user has permission to edit it? Not sure.

This revision is now accepted and ready to land.Jan 24 2016, 9:03 PM
chad updated this revision to Diff 36482.Jan 24 2016, 9:19 PM
chad edited edge metadata.
  • Add edit check
Closed by commit rP19f4e8631f00: Add an edit link on hover for Project profile images (authored by chad, committed by chad). · Explain WhyJan 24 2016, 9:20 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
src/
applications/
project/
controller/
9 lines
view/
phui/
43 lines
webroot/
rsrc/
css/
phui/
27 lines

Diff 36482

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/project/controller/PhabricatorProjectProfileController.php

Loading...
View Options

src/view/phui/PHUIHeaderView.php

Loading...
View Options

webroot/rsrc/css/phui/phui-header-view.css

Loading...
Phabricator · Built by Phacility