search

backdrop-contrib / gin

Admin theme with a strong focus on improving the Editorial Experience.
https://backdropcms.org/project/gin
GNU General Public License v2.0
5 stars 2 forks source link

"Show all columns" link doesn't work #106

Closed olafgrabienski closed 3 months ago

olafgrabienski commented 6 months ago

On the Manage content page (admin/content) there is a "Show all columns" link. The title attribute describes the link as follows:

Toggle visibility of table cells, that were hidden to make the table fit within a small screen.

For me, the link is visible all of the time, no matter how wide the screen is. And if the screen is too small to show all table columns, the link doesn't work anyway.

Screenshot:

laryn commented 3 months ago

@olafgrabienski Are you able to test the PR I just linked here? It just adds a bit of CSS that was in Seven to show and hide various table-related elements at specific breakpoints.

olafgrabienski commented 3 months ago

@laryn Thanks for the PR! I've just tested it on a local site, and it works as expected: With the PR, the "Show all columns" link is only visible on narrower screens. When I use it, all columns are displayed, and the link text changes to "Hide less important columns" (works as well).

laryn commented 3 months ago

Thanks @olafgrabienski -- merged.