bluzky / salad_ui

Phoenix Liveview component library inspired by shadcn UI
https://salad-storybook.fly.dev/welcome
MIT License
414 stars 24 forks source link

2 issues with the dialog component on ipad safari #17

Closed joepstender closed 2 months ago

joepstender commented 3 months ago

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 commented 3 months ago

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

joepstender commented 3 months ago

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

kamaroly commented 3 months ago

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

joepstender commented 3 months ago

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

bluzky commented 3 months ago

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 commented 3 months ago

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

bluzky commented 3 months ago

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

joepstender commented 3 months ago

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 commented 3 months ago

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 commented 3 months ago

Yeah, I suppose we can close this issue.