Bug: NavList SubNav active state

[gracepark]

Description

Hi there 👋 I'm running into an issue where if a NavList SubNav is closed and there are no aria-current children that are active, the active state is still on the SubNav level.

Screen.Recording.2023-07-27.at.9.53.49.AM.mov

Please let me know if I'm misunderstanding, but I would have expected that if all the aria-current in the children were false there would be no active marker on the parent of those children. Thank you so much!

Steps to reproduce

1. Create a NavList with multiple parents that have children
2. Set aria-current based on the child route being equal to the current page
3. Open and close different parents after clicking on different children

Version

v35.26.1

Browser

Chrome

[snehaljos]

Hi, Can I work on this issue?

[siddharthkp]

Related + more context: https://github.com/github/primer/issues/2512

[harshalkaigaonkar]

Hey @joshblack / @siddharthkp , i would like to take up this issue, if it's open.

[siddharthkp]

@gracepark Can we close this as completed by #3698

[gracepark]

Yes we can @siddharthkp! Closing now, thank you.