Page MenuHomePhabricator

Restore lightbox behavior for thumbnailed images
ClosedPublic

Authored by epriestley on Mar 7 2018, 3:25 PM.
Tags
None
Referenced Files
F17729774: D19188.diff
Sat, Jul 19, 8:01 PM
Unknown Object (File)
Mon, Jun 23, 5:44 PM
Unknown Object (File)
May 7 2025, 3:24 PM
Unknown Object (File)
May 7 2025, 1:14 AM
Unknown Object (File)
Apr 30 2025, 7:50 AM
Unknown Object (File)
Apr 25 2025, 7:44 PM
Unknown Object (File)
Apr 22 2025, 6:34 AM
Unknown Object (File)
Apr 12 2025, 1:45 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.