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

[BUG] Popup content sized incorrectly When WidthRequest is used with Margin/Padding #1716

Open
2 tasks done
sjordanGSS opened this issue Feb 27, 2024 · 1 comment
Open
2 tasks done
Labels
area/views Issue/Discussion/PR that has to do with Views bug Something isn't working unverified

Comments

@sjordanGSS
Copy link

sjordanGSS commented Feb 27, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Did you read the "Reporting a bug" section on Contributing file?

Current Behavior

When a dialog is sized by the content having WidthRequest set, or by setting Size on the popup itself, applying Padding or a Margin to the contents causes views within the content to be sized incorrectly and then clipped on the righthand side

image

Removing the Padding and Margin from the VerticalStackLayout causes the Label to appear as expected.

Expected Behavior

The content (in the repro, a Label) should not be clipped and the contents should wrap correctly, as it does on Android 13 and Android 14

image

Steps To Reproduce

  • Open the attached repository and run it on a device/emulator running Android 11, Android 12 or Android 12.1
  • Tap the "Click me!" button to open the dialog

Link to public reproduction project repository

https://github.com/sjordanGSS/maui-popup-padding

Environment

- .NET MAUI CommunityToolkit: 7.0.1
- OS: Android 11, Android 12, Android 12.1
- .NET MAUI: 8.0.7

Anything else?

This issue does not appear to affect Android 13 or Android 14. I have tested them using Android Emulator and could not reproduce it. I have not tested Android versions earlier than Android 11.

@sjordanGSS sjordanGSS added bug Something isn't working unverified labels Feb 27, 2024
@bijington
Copy link
Contributor

I wonder if #1683 fixes this

@vhugogarcia vhugogarcia added the area/views Issue/Discussion/PR that has to do with Views label Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/views Issue/Discussion/PR that has to do with Views bug Something isn't working unverified
Projects
None yet
Development

No branches or pull requests

3 participants