Page MenuHomePhabricator
Differential D9630

Add a masked image to Pholio Inline Comment Dialog
ClosedPublic
Actions

Authored by chad on Jun 19 2014, 4:04 PM.
Tags
None
Referenced Files
F16642987: D9630.diff
Tue, Jun 17, 10:47 AM
Unknown Object (File)
Tue, May 20, 6:01 AM
Unknown Object (File)
Mon, May 19, 9:20 PM
Unknown Object (File)
May 6 2025, 1:44 AM
Unknown Object (File)
Apr 20 2025, 4:11 PM
Unknown Object (File)
Apr 4 2025, 9:15 PM
Unknown Object (File)
Mar 27 2025, 9:40 PM
Unknown Object (File)
Mar 26 2025, 8:29 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rPc3843a9963b6: Add a masked image to Pholio Inline Comment Dialog
Summary

n/a

Test Plan

Tested both ajax and regular pages, test various comments, images.

Screen_Shot_2014-06-19_at_8.57.49_AM.png (469×650 px, 90 KB)

Screen_Shot_2014-06-19_at_9.03.02_AM.png (454×618 px, 142 KB)

Diff Detail

Repository
rP Phabricator
Branch
pholio-dialog
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1241
Build 1241: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 23094.Jun 19 2014, 4:04 PM
chad retitled this revision from to Add a masked image to Pholio Inline Comment Dialog.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJun 19 2014, 4:04 PM
chad edited the test plan for this revision. (Show Details)Jun 19 2014, 4:04 PM
Harbormaster completed remote builds in B1241: Diff 23094.Jun 19 2014, 4:05 PM
epriestley accepted this revision.Jun 19 2014, 5:49 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 19 2014, 5:49 PM
chad closed this revision.Jun 19 2014, 6:00 PM
chad updated this revision to Diff 23096.

Closed by commit rPc3843a9963b6 (authored by @chad).

Revision Contents
Changeset List

PathSize
resources/
celerity/
4 lines
src/
applications/
pholio/
controller/
53 lines
webroot/
rsrc/
css/
application/
pholio/
8 lines

Diff 23094

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/pholio/controller/PholioInlineController.php

Loading...
View Options

webroot/rsrc/css/application/pholio/pholio-inline-comments.css

Loading...
Phabricator · Built by Phacility