Skip to content

Fix Windows thread edge background#4493

Open
SunkenInTime wants to merge 4 commits into
pingdotgg:mainfrom
SunkenInTime:agent/windows-thread-right-edge-background
Open

Fix Windows thread edge background#4493
SunkenInTime wants to merge 4 commits into
pingdotgg:mainfrom
SunkenInTime:agent/windows-thread-right-edge-background

Conversation

@SunkenInTime

@SunkenInTime SunkenInTime commented Jul 25, 2026

Copy link
Copy Markdown

Summary

Fixes a dark strip that could briefly appear along the right edge of the window when switching threads on Windows.

Electron can expose the window background for a moment while the page repaints. This change maps that surface to the app's existing --background color, so it stays aligned in both light and dark mode.

Screenshots

Before After
Before After

Verification

  • formatting, lint, and web typecheck
  • tested thread switching in the Windows Electron app

Note

Low Risk
Small theme/chrome styling changes with no auth, data, or API impact; limited to initial paint and theme-color sync behavior.

Overview
Fixes a brief dark strip on the right edge when switching threads in the Windows Electron app by keeping the exposed window background aligned with the app surface during repaints.

Dark mode base moves from #161616 to #0a0a0a in index.html (theme-color meta tags, boot script, and html.dark body) so early paint matches the design token used elsewhere.

Browser chrome sync in syncBrowserChromeTheme no longer samples main[data-slot='sidebar-inset'] / sidebar inner / body backgrounds. It sets html and body to var(--app-chrome-background) and derives the dynamic theme-color meta from that resolved color, matching the existing --app-chrome-background--background wiring in CSS.

Reviewed by Cursor Bugbot for commit 1baad2b. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Fix dark mode background color for Windows thread edge

  • Updates the dark mode background color from #161616 to #0a0a0a in index.html, affecting the CSS, boot script constant, and theme-color meta tags.
  • Reworks syncBrowserChromeTheme in useTheme.ts to derive the browser chrome color from a new --app-chrome-background CSS variable on the root element, replacing the previous logic that read color from a sidebar or main surface element.

Macroscope summarized 1baad2b.

@coderabbitai

coderabbitai Bot commented Jul 25, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f46f9a7c-c579-4982-89c7-802d92b5cec3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Jul 25, 2026
@SunkenInTime
SunkenInTime marked this pull request as ready for review July 25, 2026 17:44

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 7d8709c. Configure here.

Comment thread apps/web/src/hooks/useTheme.ts
@macroscopeapp

macroscopeapp Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved

Simple UI fix that changes a dark background color value (#161616 → #0a0a0a) and simplifies theme color syncing logic by using a CSS variable instead of querying specific DOM elements. Changes are purely visual with no runtime behavior impact beyond styling.

You can customize Macroscope's approvability policy. Learn more.

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

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant