...use case where this cases UX issue: #2465275: Respect the alphabetical order of the entries in the site Status report page.

...until @RobLoach mentioned it in the related issue, I didn't realize it was because the list is sorted in a case-sensitive alphabetical order. I believe that it is more natural to have that list sorted in a case-insensitive alphabetical order instead because that's how humans look for entries in the list. Personally, because "jQuery" starts with a "j", I expected its entry to be right after "GD library rotate and desaturate effects" and before "Node Access Permissions". Because the specific project has a lower-case letter in its name, its respective entry in the site status report page is listed last though (after "Web server"):

CommentFileSizeAuthor
#13 sort_report_page_alphabetically_but_case_insensitively-2500717-13.patch504 bytesafi13
#7 sort_report_page_alphabetically_but_case_insensitively-2500717-7.patch558 bytesafi13
#1 sort_report_page_alphabetically_but_case_insensitively-2500717-1.patch504 bytesafi13
status_report_page_jquery_listed_last.png11.38 KBklonos

Comments

afi13’s picture

afi13 commented
StatusFileSize
new sort_report_page_alphabetically_but_case_insensitively-2500717-1.patch504 bytes
afi13’s picture

afi13 commented
Status: Active » Needs review
afi13’s picture

afi13 commented
Status: Needs review » Reviewed & tested by the community
cilefen’s picture

cilefen commented
Version: 7.x-dev » 8.0.x-dev
Status: Reviewed & tested by the community » Active
Issue tags: +Needs beta evaluation, +Needs backport to D7

+1 for this issue in general.

@afi13 Nice work but we are not permitted to mark our own patches as RTBC.

The problem exists in Drupal 8 so it must be fixed there first.

cilefen’s picture

cilefen commented
Issue tags: +Novice
afi13’s picture

afi13 commented

Ok, sorry.

afi13’s picture

afi13 commented
StatusFileSize
new sort_report_page_alphabetically_but_case_insensitively-2500717-7.patch558 bytes

Patch for Drupal 8

afi13’s picture

afi13 commented
Status: Active » Needs review
cilefen’s picture

cilefen commented
Issue tags: +Needs usability review
evilfurryone’s picture

evilfurryone commented
Status: Needs review » Reviewed & tested by the community

Tested the patch and works in my opinion. in my case cURL information was moved from the bottom of the list into it's proper location.

Makes sense that the sorting should be case-insensitive because otherwise could lead to situations where someone misses and key information about the site just because he did not look at the bottom of the page.

PS. I did not understand what the usability review would need to entail in this issue.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Version: 8.0.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Nice.

Committed and pushed to 8.0.x. Thanks!

Moving back to 7.x for backport.

  • webchick committed 71e5402 on 8.0.x 
    Issue #2500717 by afi13, klonos, cilefen, evilfurryone: Sort entries in...
afi13’s picture

afi13 commented
StatusFileSize
new sort_report_page_alphabetically_but_case_insensitively-2500717-13.patch504 bytes

Drupal 7 patch

afi13’s picture

afi13 commented
Status: Patch (to be ported) » Needs review
cilefen’s picture

cilefen commented
Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs beta evaluation, -Novice, -Needs usability review

I manually tested and it looks great.

David_Rothstein’s picture

David_Rothstein commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x - thanks!

  • David_Rothstein committed 186368f on 7.x 
    Issue #2500717 by afi13, cilefen, evilfurryone: Sort entries in the site...

Status: Fixed » Closed (fixed)

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