Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
drupal 8 modules page viewed in a mobile device has a link that states hide unimportant columns. This doesn't describe the functionality of the link. Hide description column would make more sense.
Comment | File | Size | Author |
---|---|---|---|
#7 | Hide lower priority columns.png | 202.73 KB | thejimbirch |
#5 | PriorityColumns-1844198-5.patch | 585 bytes | mgifford |
hideunimporantcolumns.png | 7.07 KB | pgrucza |
Comments
Comment #1
thejimbirch CreditAttribution: thejimbirch commentedFrom TableResponsive.js
So while you are only seeing the description column hidden on the modules page, this script allows developers to prioritize columns throughout Drupal.
Comment #2
thejimbirch CreditAttribution: thejimbirch commentedComment #3
mgiffordThere is no patch, so this should be just Active.
Comment #4
meeli CreditAttribution: meeli commentedI think this wording is used as part of the responsive tables module globally, so we shouldn't change it to just describe the exact column being hidden in this circumstance.
I agree that "unimportant" is a little strong though.
Some options:
Comment #5
mgiffordI like "Hide lower priority columns" myself. Thanks for those suggestions.
Also, the table name has changed to core/misc/tableresponsive.js and here's a patch for review.
Just trying to get this moving..
Comment #6
thejimbirch CreditAttribution: thejimbirch commentedI am going to review this.
Comment #7
thejimbirch CreditAttribution: thejimbirch commentedComment #8
thejimbirch CreditAttribution: thejimbirch commentedThe patch successfully replaces "Hide unimportant columns" with "Hide lower priority columns". See screenshot:
Comment #9
alexpottStrings are not yet frozen in beta. Committed a531aa9 and pushed to 8.0.x. Thanks!
Comment #11
nod_