-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
2 issues with the dialog component on ipad safari #17
Comments
Thanks for reporting, I don't have an iPad. Let me try to reproduce on desktop |
Here’s a recorded screenshot :-) if I can do anything let me know. https://vimeo.com/966712369 |
@joepstender are you referring to the redirect when a user dismisses the modal/ dialog? |
Yes, by tapping on the X in the top right side corner, or by tapping anywhere else on the screen. |
Oh, it's because I open a new link on_cancel event. Updated the example. |
Now animation is handled by tailwind-animate and this issue would be fixed in PR #22 |
@joepstender this issue has been fixed. Please verify on your iPad. |
Indeed the redirection issue has been fixed on ipad! I still have to tap a second time on the button to open the modal though, which is not the case on https://ui.shadcn.com/docs/components/dialog. There only only one tap is enough. That issue might be hard to debug without an actual ipad… |
I suspected that is storybook issue. |
Yeah, I suppose we can close this issue. |
On ipad (with safari):
This is on https://salad-storybook.fly.dev/core_components/dialog
The text was updated successfully, but these errors were encountered: