Skip to content

feat(settings): add toggle for dialog backdrop blur#46

Merged
johannesjo merged 1 commit intojohannesjo:mainfrom
Maskar:pr/dialog-blur-setting
Mar 28, 2026
Merged

feat(settings): add toggle for dialog backdrop blur#46
johannesjo merged 1 commit intojohannesjo:mainfrom
Maskar:pr/dialog-blur-setting

Conversation

@Maskar
Copy link
Copy Markdown
Contributor

@Maskar Maskar commented Mar 28, 2026

Summary

New setting to enable/disable the backdrop blur effect when dialogs are open. Found in Settings > Behavior.

Motivation

The blur effect can be distracting when monitoring terminal output behind a dialog. This lets users disable it while keeping the dark overlay for focus.

Details

  • New dialogBlur boolean in store (persisted, defaults to true)
  • CSS split: .dialog-overlay has no blur, .dialog-overlay.dialog-blur adds it
  • Dialog.tsx conditionally applies the blur class based on the setting

Test plan

  • Default: dialogs blur the background (existing behavior)
  • Uncheck setting: dialogs show dark overlay without blur
  • Setting persists across restarts

@Maskar
Copy link
Copy Markdown
Contributor Author

Maskar commented Mar 28, 2026

Also, it helps to see how the themes and font changes reflect on the actual interface.

@johannesjo
Copy link
Copy Markdown
Owner

Thank you very much! Lets just make it the default. Not a big fan of adding settings for everything :)

The blur effect can be distracting when monitoring terminal output
behind a dialog. Remove it while keeping the dark overlay for focus.
@Maskar Maskar force-pushed the pr/dialog-blur-setting branch from 1e053bc to fa954b9 Compare March 28, 2026 16:00
@Maskar
Copy link
Copy Markdown
Contributor Author

Maskar commented Mar 28, 2026

Simplified — now just removes the blur by default, no setting. Single line change.

@johannesjo
Copy link
Copy Markdown
Owner

Thank you!

@johannesjo johannesjo merged commit 38a16b5 into johannesjo:main Mar 28, 2026
2 checks passed
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