Page MenuHomePhabricator
Differential D16959

Close lightbox when clicking on image-frame
ClosedPublic
Actions

Authored by chad on Nov 29 2016, 5:21 PM.
Tags
None
Referenced Files
F15467252: D16959.id40813.diff
Thu, Apr 3, 1:52 PM
F15451454: D16959.id40813.diff
Fri, Mar 28, 10:11 PM
F15451159: D16959.id40814.diff
Fri, Mar 28, 8:30 PM
F15446095: D16959.id.diff
Thu, Mar 27, 5:08 PM
F15395572: D16959.diff
Sun, Mar 16, 8:07 AM
F15385270: D16959.id40813.diff
Fri, Mar 14, 10:07 PM
F15328788: D16959.diff
Mar 7 2025, 6:03 AM
F15283208: D16959.diff
Mar 4 2025, 6:06 AM
View All Files
Subscribers
Korvin
Tokens
"Dat Boi" token, awarded by cspeckmim.

Details

Reviewers
epriestley
Maniphest Tasks
T3612: Lightbox v2
Commits
rPd1838fa5ec5e: Close lightbox when clicking on image-frame
Summary

Ref T3612. Moves the listener to the frame of the image.

Test Plan

Click on image, no close, click on grey frame, closes image. Test image and document, clicking on arrows.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 40813.Nov 29 2016, 5:21 PM
chad retitled this revision from to Close lightbox when clicking on image-frame.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T3612: Lightbox v2.
Herald added a subscriber: Korvin. · View Herald TranscriptNov 29 2016, 5:21 PM
cspeckmim awarded a token.Nov 29 2016, 5:24 PM
epriestley accepted this revision.Nov 29 2016, 6:34 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 29 2016, 6:34 PM
Closed by commit rPd1838fa5ec5e: Close lightbox when clicking on image-frame (authored by chad, committed by chad). · Explain WhyNov 29 2016, 6:35 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
30 lines
webroot/
rsrc/
css/
phui/
1 line
js/
core/
5 lines

Diff 40814

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/css/phui/phui-lightbox.css

Loading...
View Options

webroot/rsrc/js/core/behavior-lightbox-attachments.js

Loading...
Phabricator · Built by Phacility