-
Notifications
You must be signed in to change notification settings - Fork 525
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
[v3] Modal always render fullscreen on mobile #2463
Comments
Hi, there is the specific example with the close button. |
By now all the docs example should already include a close button by default, so if we have missed any feel free to call them out! 🙏
@gioboa Nuxt UI v2 uses
Indeed it should not (that's why there's a dedicated |
Ah, my bad! It is working as intended. This is actually a long term debate, all the way back to the beginning of Nuxt UI v2. The three main proposed variants for mobile/small screens are:
a definite choice hasn't been picked, yet In the meantime you could edit your specified modal like so: <template>
<UModal
:ui="{
content: 'max-w-lg w-[90vw] h-auto rounded-[calc(var(--ui-radius)*2)] shadow-lg ring',
}"
>
<!-- content -->
</UModal>
</template> or project wide via export default defineAppConfig({
ui: {
modal: {
variants: {
fullscreen: {
false: {
content: 'max-w-lg w-[90vw] h-auto rounded-[calc(var(--ui-radius)*2)] shadow-lg ring',
},
},
},
},
},
}) |
Environment
MacOS 15.0.1 / Arc 1.63.0
Also tested on iOS 18.0.1 / Safari
Version
v3.0.0-alpha.7
Reproduction
https://ui3.nuxt.dev/components/modal
Description
In version 3.0-alpha, I noticed that modals are now displayed in fullscreen mode on mobile devices / small screens by default. This can be reproduced with the examples in the documentation.
I don't expect this to be intended as default. Also in the first example in the modal documentation it's hard to leave the dialoge, as there is no button for closing.
Additional context
Logs
No response
The text was updated successfully, but these errors were encountered: