Skip to content

Temporary images for workspace card fix - #110

Closed
Bil0000 wants to merge 2815 commits into
mainfrom
t3/fix-worktree-card-warnings
Closed

Bil0000 wants to merge 2815 commits into
mainfrom
t3/fix-worktree-card-warnings

Conversation

@Bil0000

@Bil0000 Bil0000 commented Oct 7, 2026

Copy link
Copy Markdown
Owner

The workspace card repeats the server version warning already shown above the composer. Split buttons also stack two hover fills, which gives each half a different shade.

Remove the duplicate warning and its old attention dot. Keep Update and Dismiss above the composer. Use one fill on the active half of each split row, including when its menu is open. The card audit found no other duplicate banners.

Verified with 89 focused tests, the web type check, scoped lint and format checks, and a real browser pass for both button halves, open menus, and keyboard use. This changes the shared web/desktop card. Native mobile is unchanged.

Before After
Duplicate warning and primary hover Single composer warning and primary hover
Chevron hover before Chevron hover after

Built with GPT-6.1-Sol High in the Codex harness. Audited with Claude Opus 5.5 High and GPT-6.1-Sol High Fast through T3 Code.

before-primary

after-primary

before-chevron

after-chevron

eimexdev and others added 30 commits October 4, 2026 02:44
…gg#15427)

Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
…15504)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…tements (pingdotgg#15411)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ailing (pingdotgg#15488)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
… wake (pingdotgg#15418)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ingdotgg#15551)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…tion (pingdotgg#15563)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
…ingdotgg#15505)

Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
… cards (pingdotgg#15506)

Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
maria-rcks and others added 24 commits October 6, 2026 22:34
…6661)

Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…g#9786)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…#9788)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…#10298)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…rver (pingdotgg#16718)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…g#16741)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…16752)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ngdotgg#16290)

Co-authored-by: spoukyii <61633921+spoukyii@users.noreply.github.com>
Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ices (pingdotgg#16631)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…tgg#16762)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…s of untracked files (pingdotgg#16771)

Co-authored-by: Braulio Oliveira <brauliobo@gmail.com>
Co-authored-by: Claude Sonnet 5.5 <noreply@anthropic.com>
…eep (pingdotgg#16760)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ingdotgg#16761)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ingdotgg#16782)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Oct 7, 2026
@Bil0000 Bil0000 closed this Oct 7, 2026
@Bil0000

Bil0000 commented Oct 7, 2026

Copy link
Copy Markdown
Owner Author

The full split row now uses the same hover fill as the worktree and branch rows. Both hover areas and open menus passed in dark and light mode.

One composer warning and uniform row hover

Chevron hover uses the same fill across the row

@Bil0000

Bil0000 commented Oct 7, 2026

Copy link
Copy Markdown
Owner Author

Focused crops of the existing browser captures for pingdotgg#16823. Captured from the local web dev build; the warning was reproduced with a client version override.

Duplicate card warning before

Workspace after the duplicate warning is removed

Primary hover before has two fills

Primary hover after has one fill

Chevron hover before has two fills

Chevron hover after has one fill

@Bil0000

Bil0000 commented Oct 7, 2026

Copy link
Copy Markdown
Owner Author

Evidence for pingdotgg#16823 at 3130480. Real screenshots, cropped only. The header before is user supplied from the installed client. After images are from the local web dev build with isolated state on a different machine.

User supplied installed-client before: separate machine and workspace rows

Local web dev build after: combined header, light theme

Local web dev build after: combined header, dark theme

Local web dev build before final hover fix: full Git row tint

Final local web dev build: secondary Git half hovered, light theme

Final local web dev build: primary Git half hovered, dark theme

@Bil0000

Bil0000 commented Oct 7, 2026

Copy link
Copy Markdown
Owner Author

Updated hover evidence for pingdotgg#16823. Real local web build captures of the same Project Actions split row, cropped without other changes. Before: 06406e1. After: d3f8e62098.

Before: Project Actions primary half hovered, dark theme

After: Project Actions secondary half hovered, light theme

After: Project Actions primary half hovered, dark theme

t3-hover-before-dark-06406e1

t3-hover-after-light-d3f8e62098

t3-hover-after-dark-d3f8e62098

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.