Paths

Table of Contentst

Fix lightbox downloads for embeded images and a warning
ClosedPublic
Actions

Authored by epriestley on Oct 17 2013, 6:16 PM.

Tags

None

Referenced Files

	F15534306: D7341.id16534.diff
	Wed, Apr 23, 11:42 PM

	F15524185: D7341.id.diff
	Mon, Apr 21, 6:29 AM

	F15505647: D7341.diff
	Tue, Apr 15, 4:39 AM

	F15432276: D7341.diff
	Mar 24 2025, 5:13 PM

	F15427956: D7341.diff
	Mar 23 2025, 5:21 PM

	F15390644: D7341.id.diff
	Mar 15 2025, 6:23 AM

	F15385216: D7341.id16533.diff
	Mar 14 2025, 9:59 PM

	F15343910: D7341.id16534.diff
	Mar 10 2025, 1:49 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan
chad
dctrwatson

Commits

Restricted Diffusion Commit
rP32dca4b553f8: Fix lightbox downloads for embeded images and a warning

Summary

I refactored this recently and accidentally dropped the download URI.

Also fix a warning with, e.g., files named README.

Test Plan

Clicked a thumb, clicked "Download", got a file.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

chad accepted this revision.Oct 17 2013, 6:17 PM

Closed by commit rP32dca4b553f8 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

view/

DifferentialChangesetDetailView.php

2 lines

files/

remarkup/

PhabricatorRemarkupRuleEmbedFile.php

1 line

Diff 16534

View Options

src/applications/differential/view/DifferentialChangesetDetailView.php