fix(web): keep workspace card action hover uniform - #12766
Merged
maria-rcks merged 1 commit intoSep 20, 2026
Merged
maria-rcks merged 1 commit into
maria-rcks merged 1 commit into
Conversation
Contributor
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This is a narrowly scoped visual fix that standardizes hover and pressed backgrounds for existing thread-details controls without changing interaction behavior or broader product defaults. The remaining comment requests a component-variant refactor for consistency but does not identify a functional defect. You can add or adjust custom eligibility rules. Learn more. |
maria-rcks
merged commit Sep 20, 2026
37671bd
into
pingdotgg:t3code/codex-turn-mapping
26 checks passed
juliusmarminge
pushed a commit
that referenced
this pull request
Sep 21, 2026
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.
Workspace card split actions added a second background when hovering their right-hand button. Keep both halves transparent so the row supplies one hover color, matching the branch selector. Apply the same shared style to PR checks.
Covers editor, project script, Git, PR action, and lineage split controls in the inline card and popover. This changes panel styles only; toolbar controls and action behavior stay the same.
Verified: 123 focused tests, web typecheck and build, targeted lint and formatting. Checked hover and pressed backgrounds in the real app in dark/light themes, at 1280px and 900px, and opened the action menus. The fixture did not expose every PR/lineage state; those use the shared styles reviewed here.
Before and after show the same Actions arrow hover. Other app content is hidden and the card is enlarged for clarity.
Model: GPT-6. Harness: Codex.