Related: #2098099: The issue queue *must* make it easier to upload/review patches
To-dos:
Change sort order in the files table to show most recent on top, on both issue view and issue edit.
Original report
I absolutely love the idea of having a list of all the files attached to an issue right on top below the issue summary. However, it does not change my habit of being most interested in the last file attached.
So right now, I get surely get a list of all the file, but I still have to scroll to the last one.
tvn just pointed me to the fact that it is possible to toggle the visibility status of a file by editing the issue summary. This surely is a possibility to keep the list of attached files short and prevent me from scrolling.
But on the other hand: Everytime someone posts an other patch to an issue, I'd need to take care of the visibility of the files in the list.
My proposal is simple: Change the sorting order to newest on top of the list. So everytime some posts a patch it will pop up right on top of the list and I won't have to scroll to access it.
Bonus Proposal: How about some js to collapse all files after the 5th file, still allowing me to expand the list, if I really need to see that patch from 9 month ago?
Comment | File | Size | Author |
---|---|---|---|
#9 | 2097977-update_extended_file_field_settings_v3.patch | 1.52 KB | jthorson |
#8 | 2097977-update_extended_file_field_settings_v2.patch | 1.21 KB | jthorson |
#7 | 2097977-update_extended_file_field_settings.patch | 938 bytes | jthorson |
#4 | 2097977_change_file_formatter_sort_order.patch | 617 bytes | jthorson |
Comments
Comment #1
webchickYep.
Comment #2
jthorson CreditAttribution: jthorson commentedTagging ... at least the file order change should go in before launch.
Comment #3
tvn CreditAttribution: tvn commentedComment #3.0
tvn CreditAttribution: tvn commented.
Comment #4
jthorson CreditAttribution: jthorson commentedStep 1: Change the sort order on view.
This is actually a drupalorg patch ... but changing the sort order on the 'edit' page is going to require extended_file_field code additions so leaving the issue in this queue for now.
Comment #5
drummCommitted & deploying #4.
Comment #6
jthorson CreditAttribution: jthorson commentedNew 'reverse field order' sort option added to the field formatter 'sortby' settings, and committed to 7.x-1.x-dev. This resolves the sort order for the field formatter, but not the widget.
This will be included on git7site after the next bzr merge, but will require another drupalorg patch to change the field configuration (and have the change survive subsequent git7site rebuilds). Because there are a number of other changes happening with this field formatter at the moment, I'm going to hold off on the drupalorg patch until they've all been completed (and will then tackle the widget seperately).
Comment #7
jthorson CreditAttribution: jthorson commentedHere's the most recent drupalorg patch, to update the settings and maintain pace with the latest extended_file_field code changes.
These should be deployed in conjunction with a bzr merge of extended_file_field 7.x-1.x-dev.
Comment #8
jthorson CreditAttribution: jthorson commentedAnother small tweak, to change the label from 'Files' to 'Recent Files'.
Comment #9
jthorson CreditAttribution: jthorson commentedMissed defining $t(). :(
As before, this should be deployed in conjunction with a bzr merge of extended_file_field 7.x-1.x-dev.
Comment #10
drummCommitted & merging both projects. Since this changes an existing update function, it will need a full migration.
Comment #11
jthorson CreditAttribution: jthorson commentedIn the meantime, we could manually make those changes to the field formatter settings via the UI.
Comment #12.0
(not verified) CreditAttribution: commented.