[Tidy] Improve semantics of `NavBar` and `Accordion` - Githubissues

mckinsey / vizro

Vizro is a toolkit for creating modular data visualization applications.

https://vizro.readthedocs.io/en/stable/

Apache License 2.0

2.47k stars 109 forks source link

[Tidy] Improve semantics of `NavBar` and `Accordion` #393

Closed huong-li-nguyen closed 3 months ago

huong-li-nguyen commented 3 months ago

Description

Replace html.Div with dbc.Nav and dbc.Navbar (for context on why we should do the switch)

Screenshot

Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":
- I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.
- I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorized to submit this contribution on behalf of the original creator(s) or their licensees.
- I certify that the use of this contribution as authorized by the Apache 2.0 license does not violate the intellectual property rights of anyone else.
- I have not referenced individuals, products or companies in any commits, directly or indirectly.
- I have not added data or restricted code in any commits, directly or indirectly.