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.

Comments

thejimbirch’s picture

Issue summary: View changes

From TableResponsive.js

   * The TableResponsive object optimizes table presentation for all screen sizes.
   *
   * A responsive table hides columns at small screen sizes, leaving the most
   * important columns visible to the end user. Users should not be prevented from
   * accessing all columns, however. This class adds a toggle to a table with
   * hidden columns that exposes the columns. Exposing the columns will likely
   * break layouts, but it provides the user with a means to access data, which
   * is a guiding principle of responsive design.

   ...

   * Columns are assumed to be hidden if their header has the class priority-low
   * or priority-medium.

So while you are only seeing the description column hidden on the modules page, this script allows developers to prioritize columns throughout Drupal.

thejimbirch’s picture

Status: Active » Needs review
mgifford’s picture

Status: Needs review » Active
Issue tags: +Usability

There is no patch, so this should be just Active.

meeli’s picture

I 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:

  • "Hide less important columns"
  • "Hide lower priority columns"
  • "Priority columns view"
  • "Show priority columns only"
  • "Show critical columns only"
mgifford’s picture

Status: Active » Needs review
StatusFileSize
new585 bytes

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

thejimbirch’s picture

I am going to review this.

thejimbirch’s picture

StatusFileSize
new202.73 KB
thejimbirch’s picture

Status: Needs review » Reviewed & tested by the community

The patch successfully replaces "Hide unimportant columns" with "Hide lower priority columns". See screenshot:

Hide lower priority columns

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Strings are not yet frozen in beta. Committed a531aa9 and pushed to 8.0.x. Thanks!

  • alexpott committed a531aa9 on 8.0.x
    Issue #1844198 by mgifford: wording of unimportant columns doesn't...
nod_’s picture

Issue tags: +JavaScript

Status: Fixed » Closed (fixed)

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