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.

Support from Acquia helps fund testing for Drupal Acquia logo

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
FileSize
585 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

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.