Replies: 1 comment
-
|
I'll take a look tomorrow and answer your questions as it's my free day but thank you for the PR |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Why this feels missing
OmniWM already lets outer margins follow each display, while inner gaps remain global. That makes the General spacing model feel incomplete on mixed-display setups. A gap that feels balanced on a large external display can feel oversized on a built-in panel, even though both belong to the same workspace configuration.
Proposed interaction
Extend the existing Configuration Scope control in General so it governs both Layout > Inner Gaps and Outer Margins:
0remains a valid explicit override.This reuses the existing
monitorGapOverridesmodel instead of adding another settings path.Layout behavior
Compatibility and IPC
The persisted
innerGapvalue would be optional, so existing configurations need no migration and continue inheriting the current global gap. The display IPC query could expose the resolved value through an optionalinner-gapfield. Gap changes would also emit the existingdisplay-changedevent so subscribers can refresh resolved display values.Implementation status
I have a clean, standalone implementation against current
main(v0.5.6) ready to submit. It passes the full local test suite and has been validated with distinct gaps on two physical displays in both Niri and Dwindle. I have opened the working implementation as draft PR #477 so it is available for inspection while the direction is discussed. I am also happy to adjust the interaction or IPC scope before marking it ready for review.Questions
inner-gapdisplay query field ship with the settings change, or would you prefer it separately?Beta Was this translation helpful? Give feedback.
All reactions