Page MenuHomePhabricator
Differential D21840

Update "Files" attachment table to show more attachment details and support detachment
ClosedPublic
Actions

Authored by epriestley on May 25 2022, 7:53 PM.
Tags
None
Referenced Files
F13197486: D21840.diff
Mon, May 13, 12:13 AM
F13183035: D21840.diff
Fri, May 10, 6:22 AM
F13179952: D21840.diff
Wed, May 8, 9:33 PM
Unknown Object (File)
Tue, May 7, 1:12 PM
Unknown Object (File)
Sun, May 5, 4:48 AM
Unknown Object (File)
Sat, May 4, 7:27 PM
Unknown Object (File)
Sat, May 4, 10:34 AM
Unknown Object (File)
Wed, May 1, 1:02 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13682: Improve Remarkup file reference attachment behaviors after removal of "attach by default"
Commits
rP7e5f7b96404f: Update "Files" attachment table to show more attachment details and support…
Summary

Ref T13682. Make the "Attached" list in Files a bit more detailed, and add a "Detach" button.

Test Plan

Tried to detach unrelated, referenced, and attached files. Saw attached files detach.

Diff Detail

Repository
rP Phabricator
Branch
table1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25747
Build 35575: arc lint + arc unit

Event Timeline

epriestley requested review of this revision.May 25 2022, 7:53 PM
epriestley created this revision.
Harbormaster completed remote builds in B25747: Diff 52054.May 25 2022, 7:53 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 25 2022, 7:56 PM
Closed by commit rP7e5f7b96404f: Update "Files" attachment table to show more attachment details and support… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP7e5f7b96404f: Update "Files" attachment table to show more attachment details and support….

Revision Contents
Changeset List

PathSize
src/
applications/
files/
application/
2 lines
controller/
PhabricatorFileDetachController.php
PhabricatorFileUICurtainAttachController.php
82 lines
5 lines
98 lines
storage/
16 lines
infrastructure/
internationalization/
translation/
7 lines

Diff 52054

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/files/application/PhabricatorFilesApplication.php

Loading...
View Options

src/applications/files/controller/PhabricatorFileDetachController.php

Loading...
View Options

src/applications/files/controller/PhabricatorFileUICurtainAttachController.php

Loading...
View Options

src/applications/files/controller/PhabricatorFileViewController.php

Loading...
View Options

src/applications/files/storage/PhabricatorFileAttachment.php

Loading...
View Options

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php

Loading...
Phabricator · Built by Phacility