Page MenuHomePhabricator

Restore lightbox behavior for thumbnailed images
ClosedPublic

Authored by epriestley on Mar 7 2018, 3:25 PM.
Tags
None
Referenced Files
F22575091: D19188.diff
Sun, Sep 27, 9:16 AM
Unknown Object (File)
Aug 18 2026, 11:55 AM
Unknown Object (File)
Mar 28 2026, 7:42 AM
Unknown Object (File)
Feb 11 2026, 6:26 AM
Unknown Object (File)
Jan 16 2026, 7:22 AM
Unknown Object (File)
Nov 22 2025, 6:27 PM
Unknown Object (File)
Nov 2 2025, 6:09 PM
Unknown Object (File)
Nov 1 2025, 7:17 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.