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

2 issues with the dialog component on ipad safari #17

Closed
joepstender opened this issue Jun 24, 2024 · 10 comments
Closed

2 issues with the dialog component on ipad safari #17

joepstender opened this issue Jun 24, 2024 · 10 comments

Comments

@joepstender
Copy link

On ipad (with safari):

  1. The modal won’t open when you tap on it the first time. You have to tap it twice.
  2. After closing the modal it redirects to a route that isn’t found.

This is on https://salad-storybook.fly.dev/core_components/dialog

@bluzky
Copy link
Owner

bluzky commented Jun 24, 2024

Thanks for reporting, I don't have an iPad. Let me try to reproduce on desktop

@joepstender
Copy link
Author

Here’s a recorded screenshot :-) if I can do anything let me know. https://vimeo.com/966712369

@kamaroly
Copy link
Collaborator

@joepstender are you referring to the redirect when a user dismisses the modal/ dialog?

@joepstender
Copy link
Author

Yes, by tapping on the X in the top right side corner, or by tapping anywhere else on the screen.

@bluzky
Copy link
Owner

bluzky commented Jun 25, 2024

Oh, it's because I open a new link on_cancel event. Updated the example.
The other issue happened on Safari for Mac too. I'm investigating

@bluzky
Copy link
Owner

bluzky commented Jun 25, 2024

Now animation is handled by tailwind-animate and this issue would be fixed in PR #22

@bluzky
Copy link
Owner

bluzky commented Jun 26, 2024

@joepstender this issue has been fixed. Please verify on your iPad.
https://salad-storybook.fly.dev/salad_ui_component/dialog

@joepstender
Copy link
Author

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…

@bluzky
Copy link
Owner

bluzky commented Jun 26, 2024

I suspected that is storybook issue.
This issue occurred on my iPhone, but when I switch to playground, it doesn't happen any more

@joepstender
Copy link
Author

Yeah, I suppose we can close this issue.

@bluzky bluzky closed this as completed Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants