Standardise area highlights

[lawsie]

Summary

Makes highlights within the toolbox and workspace more consistent.

• Blue highlight on workspace replaced by flock's darker yellow highlight
• Coral highlight on workspace search removed
• Categories within the toolbar now highlight in the lighter yellow for consistency, and the highlighting is improved

AI usage

Claude Sonnet 4.6 used throughout, all changes reviewed by a human before being applied.

Summary by CodeRabbit

• New Features

  • Keyboard navigation is now activated when using the Mod+KeyT shortcut.

• Style

  • Improved visual feedback for keyboard navigation focus indicators in the toolbox.
  • Removed default border from search highlight overlay for cleaner appearance.
  • Enhanced focus ring styling for better accessibility support.

[coderabbitai]

Warning

Rate limit exceeded

@lawsie has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 53 minutes and 20 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 414d14dc-9ac8-4c69-a784-312cd000da35

📥 Commits

Reviewing files that changed from the base of the PR and between 28cec93 and 43411ae.

📒 Files selected for processing (1)

• style/blockly.css

📝 Walkthrough

Walkthrough

This PR activates Blockly's keyboard navigation controller in the Mod+KeyT handler and updates CSS focus styling to support keyboard-focused toolbox categories using inset box-shadow focus rings, with supporting theme token and low-vision selector adjustments.

Changes

Keyboard Navigation and Focus Styling

Layer / File(s)	Summary
Keyboard Navigation Activation main/main.js	The deferred Mod+KeyT handler now activates Blockly's keyboard navigation controller via setIsActive(true) before continuing with toolbox selection logic.
Toolbox Focus Styling and Theme Token Update style/blockly.css	The --color-toolbox-focus theme token now references --block-outline-focus instead of --color-outline-focus. Toolbox category focus rules are refactored from aria-selected/outline selectors to keyboard-navigation/focus-manager selectors using inset box-shadow focus rings on .blocklyActiveFocus elements.
Focus Ring and Field Styling style/blockly.css	New .blocklyWorkspaceFocusRing rule sets stroke to var(--color-outline-focus) with !important. Search overlay border rule is commented out, and low-vision field grid selector formatting is adjusted.
Console Logging Reformatting main/main.js	"Moving top block" status message splits console.log arguments across multiple lines for readability; session initialization logic unchanged.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• flipcomputing/flock#624: Both PRs modify the Mod+KeyT toolbox keyboard shortcut behavior in main/main.js and update overlapping toolbox focus styling rules including --color-toolbox-focus and focus-ring/outline handling in style/blockly.css.
• flipcomputing/flock#625: Both PRs refactor Blockly keyboard-navigation and focus-ring styling in style/blockly.css (toolbox/flyout/category focus rules), with this PR also activating Blockly.keyboardNavigationController in the shortcut handler.

Poem

🐰 A keyboard navigation controller springs to life,
With focus rings and styled toolbox, no strife,
Theme tokens dance to block-outline's tune,
Accessible shortcuts bloom—what a boon!
CSS and JS now waltz in sync.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Standardise area highlights' is directly related to the main changes: updating highlight styling for toolbox focus, workspace search, and keyboard navigation focus rings to use consistent color tokens.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@style/blockly.css`:
- Line 266: Remove the commented-out CSS declaration "/*border: solid
var(--color-border-highlight) 4px;*/" from blockly.css to satisfy the
comment-whitespace-inside stylelint rule; since the border is intentionally
removed, delete that commented line rather than leaving it commented so linting
will pass.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f93b1032-7e51-4a21-ab2a-e8341273d0e6

📥 Commits

Reviewing files that changed from the base of the PR and between 66fa37e and 28cec93.

📒 Files selected for processing (2)

• main/main.js
• style/blockly.css