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

[EuiCollapsibleNavBeta] Resizing away from small screen doesn't restore original uncollapsed state #7318

Closed
eokoneyo opened this issue Oct 26, 2023 · 2 comments · Fixed by #7319
Assignees
Labels
bug ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible

Comments

@eokoneyo
Copy link
Contributor

Describe the bug

As a user when I resize a page that implements EuiCollapsibleNavBeta with it's nav "uncollapsed", from a standard desktop viewport to tablet viewport the nav remains as is. On resizing the page viewport to mobile size the side nav is hidden, which is quite expected, however on resizing the viewport away from the mobile to tablet, one would notice that the side nav that was previously uncollapsed, is now collapsed, and remains so when the viewport is extended to desktop size.

Relates to elastic/kibana#169355

Environment and versions

  • EUI version: 89.1.0

To Reproduce

  1. Expand the nav
  2. Reduce the screen size
  3. Expand the screen size

See the nav is in collapsed state fully visible

Expected behavior

  1. Expand the nav
  2. Reduce the screen size - the side nav should collapse (mobile view)
  3. Expand the screen size - the side nav should be expanded (back to the state in step 1)

Minimum reproducible sandbox
As of version 89.1.0 this is reproducible running the eui storybook.

Screenshots

Screen.Recording.2023-10-26.at.11.38.40.mov

Additional context
Add any other context about the problem here.

@eokoneyo eokoneyo added bug ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible labels Oct 26, 2023
@cee-chen cee-chen self-assigned this Oct 26, 2023
@cee-chen
Copy link
Member

I honestly consider this a feature request, not a bug (again, just given the incredibly low number of end users who actually resize their screens), but maybe I'm quibbling :)

@eokoneyo
Copy link
Contributor Author

I honestly consider this a feature request, not a bug (again, just given the incredibly low number of end users who actually resize their screens), but maybe I'm quibbling :)

Thanks for picking it up still :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible
Projects
None yet
2 participants