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

[Accordion] layout shift with initial open items. #2891

Open
jinwoo0103 opened this issue May 10, 2024 · 2 comments
Open

[Accordion] layout shift with initial open items. #2891

jinwoo0103 opened this issue May 10, 2024 · 2 comments

Comments

@jinwoo0103
Copy link

jinwoo0103 commented May 10, 2024

Bug report

Current Behavior

Can check layout shift when click (Command + Shift + R) on Accordion documentation example

2024-05-10.5.15.17.mov

Expected behavior

It should be initially open when defaultValue / value has some open accordion items

Reproducible example

Can check layout shift when click (Command + Shift + R) on Accordion documentation example

Suggested solution

Since accordion is built on collapsible (might be right?),
collapsible's default open determination should consider defaultValue / value, not just false.

Additional context

It seems icon is displaying correctly...? or maybe it is too fast...

Your environment

Since it can be checked on chrome, I attach my mac and chrome info only

  • macOS(m1 pro, Sonoma 14.4.1)
  • Chrome(124.0.6367.158 (arm64))
@thexpand
Copy link

It's not Chrome-specific, it's on Safari as well.

M3 Max, macOS 14.3.1
Safari Version 17.3.1 (19617.2.4.11.12)

@alexpuertasr
Copy link

How is this going? We have the same issue in our nav that is using this component

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