Skip to content

fix: make settings dialog scrollable - #4

Open
60dage wants to merge 1 commit into
manustays:mainfrom
60dage:bugfix/settings-dialog-overflow
Open

fix: make settings dialog scrollable#4
60dage wants to merge 1 commit into
manustays:mainfrom
60dage:bugfix/settings-dialog-overflow

Conversation

@60dage

@60dage 60dage commented Aug 23, 2026

Copy link
Copy Markdown

Summary

  • constrain the Settings dialog to the available popover height
  • enable vertical scrolling when its content exceeds that height
  • reuse the same max-h-[88vh] / overflow-y-auto pattern already used by ServiceForm

The Agent Radar hook rows can make Settings taller than the webview. Since the dialog is vertically centered and the root clips overflow, its top and bottom could become inaccessible.

Closes #3

 2026-08-23 16 25 35  2026-08-23 16 25 43

Testing

  • npm test — 8 passed
  • npx tsc --noEmit
  • npm run build
  • cargo test --manifest-path src-tauri/Cargo.toml — 95 passed
  • cargo build --manifest-path src-tauri/Cargo.toml

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.

Settings dialog is clipped vertically in the popover

1 participant