Skip to content

Constrain Omawrite's save dialog - #9134

Closed
yashranaway wants to merge 1 commit into
omacom:quattrofrom
yashranaway:fix/omawrite-save-dialog-size
Closed

Constrain Omawrite's save dialog#9134
yashranaway wants to merge 1 commit into
omacom:quattrofrom
yashranaway:fix/omawrite-save-dialog-size

Conversation

@yashranaway

Copy link
Copy Markdown
Contributor

Keeps Omawrite's save dialog inside scaled laptop displays. Fixes #9046.

<<<< AI wording below >>>>

Problem

Omawrite can create a 1231x950 save dialog on a 1280x832 logical display, clipping its title and action controls beyond the monitor work area.

Fix

Match only Omawrite's Save File window and apply the reporter-validated floating, centered 875x600 geometry.

Verification

  • bash -n test/shell.d/omawrite-window-rule-test.sh
  • git diff --check
  • The semantic Lua test checks the emitted class, title, float, center, and size properties.
  • The focused test cannot run here because lua is not installed.
  • Live verification at 2x scale is pending, so this PR is a draft.

@yuzuwoeser

yuzuwoeser commented Aug 30, 2026

Copy link
Copy Markdown

Thanks for looking into this. The 875x600 rule was only a workaround tested on my display, so I’m unsure it should become the default fix for everyone. Treat this as a mitigation and investigate the sizing issue in Omawrite/Qt instead?

@yashranaway

Copy link
Copy Markdown
Contributor Author

Agreed, 875x600 is too display-specific, so I am moving this fix into Omawrite's file-dialog implementation.

@yashranaway

Copy link
Copy Markdown
Contributor Author

Replaced by omacom/omawrite#39, which fixes the dialog in Omawrite instead of applying a display-specific Hyprland rule.

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.

Omawrite Save File dialog exceeds 2x logical display bounds

2 participants