Skip to content

Remove deprecated window opacity controls#256

Merged
BunsDev merged 1 commit into
mainfrom
okcode/remove-window-opacity
Apr 5, 2026
Merged

Remove deprecated window opacity controls#256
BunsDev merged 1 commit into
mainfrom
okcode/remove-window-opacity

Conversation

@BunsDev

@BunsDev BunsDev commented Apr 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove the deprecated window opacity setting from desktop, web, and shared IPC contracts.
  • Delete the renderer and main-process wiring that applied opacity to the native window.
  • Keep the desktop window explicitly reset to full opacity on creation.
  • Clean up the settings UI and localized strings for the removed control.
  • Add a regression test to ensure persisted settings drop legacy windowOpacity values.
  • Simplify the PR actions menu rendering by removing the extra group wrapper.

Testing

  • bun fmt
  • bun lint
  • bun typecheck
  • apps/web/src/appSettings.test.ts: verifies legacy windowOpacity is stripped during schema decode

- Drop deprecated window opacity setting from desktop and web UI
- Keep persisted settings and IPC contracts aligned
@vercel

vercel Bot commented Apr 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 5, 2026 6:46am
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 5, 2026 6:46am

@github-actions github-actions Bot added size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Apr 5, 2026
@BunsDev BunsDev merged commit 78d5757 into main Apr 5, 2026
10 of 11 checks passed
@BunsDev BunsDev deleted the okcode/remove-window-opacity branch April 6, 2026 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 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.

1 participant