Skip to content

Polish Buzz sidebar theme#1671

Merged
wesbillman merged 4 commits into
mainfrom
kennylopez-buzz-debug-panel
Jul 9, 2026
Merged

Polish Buzz sidebar theme#1671
wesbillman merged 4 commits into
mainfrom
kennylopez-buzz-debug-panel

Conversation

@klopez4212

@klopez4212 klopez4212 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add the finished Buzz sidebar translucency treatment with native macOS vibrancy
  • Keep the sidebar gradient as one continuous layer across the nav, header, footer, rail, and gutters
  • Tune hover opacity and align the search row with the channel header chrome

Snapshots

Buzz light sidebar

Buzz dark sidebar

Testing

  • pnpm check
  • pnpm typecheck
  • cargo check --manifest-path desktop/src-tauri/Cargo.toml
  • pre-push hooks: rust-tests, desktop-test, mobile-test, desktop-tauri-test
  • pnpm exec playwright test tests/e2e/buzz-theme-screenshots.spec.ts --project=smoke -g "buzz (light|dark) sidebar gradient"

@klopez4212 klopez4212 marked this pull request as ready for review July 9, 2026 13:44

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e56ca29174

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread desktop/src-tauri/Cargo.toml Outdated
Comment thread desktop/src-tauri/src/commands/window_vibrancy.rs
klopez4212 added a commit that referenced this pull request Jul 9, 2026
@klopez4212 klopez4212 requested a review from wesbillman July 9, 2026 13:58
… re-apply

Pin window-vibrancy to 0.6 so it unifies with the 0.6.0 that Tauri
2.11.2 already links, instead of pulling a second 0.7.1 copy. Both
versions define the same Objective-C tagged NSVisualEffectView classes,
which risks duplicate-symbol link failures in macOS release bundles.

Clear any existing vibrancy view before applying a new material. Since
apply_vibrancy appends a tagged view each call while clear_vibrancy
removes only one, repeated Buzz enables (theme switches, follow-system
flips) would otherwise stack views and leave a stale blur behind when
switching back to a non-Buzz theme.

Signed-off-by: npub13fn4ahfnvaa2qwylvegdgeajqs0mph6v4qsw4jcqnw4mjh3hzh2quuucm5 <8a675edd33677aa0389f6650d467b2041fb0df4ca820eacb009babb95e3715d4@sprout-oss.stage.blox.sqprod.co>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2bafcf40ae

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread desktop/src/shared/theme/ThemeProvider.tsx Outdated
The Buzz translucent treatment (transparent root/body + semi-transparent
sidebar gradient) relies on the native macOS NSVisualEffectView vibrancy
layer painting behind the webview. On Windows/Linux set_window_vibrancy is
a no-op, but the window is transparent globally, so a transparent root
showed raw desktop content through the UI. Gate the data-buzz-translucent
marker and transparent root background to macOS; other platforms fall back
to the opaque Buzz gradient with the normal bg-background body fill.

Signed-off-by: npub13fn4ahfnvaa2qwylvegdgeajqs0mph6v4qsw4jcqnw4mjh3hzh2quuucm5 <8a675edd33677aa0389f6650d467b2041fb0df4ca820eacb009babb95e3715d4@sprout-oss.stage.blox.sqprod.co>
@klopez4212 klopez4212 force-pushed the kennylopez-buzz-debug-panel branch from 2b0fe79 to 075c1fc Compare July 9, 2026 14:29
@klopez4212 klopez4212 requested a review from a team July 9, 2026 14:29
@wesbillman wesbillman merged commit cdba2a0 into main Jul 9, 2026
25 checks passed
@wesbillman wesbillman deleted the kennylopez-buzz-debug-panel branch July 9, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants