Skip to content

fix: restore initial dimensions and placeholder on application restart#708

Draft
Kasui92 wants to merge 1 commit intoabenz1267:masterfrom
omakasui:fix-race-condition
Draft

fix: restore initial dimensions and placeholder on application restart#708
Kasui92 wants to merge 1 commit intoabenz1267:masterfrom
omakasui:fix-race-condition

Conversation

@Kasui92
Copy link
Copy Markdown

@Kasui92 Kasui92 commented Mar 30, 2026

To be transparent, I got help from Claude Code for this fix, so consider this a suggestion rather than the definitive solution.

Unfortunately, I can't figure it out without it, and I'm still not sure if it's a problem with me (perhaps related to some configuration or GNOME) or a logical one.

I apologize if there are any suboptimal choices. I'll just leave the suggestion, hoping to address this difficulty.


When using walker as a --gapplication-service, closing and immediately reopening with different --maxheight/--minheight values causes the second invocation to be silently capped to the dimensions of the first.

Capture the true XML defaults once at window creation time and store them in WindowData.

This fix seems to work on GNOME 48 (Debian Trixie), but not on GNOME 46 (Ubuntu 24.04).

Closes #703

@Kasui92 Kasui92 force-pushed the fix-race-condition branch from 6060419 to 83f1d90 Compare March 31, 2026 11:00
@Kasui92 Kasui92 force-pushed the fix-race-condition branch from 0a87fa5 to 487e59f Compare March 31, 2026 11:03
@Kasui92
Copy link
Copy Markdown
Author

Kasui92 commented Mar 31, 2026

Updated.
I'm reverting the PR to draft, as it doesn't fully function.

@Kasui92 Kasui92 marked this pull request as draft March 31, 2026 11:12
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.

[Ubuntu/Debian]: window size parameters not applied correctly when service is reused quickly

1 participant