WordPress / gutenberg

The Block Editor project for WordPress and beyond. Plugin is available from the official repository.
https://wordpress.org/gutenberg/
Other
10.5k stars 4.19k forks source link

Table layout: Only show row hover styles when bulk actions exist #65234

Open jameskoster opened 1 month ago

jameskoster commented 1 month ago

A hover style suggests interactivity. Normally clicking a table row will select it. However when there are no bulk actions clicking a row will do nothing, which arguably makes the hover style a bit misleading.

As an example, view Theme Patterns:

https://github.com/user-attachments/assets/27c6e693-d283-4688-898d-ae12b6bcec0c

One potential argument for retaining the hover style is that it can be a useful visual aid when scanning dense data tables horizontally, but I think there are better solutions to that like pinning columns (https://github.com/WordPress/gutenberg/issues/64679).

cc @WordPress/gutenberg-design for thoughts on this one.

jasmussen commented 1 month ago

Makes sense.