Differential D19188

Restore lightbox behavior for thumbnailed images
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15469046: D19188.id45957.diff
	Fri, Apr 4, 8:29 AM

	F15459868: D19188.id.diff
	Mon, Mar 31, 5:31 PM

	F15459331: D19188.id45959.diff
	Mon, Mar 31, 11:59 AM

	F15459330: D19188.id45957.diff
	Mon, Mar 31, 11:59 AM

	F15459329: D19188.id45958.diff
	Mon, Mar 31, 11:59 AM

	F15456054: D19188.id45958.diff
	Sun, Mar 30, 6:41 AM

	F15454871: D19188.id.diff
	Sat, Mar 29, 8:56 PM

	F15449192: D19188.diff
	Fri, Mar 28, 8:48 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