Page MenuHomePhabricator
Differential D9574

Make the Pholio download button actually download
ClosedPublic
Actions

Authored by epriestley on Jun 16 2014, 3:56 PM.
Tags
None
Referenced Files
F18228637: D9574.id.diff
Wed, Aug 20, 7:44 AM
F18220311: D9574.diff
Tue, Aug 19, 2:25 PM
F18087882: D9574.diff
Wed, Aug 6, 6:01 AM
F17802674: D9574.id22943.diff
Jul 25 2025, 9:23 AM
F17801557: D9574.id22943.diff
Jul 25 2025, 7:57 AM
F17801553: D9574.id22941.diff
Jul 25 2025, 7:56 AM
F17767719: D9574.id22941.diff
Jul 23 2025, 5:56 AM
F17758138: D9574.diff
Jul 22 2025, 4:53 PM
View All Files
Subscribers
epriestley

Details

Reviewers
chad
Commits
Restricted Diffusion Commit
rP0efa1ca6430a: Make the Pholio download button actually download
Summary
  • When the button is clicked, actually download the file or image.
  • Add aural hints for the icon-only buttons.
  • Use a "photo" icon for "view raw image", so the "arrows pointing outward" icon can be used for "fullscreen" some day.
Test Plan

Clicked link, got a download.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 22941.Jun 16 2014, 3:56 PM
epriestley retitled this revision from to Make the Pholio download button actually download.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 16 2014, 3:56 PM
Harbormaster completed remote builds in B1142: Diff 22941.Jun 16 2014, 3:57 PM
chad accepted this revision.Jun 16 2014, 4:00 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 16 2014, 4:00 PM
epriestley closed this revision.Jun 16 2014, 4:15 PM
epriestley updated this revision to Diff 22943.

Closed by commit rP0efa1ca6430a (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
pholio/
view/
16 lines
webroot/
rsrc/
css/
application/
pholio/
22 lines
js/
application/
pholio/
23 lines

Diff 22943

View Options

src/applications/pholio/view/PholioMockImagesView.php

Loading...
View Options

webroot/rsrc/css/application/pholio/pholio.css

Loading...
View Options

webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js

Loading...
Phabricator · Built by Phacility