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
F18828737: D19157.id.diff
Fri, Oct 24, 5:57 PM
F18790863: D19157.diff
Wed, Oct 15, 8:12 PM
F18733746: D19157.id.diff
Tue, Sep 30, 10:41 PM
F18696775: D19157.id.diff
Sat, Sep 27, 8:26 AM
F18689980: D19157.diff
Fri, Sep 26, 6:58 PM
F18606371: D19157.diff
Sep 13 2025, 10:12 PM
F18509374: D19157.id.diff
Sep 5 2025, 3:22 AM
F18502273: D19157.diff
Sep 4 2025, 10:19 PM
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