Skip to content

Fix window pos - #332

Merged
eristhea merged 4 commits into
QuiltMC:develop/2.8from
supersaiyansubtlety:fix-window-pos
Dec 12, 2025
Merged

eristhea merged 4 commits into
QuiltMC:develop/2.8from
supersaiyansubtlety:fix-window-pos

Conversation

@supersaiyansubtlety

@supersaiyansubtlety supersaiyansubtlety commented Nov 9, 2025 •

Copy link
Copy Markdown
Member

based on #307 because it adds Utils::clamp rebased


I was testing #272 and I think the 2.1+ behavior described was because the window was off-screen:

2.1+: Creates an empty window that can't be interacted with (Can't maximize, bring to front, has no title, has no preview when hovering over the windows dock bar icon)

Though the window preview was blank, not missing or transparent:
enigma-272-config-dock-preview

The config attached to #272 has this:

# How big the Enigma window will open, in pixels.
# default: Vec2i[x=1024, y=576]
[window_size]
	x = 1936
	y = 1056

# The position the top-left corner of Enigma's window will be the next time it opens, in pixels.
# default: Vec2i[x=0, y=0]
[window_pos]
	x = -32000
	y = -32000

@supersaiyansubtlety supersaiyansubtlety self-assigned this Nov 9, 2025
@supersaiyansubtlety supersaiyansubtlety added bug Something isn't working user interface affects the enigma ui labels Nov 9, 2025
@supersaiyansubtlety supersaiyansubtlety linked an issue Nov 9, 2025 that may be closed by this pull request
@supersaiyansubtlety
supersaiyansubtlety force-pushed the fix-window-pos branch 4 times, most recently from 38e546b to 5290637 Compare November 9, 2025 20:32
@supersaiyansubtlety
supersaiyansubtlety marked this pull request as ready for review November 9, 2025 20:46
@supersaiyansubtlety

Copy link
Copy Markdown
Member Author

ready!

@supersaiyansubtlety
supersaiyansubtlety changed the base branch from develop/2.7 to develop/2.8 November 18, 2025 22:31

@eristhea eristhea left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good! i'm unable to reproduce the off-screen issue on KDE, but the resizing behaviour is much nicer with this PR

@eristhea
eristhea merged commit c25687b into QuiltMC:develop/2.8 Dec 12, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working user interface affects the enigma ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

broken configs can cause UI to silently fail

2 participants