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

[net7.0] Fix Modal page offset measuring for AdjustPan and AdjustResize #13401

Merged
merged 1 commit into from
Feb 16, 2023

Conversation

PureWeen
Copy link
Member

Backport of #12661 to net7.0

/cc @PureWeen


namespace Microsoft.Maui.Controls.Platform
{
internal class GenericGlobalLayoutListenerImproved : Object, ViewTreeObserver.IOnGlobalLayoutListener
Copy link
Member Author

Choose a reason for hiding this comment

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

For the backport I added this class to minimize the amount of code being changed.

For .NET8 I've just modified GenericGlobalLayoutListener and updated everywhere that uses that code.

@hartez hartez merged commit a8ca3b5 into net7.0 Feb 16, 2023
@hartez hartez deleted the backport/pr-12661-to-net7.0 branch February 16, 2023 20:40
@noque-lind
Copy link

noque-lind commented Feb 16, 2023

@hartez So happy this bug is fixed - and can't wait to get it running in our app. Do you know when the next service release is? Asking because we are supposed to release our .net maui app EOB next friday, and this is one of the last release blockers.

Kind regards Emil Lind

@PureWeen
Copy link
Member Author

@noque-lind you can try the nuget I have here

https://www.nuget.org/packages/PureWeen.Maui.FixesAndWorkarounds

If that still doesn't fix your modals then please log a bug here
https://github.com/PureWeen/ShanedlerSamples/issues

@github-actions github-actions bot locked and limited conversation to collaborators Dec 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants