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

containsCurrentItem State Not Reset in NavList Component. #4721

Closed
Hanskrogh opened this issue Jul 7, 2024 · 3 comments
Closed

containsCurrentItem State Not Reset in NavList Component. #4721

Hanskrogh opened this issue Jul 7, 2024 · 3 comments
Labels

Comments

@Hanskrogh
Copy link

Description

When the navlist component includes a subnav and there is a current item within that subnav, the containsCurrentItem state is not being reset to false. As a result, the visual indication of the current item remains displayed until the component is fully re-rendered.

Steps to reproduce

  1. Add a subnav within the navlist component.
  2. Ensure there is a current item within the subnav.
  3. Observe that the containsCurrentItem state remains true even after the current item is no longer relevant.

Version

36.24.0

Browser

Chrome

@broccolinisoup
Copy link
Member

Hello @TylerJDev 👋 Would you be able to review the PR #4722 that @Hanskrogh pushed? Thanks!!

Copy link
Contributor

Hi! This issue has been marked as stale because it has been open with no activity for 180 days. You can comment on the issue or remove the stale label to keep it open. If you do nothing, this issue will be closed in 7 days.

@github-actions github-actions bot added the Stale label Jan 12, 2025
@lesliecdubs
Copy link
Member

Closing as stale. Please comment back or reopen if this requires additional attention.

@lesliecdubs lesliecdubs closed this as not planned Won't fix, can't repro, duplicate, stale Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants