Page MenuHomePhabricator

Restore lightbox behavior for thumbnailed images
ClosedPublic

Authored by epriestley on Mar 7 2018, 3:25 PM.
Tags
None
Referenced Files
F14126926: D19188.id45959.diff
Sat, Nov 30, 6:18 PM
Unknown Object (File)
Fri, Nov 29, 10:51 AM
Unknown Object (File)
Wed, Nov 27, 1:56 AM
Unknown Object (File)
Mon, Nov 25, 11:32 AM
Unknown Object (File)
Sat, Nov 23, 5:34 AM
Unknown Object (File)
Mon, Nov 18, 6:32 PM
Unknown Object (File)
Fri, Nov 15, 12:49 PM
Unknown Object (File)
Mon, Nov 11, 10:54 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.