Warning: Type of PR label mismatch

To merge this PR, it requires exactly 1 label indicating the type of PR. Other labels are optional and not being checked here.

• Required label: Any label starting with [Type].
• Labels found: .

Read more about Type labels in Gutenberg. Don't worry if you don't have the required permissions to add labels; the PR reviewer should be able to help with the task.

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: jameskoster <jameskoster@git.wordpress.org>
Co-authored-by: jasmussen <joen@git.wordpress.org>
Co-authored-by: ntsekouras <ntsekouras@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Size Change: +284 B (+0.01%)

Total Size: 2.58 MB

_{compressed-size-action}

Flaky tests detected in 6437e91.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/20238406306
📝 Reported issues:

• [Flaky Test] should open and edit templates correctly when active_templates experiment is disabled #73743 in /test/e2e/specs/site-editor/template-id-format.spec.js

As-is, this PR is an improvement, and I think there's a system to it as well. But help me understand the rules though: most of the time, the blue accent color is a hover color. In this concept it's the onselect color. I guess that's because it's particular to the List view type, where the entire row is clickable to select, with no other actions like you'd have on Table or Grid?

@jasmussen I mostly just wanted to tidy up the mess we have on trunk, but good nudge to dig a bit deeper to iterate and find a wider cohesion. I agree blue is a more conventional hover color, but I don't love how it looks on the light gray background. Any thoughts on that? I think a light shade of the theme color might work better (and would align with button styles). Here's a comparison:

I recognise we currently use light gray for table row hover styles, so this would be inconsistent, but those styles might be going away (#73873). Obviously we should use the same shade background for selected rows in both list and table layouts.

Selected rows could use dark gray text and a slightly darker background:

Colors updated to match the mockups above: