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
F13209177: D15109.id36481.diff
Thu, May 16, 10:22 PM
F13208410: D15109.diff
Thu, May 16, 2:52 PM
F13199445: D15109.diff
Mon, May 13, 2:52 PM
F13197870: D15109.id36481.diff
Mon, May 13, 1:36 AM
F13197868: D15109.id36480.diff
Mon, May 13, 1:36 AM
F13197866: D15109.id36482.diff
Mon, May 13, 1:36 AM
F13192699: D15109.diff
Sun, May 12, 9:25 AM
F13176548: D15109.diff
Wed, May 8, 2:59 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
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