Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix mobile mobile driver absolute position computation for padded child windows #4857

Merged
merged 6 commits into from
May 23, 2024

Conversation

toaster
Copy link
Member

@toaster toaster commented May 21, 2024

Description:

This PR fixes a bug where the absolute position of an element is computed wrong by the mobile driver for child windows. The chrome box is not taken into account when the window is padded and therefore the position is not adjusted accordingly.
This affects the position of drop-downs.

Checklist:

  • Tests included.
  • Lint and formatter run with no errors.
  • Tests all pass.

This improves the handling of the window-head offset.
…mobileCanvas#sizeContent

This simplifies the next diff where the offset computation is extracted.
This changes the local decision for the full-width window-head
from “chrome box contains more than one element”
to “chrome box displaces content”.
This is still not a total straight forward decision but not worse then
the old one which will survive in the extracted offset computation.
This is also used as height for a displacing chrome box to save another `MinSize()` call.
… child windows

This led to misplaced pop-ups.
@toaster toaster requested a review from andydotxyz May 21, 2024 13:39
@coveralls
Copy link

Coverage Status

coverage: 65.322% (+0.3%) from 65.068%
when pulling eea39e2 on toaster:bugfix/mobile_absolute_position
into edb9b28 on fyne-io:develop.

Copy link
Member

@andydotxyz andydotxyz left a comment

Choose a reason for hiding this comment

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

Great thanks for this fix

@toaster toaster merged commit 09c078f into fyne-io:develop May 23, 2024
12 checks passed
@toaster toaster deleted the bugfix/mobile_absolute_position branch May 26, 2024 10:58
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.

None yet

3 participants