Move raw file download icon to file box in Diffusion
ClosedPublic
Actions

Authored by chad on Feb 21 2014, 11:18 PM.

Tags

None

Referenced Files

	F19249824: D8302.id19748.diff
	Mon, Dec 22, 7:14 AM

	F19064003: D8302.id.diff
	Sat, Nov 29, 8:03 PM

	F19061548: D8302.id.diff
	Sat, Nov 29, 12:41 PM

	F19058752: D8302.id.diff
	Sat, Nov 29, 3:20 AM

	F19052307: D8302.diff
	Fri, Nov 28, 4:30 AM

	F18868170: D8302.diff
	Nov 4 2025, 5:01 AM

	F18854796: D8302.id19750.diff
	Nov 1 2025, 12:38 AM

	F18854795: D8302.id19749.diff
	Nov 1 2025, 12:38 AM

Subscribers

Details

Reviewers

epriestley

Maniphest Tasks

T4467: Minimize actionlist in Diffusion Browse View

Commits

Restricted Diffusion Commit
rP8662b27f89c6: Move raw file download icon to file box in Diffusion

Summary

For images and text, show the "Raw" buttons on the file's ObjectBox

Test Plan

View an image and a text file in Diffusion, click on the download link in each.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

Oops

{F116383}

{F116385}

It looks like binary, non-image files have some sketchiness. Here's an example:

https://secure.phabricator.com/diffusion/P/browse/master/webroot/rsrc/swf/aphlict.swf

Closed by commit rP8662b27f89c6 (authored by @chad).

good catch, will hit you up with another diff here

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

controller/

DiffusionBrowseFileController.php

62 lines

Diff 19750

View Options

src/applications/diffusion/controller/DiffusionBrowseFileController.php

Move raw file download icon to file box in DiffusionClosedPublicActions