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(ui5-dialog): width on mobile is not bigger than the phone width #10255

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

LidiyaGeorgieva
Copy link
Contributor

Issue: When the base font in the 'html' element is changed to something different than the default 16px (for example 32px) the width of the dialog can become larger than the width of the phone's display. This is caused by the 'min-width' (20rem) of the dialog. For the dialog on phone it is recommended by the design to set the dialog's property 'stretch' to true to use the full screen size.

The solution: When we have stretched dialog on phone the 'min-width' should
not be applied (the width is 100%).

fixes: #10000
downport of #10199

…10199)

Issue: When the base font in the 'html' element is changed to something
different than the default 16px (for example 32px) the width
of the dialog can become larger than the width of the phone's
display. This is caused by the 'min-width' (20rem) of the dialog.
For the dialog on phone it is recommended by the design to set the dialog's
property 'stretch' to true to use the full screen size.

The solution: When we have stretched dialog on phone the 'min-width' should
 not be applied (the width is 100%).

fixes: #10000
downport of #10199
@LidiyaGeorgieva LidiyaGeorgieva merged commit 50a918d into release-1.24 Dec 2, 2024
8 checks passed
@LidiyaGeorgieva LidiyaGeorgieva deleted the ui5-dialog-stretch-min-width-1.24 branch December 2, 2024 06:37
@ui5-webcomponents-bot
Copy link
Collaborator

🎉 This PR is included in version v1.24.14 🎉

The release is available on v1.24.14

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants