supabase / supabase / 29269330786

74%

push

github

fix: ensure sidebar has valid html (#47826)

## Problem

The sidebar in both org or project pages has invalid html, preventing
screen reader users to understand them.
See https://dequeuniversity.com/rules/axe/4.12/list

Besides, we were using some shadcn components incorrectly (groups inside
menus when it should be the other way around)

## Solution

- Invert group/menu relation so that we don't have `div` inside `ul`
- No visual changes
- No functional changes

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Style**
* Improved sidebar navigation by splitting routes into clearer sections
(Home/Tools, Products, Other routes, and Settings).
* Updated the advisors link so its active indicator renders directly
within the navigation item, while keeping the existing
disabled/error/warning behavior.
* Adjusted organization navigation layout for more consistent spacing
and alignment. Animations remain smooth.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

1198 of 1655 branches covered (72.39%)

Branch coverage included in aggregate %.

1771 of 2359 relevant lines covered (75.07%)

246.32 hits per line