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?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

Category: feature » task
Priority: Normal » Major

Yep.

jthorson’s picture

Status: Active » Needs review
Issue tags: +drupal.org D7

Tagging ... at least the file order change should go in before launch.

tvn’s picture

Project: [Archive] Drupal.org D7 upgrade QA » Extended File Field
Version: » 7.x-1.x-dev
Status: Needs review » Needs work
Issue tags: +project
tvn’s picture

Issue summary: View changes

.

jthorson’s picture

Step 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.

drumm’s picture

Committed & deploying #4.

jthorson’s picture

Status: Needs work » Needs review

New '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).

jthorson’s picture

Here'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.

jthorson’s picture

Another small tweak, to change the label from 'Files' to 'Recent Files'.

jthorson’s picture

Missed defining $t(). :(

As before, this should be deployed in conjunction with a bzr merge of extended_file_field 7.x-1.x-dev.

drumm’s picture

Status: Needs review » Fixed

Committed & merging both projects. Since this changes an existing update function, it will need a full migration.

jthorson’s picture

In the meantime, we could manually make those changes to the field formatter settings via the UI.

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

.

  • Commit dfc1ce9 on 7.x-1.x, empty-file-2150029 by jthorson:
    [#2097977] by jthorson: Add reverse field order sorting capability