Skip to content

fix: fix off-by-one error when calculating popup panel dimensions (fi…#5314

Open
skdas20 wants to merge 1 commit intojesseduffield:masterfrom
skdas20:fix/popup-dimension-off-by-one
Open

fix: fix off-by-one error when calculating popup panel dimensions (fi…#5314
skdas20 wants to merge 1 commit intojesseduffield:masterfrom
skdas20:fix/popup-dimension-off-by-one

Conversation

@skdas20
Copy link

@skdas20 skdas20 commented Feb 22, 2026

Fixes an off-by-one error in popup panel dimension calculations when nested popups are displayed. The panel height calculation was adding an extra pixel, causing improper spacing and positioning. This one-line fix corrects the calculation to ensure popups display at the correct dimensions.

Fixes #5312

@stefanhaller
Copy link
Collaborator

Huh, what is this? If you noticed a bug in #5312, that's great; there was indeed one (fixed by fa6f3ea if you want to have a look), but this PR doesn't fix it, and making a PR against master doesn't make sense anyway as long as #5312 isn't even merged yet.

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.

2 participants