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

Bug when using tabs inside a dialog with safari browser only #1629

Open
robthepaper opened this issue Jan 17, 2025 · 1 comment
Open

Bug when using tabs inside a dialog with safari browser only #1629

robthepaper opened this issue Jan 17, 2025 · 1 comment

Comments

@robthepaper
Copy link

robthepaper commented Jan 17, 2025

Describe the bug

Hi, does anyone encounter this bug on ios (I tested on iphone and ipad) ?
When trigger the dialog to open, i can only change the tabs one time thenit's like frozen, cannot change tabs anymore.

On Chrome works well

Reproduction

Here is the repo
it works on Chrome
but i couldn't test it on my ipad/safari

Can someone have a look please?

https://stackblitz.com/edit/shadcn-svelte-8fzwgdq8?file=src%2Froutes%2F%2Bpage.svelte

Logs

System Info

System:
    OS: Windows 10 10.0.18362
    CPU: (4) x64 Intel(R) Core(TM) i5-7200U CPU @ 2.50GHz
    Memory: 6.71 GB / 23.88 GB
  Binaries:
    Node: 20.12.0 - C:\Program Files\nodejs\node.EXE     
    Yarn: 1.22.19 - C:\Program Files\nodejs\yarn.CMD     
    npm: 10.5.0 - C:\Program Files\nodejs\npm.CMD        
    pnpm: 9.4.0 - C:\Program Files\nodejs\pnpm.CMD       
  Browsers:
    Chrome: 131.0.6778.265
    Edge: Spartan (44.18362.449.0)
    Internet Explorer: 11.0.18362.1
  npmPackages:
    @sveltejs/kit: ^2.9.0 => 2.9.0
    bits-ui: ^0.21.11 => 0.21.11
    cmdk-sv: ^0.0.17 => 0.0.17
    lucide-svelte: ^0.399.0 => 0.399.0
    mode-watcher: ^0.3.1 => 0.3.1
    svelte: 5.9.0 => 5.9.0
    svelte-sonner: ^0.3.24 => 0.3.24
    sveltekit-superforms: ^2.21.1 => 2.21.1
    vaul-svelte: ^0.3.1 => 0.3.1

Severity

annoyance

@huntabyte
Copy link
Owner

You should be using bits-ui@next if you're using Svelte 5+.

As well as vaul-svelte@next and you can replace cmdk-sv with Bits UI's Command component:

https://next.bits-ui.com/docs/components/command

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

2 participants