Differential D19188

Restore lightbox behavior for thumbnailed images
ClosedPublic
Actions

Authored by epriestley on Mar 7 2018, 3:25 PM.

Tags

None

Referenced Files

	F18790831: D19188.id45959.diff
	Wed, Oct 15, 7:36 PM

	F18775842: D19188.id.diff
	Fri, Oct 10, 8:23 PM

	F18764676: D19188.id45959.diff
	Tue, Oct 7, 7:15 AM

	F18762635: D19188.diff
	Mon, Oct 6, 7:59 PM

	F18742276: D19188.id.diff
	Thu, Oct 2, 6:22 PM

	F18733757: D19188.id.diff
	Tue, Sep 30, 10:41 PM

	F18701715: D19188.id45957.diff
	Sat, Sep 27, 8:45 PM

	F18672940: D19188.id45957.diff
	Sep 25 2025, 8:41 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13099: Plans: 2018 Week 10 Bonus Content
Commits: rP05b9016fc385: (stable) Restore lightbox behavior for thumbnailed images
rP229d4677701a: Restore lightbox behavior for thumbnailed images

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 7 2018, 3:25 PM

Harbormaster failed remote builds in B19773: Diff 45957!Mar 7 2018, 3:26 PM

Update Celerity map.

Harbormaster completed remote builds in B19774: Diff 45958.Mar 7 2018, 3:29 PM

epriestley requested review of this revision.Mar 7 2018, 3:29 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 7 2018, 3:33 PM

Closed by commit rP229d4677701a: Restore lightbox behavior for thumbnailed images (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

24 lines

webroot/

rsrc/

js/

core/

behavior-lightbox-attachments.js

5 lines

Diff 45959

View Options

resources/celerity/map.php