Page MenuHomePhabricator

Restore lightbox behavior for thumbnailed images
ClosedPublic

Authored by epriestley on Mar 7 2018, 3:25 PM.
Tags
None
Referenced Files
F15449192: D19188.diff
Fri, Mar 28, 8:48 AM
F15387265: D19188.id45959.diff
Sat, Mar 15, 2:07 AM
F15376378: D19188.diff
Thu, Mar 13, 3:11 AM
F15356016: D19188.id45959.diff
Tue, Mar 11, 6:07 AM
Unknown Object (File)
Sun, Mar 2, 2:43 PM
Unknown Object (File)
Thu, Feb 27, 8:34 AM
Unknown Object (File)
Feb 18 2025, 9:08 AM
Unknown Object (File)
Feb 9 2025, 5:38 AM
Subscribers
None

Details

Summary

Ref T13099. See https://discourse.phabricator-community.org/t/lightbox-not-working/1201. The Content-Security-Policy changes rewrote some of this code and the handling for "Download" links is incorrectly catching clicks on thumbnailed images.

Test Plan

Clicked a thumbnailed image, got a lightbox. Command-clicked a download link, still got link behavior instead of a lightbox.

Diff Detail

Repository
rP Phabricator
Branch
lightbox1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 19773
Build 26784: Run Core Tests
Build 26783: arc lint + arc unit

Unit TestsFailed

TimeTest
288 msPhabricatorCelerityTestCase::Unknown Unit Message ("")
Assertion failed, expected 'true' (at PhabricatorCelerityTestCase.php:32): When this test fails, it means the Celerity resource map is out of date. Run `bin/celerity map` to rebuild it. ACTUAL VALUE
1 msAlmanacNamesTestCase::Unknown Unit Message ("")
30 assertions passed.
0 msAlmanacServiceTypeTestCase::Unknown Unit Message ("")
1 assertion passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
6 assertions passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
3 assertions passed.
View Full Test Results (1 Failed · 348 Passed)

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 7 2018, 3:33 PM
This revision was automatically updated to reflect the committed changes.