Skip to content

fix: menu — on is an orange fill, focus a thick light blue frame (#143) - #144

Merged
compot-dev merged 1 commit into
v2from
fix/menu-focus-vs-on
Oct 13, 2026
Merged

compot-dev merged 1 commit into
v2from
fix/menu-focus-vs-on

Conversation

@compot-dev

Copy link
Copy Markdown
Collaborator

Closes #143.

One cue per state in the menu:

  • On / active (D on, Q, S, U, the active slot): orange fill (#B85C00), like a pressed key; no coloured border any more.
  • Focus: a 3 dp light blue frame (#40C4FF) only — instead of the 2 dp dark blue one that was hard to tell from the orange border.
  • Both: orange fill inside the blue frame.

Checked on AG-216 (CI build f246f79) with the D-pad: D and Q filled orange, the focus frame on U clearly different; with the focus on Q, the orange fill inside the blue frame.

…coloured border for on (#143)

Co-Authored-By: Claude <noreply@anthropic.com>
@compot-dev

Copy link
Copy Markdown
Collaborator Author

/review

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can turn these tips off under Display > Comment parts

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@compot-dev
compot-dev merged commit ad89898 into v2 Oct 13, 2026
4 checks passed
@compot-dev
compot-dev deleted the fix/menu-focus-vs-on branch October 13, 2026 15:50
compot-dev added a commit that referenced this pull request Oct 15, 2026
fix: menu — on is an orange fill, focus a thick light blue frame (#143)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Menu: the focus and the "on" state look alike (both a coloured border)

1 participant