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
F18862530: D15109.id.diff
Sun, Nov 2, 7:46 PM
F18856578: D15109.diff
Sat, Nov 1, 10:25 AM
F18803407: D15109.id36480.diff
Oct 17 2025, 11:31 PM
F18790827: D15109.id36483.diff
Oct 15 2025, 7:35 PM
F18771935: D15109.id36481.diff
Oct 8 2025, 11:02 PM
F18755828: D15109.diff
Oct 5 2025, 7:36 AM
F18630117: D15109.id36482.diff
Sep 16 2025, 10:48 AM
F18630114: D15109.id36482.diff
Sep 16 2025, 10:48 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 36483

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