Add subtle selected-row highlight to Walker#5402
Add subtle selected-row highlight to Walker#5402pipetogrep wants to merge 2 commits intobasecamp:devfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a subtle, theme-aware background highlight for the currently selected Walker result in the default Omarchy theme, improving visual focus without introducing a new color token.
Changes:
- Add a
child:selectedCSS rule that sets a semi-transparent background using the existing@textcolor token.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
This is a great accessibility improvement. I've had problems before seeing which menu item is selected. The contrast of the selected text is a bit low, is there a way to make it more distinct from the background without changing the aesthetic? |
I think it's intentionally subtle, I didn't want to touch those parts of the themes. |
Summary
Adds a subtle, theme-aware background to the selected Walker result in the default Omarchy theme.
This makes the active item easier to pick out while keeping the highlight tied to the existing text color token and transparency.