Header: reopen the last tool view from the workspace strip - #124
Merged
Merged
Conversation
A button at the right end of the workspace strip opens whichever view the tool window showed last (Library, Algorithms, Grammar or Reference), named and iconed from AUX_META, with its shortcut in the tooltip. - The strip gets a frame (.tab-strip) that holds the border, so the button sits inside the strip but outside the scroller: with many workspaces the tabs scroll under it and it never scrolls away. The overflow chevron moves into the frame for the same reason and is drawn flat, like the tabs it lists. - On a phone the button shares a pill with the workspace switcher - icon only below 560px. The window opens below the header there, so the button is lit while its view is open and a second press closes it. - The view is remembered per reader (automata-aux-last); Library until another view has been opened. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HPVKDwzQWjyqE1nApBQt6s
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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
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.
Adds a header button that reopens whichever tool view (Library, Algorithms, Grammar or Reference) was open last. It's the design we settled on after the header mock-ups.
What changed
Desktop. The button sits at the right end of the workspace tab strip, behind a thin divider. It looks like an unselected tab with an icon and the view's name: no box at rest, and the same hover as the tabs. The tooltip shows the view's shortcut key.
.tab-strip), and the border moved from.tab-barto that frame. The tabs scroll inside the frame; the button stays fixed outside the scrolling part. With many workspaces the tabs scroll underneath it and it never scrolls out of view, which was the edge case found in the mock-ups.Phone (below 900px). There's no tab strip on phones, so the button shares a pill with the workspace switcher: the workspace name, a divider, then the button. Below 560px it shows the icon only. On phones the tool window opens below the header, so the button stays visible: it's highlighted while its view is open, and pressing it again closes the view.
Remembering the view. The last view is saved per browser in
automata-aux-last. It shows Library until another view has been opened.Nothing new was added to
bridge.js: the click handler is attached in JS the first time the button is updated.Testing
npm test: 2626 passed, 0 failed.tests/aux-window.test.jsadds five tests:.tab-bar) holds the border;npm run buildsucceeds.🤖 Generated with Claude Code
https://claude.ai/code/session_01HPVKDwzQWjyqE1nApBQt6s
Generated by Claude Code