feat(web): clearer icons for the workspace panel controls - #12838
Merged
maria-rcks merged 2 commits intoSep 21, 2026
Merged
maria-rcks merged 2 commits into
maria-rcks merged 2 commits into
Conversation
Contributor
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This single-file change only replaces an existing panel-control icon while preserving the toggle’s state, interactions, labels, and panel behavior. It introduces no schema, configuration, deployment, or sensitive-data changes. You can add or adjust custom eligibility rules. Learn more. |
macroscopeapp
Bot
dismissed
their stale review
September 21, 2026 04:34
Dismissing prior approval to re-evaluate 63d995b
maria-rcks
merged commit Sep 21, 2026
283fe9c
into
pingdotgg:t3code/codex-turn-mapping
25 checks passed
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 21, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 23, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 23, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 23, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 23, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 23, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 23, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 24, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 24, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 24, 2026
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 25, 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
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.
replaces the filter glyph on thread details with
square-menu. the terminal drawer keeps its originalpanel-bottom;PanelRightandMaximize2/Minimize2stay unchanged.original candidate galleries below use the actual control components. each candidate appears at 16px in the app's ghost toggle, unpressed then pressed.
runner-ups:
Info: readable, but suggests passive information; thread details also contains workspace and git actions.LayoutList: conveys organized content, but the small boxes add visual noise at 16px.PanelRightOpen: suggests a one-way opening action and resembles the adjacent right-panel toggle.Terminal/SquareTerminal: considered for the terminal drawer; the originalPanelBottomis retained.verification: web typecheck and scoped lint passed on blacksmith for
63d995b018. captured the actual unpressed control row in dark and light. details and right-panel toggles open and close. the terminal drawer opens and closes with the correct pressed state; terminal session content remains unavailable in the copied dev state. no matching action icons needed changes in the command palette, keybinding help, mobile, or user docs. desktop shares this component; native desktop was not separately exercised. provider adapters and contracts are unchanged.Written by gpt-6-astra (Codex) in T3 Code