Repository navigation
chore(web): inbox and fleet rail entries use inbox and radar icons - #219
Merged
Merged
Conversation
The Inbox control in the sidebar header showed a notification bell while the Inbox page itself uses the inbox tray icon, and the Fleet control showed the people icon that Personas already uses. Both rail entries now show an icon for the page they open: the inbox tray for Inbox and a radar for Fleet. The user guide no longer calls the control the "inbox bell". Closes #218 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
bryantderosier
added
the
j5 branding
github-actions
Bot
added
vouch:trusted
bryantderosier
requested review from
BastiHu,
Jacksondr5 and
tyler-barton-horizon
September 21, 2026 15:51
Shiki stops tokenizing a line after 500 ms of wall clock, stretches the last matched token to the end of that line, and carries the halted grammar state into the next line. On a loaded CI runner a cold regex engine crossed that budget on the first template literal, so "keeps grammar state across inline comment rows" saw the closing row highlighted from the wrong state while the same commit passed in another run. The file now freezes Date.now for every test, the same fixture treatment shikiReviewHighlighter.test.ts already uses, so the tests measure grammar state handling and not runner speed. The runtime budget is unchanged. The FORK.md fixtures entry records the new test delta. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Jacksondr5
approved these changes
Sep 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #218
The Inbox control in the sidebar header showed a notification bell while the Inbox page it opens uses the inbox tray icon, and the Fleet control showed the same people icon family the Personas settings entry uses. Neither icon said what it opened.
I swapped the bell for lucide's
InboxIconand the people icon for lucide'sRadarIcon, and updated the two lines in the user guide that called the control the "inbox bell". Both components are J5-owned, so no upstream file changed and the FORK.md inventory is unchanged. Mobile has no equivalent rail controls.Verified with the web app typecheck, lint on the two files, the existing
HumanInboxBellunit test, and a format check.Built with Claude Fable 5.1 in Claude Code.
🤖 Generated with Claude Code