Page MenuHomePhabricator
Differential D19157

Stop using forms to download files in file embed and lightbox elements
ClosedPublic
Actions

Authored by epriestley on Mar 1 2018, 1:19 AM.
Tags
None
Referenced Files
F15562629: D19157.diff
Wed, Apr 30, 3:31 AM
F15531394: D19157.id45888.diff
Wed, Apr 23, 11:07 AM
F15508095: D19157.id.diff
Wed, Apr 16, 1:30 AM
F15484715: D19157.id45888.diff
Wed, Apr 9, 7:43 PM
F15466083: D19157.id45892.diff
Thu, Apr 3, 1:24 AM
F15461639: D19157.id.diff
Tue, Apr 1, 9:45 AM
F15460150: D19157.diff
Mon, Mar 31, 8:41 PM
F15447670: D19157.id45888.diff
Mar 28 2025, 12:53 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13094: Improve file behaviors around POST requests and downloads
Commits
rPa2fdf14275f9: Stop using forms to download files in file embed and lightbox elements
Summary

Depends on D19156. Ref T13094. This replaces the remaining forms in the file embed view and lightbox with normal download links.

Test Plan

Clicked "Download" and lightbox -> download for embedded files.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 1 2018, 1:19 AM
Harbormaster completed remote builds in B19699: Diff 45888.Mar 1 2018, 1:21 AM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 1 2018, 1:21 AM
epriestley requested review of this revision.
Closed by commit rPa2fdf14275f9: Stop using forms to download files in file embed and lightbox elements (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
24 lines
src/
view/
layout/
20 lines
page/
30 lines
webroot/
rsrc/
js/
core/
59 lines

Diff 45892

View Options

resources/celerity/map.php

Loading...
View Options

src/view/layout/PhabricatorFileLinkView.php

Loading...
View Options

src/view/page/PhabricatorStandardPageView.php

Loading...
View Options

webroot/rsrc/js/core/behavior-lightbox-attachments.js

Loading...
Phabricator · Built by Phacility