From c5ca30087d8ba180fbfc107e2c9539d554ba3e09 Mon Sep 17 00:00:00 2001 From: Besser Sehen Landshut Date: Mon, 24 Aug 2026 17:52:04 +0200 Subject: [PATCH] fix(ui): give the admin area a main landmark MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Follow-up to #1587, which covered the two layouts that carry the side navigation. The admin area has a layout of its own, sits directly under `pages/Layout` in the router, and was therefore not included. Same two changes as there: the wrapper around `Outlet` becomes `main`, the sidebar container becomes `nav`. Class names untouched, so nothing moves. It covers 28 routes. One correction to what #1587 says about this: it names `Users/Settings` and `Legal` as also needing their own landmark. They do not — both are nested inside `SideNavLayout`, so they already receive the one added there. Adding a second would be worse than none, since only one `main` per page is valid. Co-Authored-By: Claude Opus 5 (1M context) --- ui/src/pages/Admin/index.tsx | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/ui/src/pages/Admin/index.tsx b/ui/src/pages/Admin/index.tsx index da167ac40..5114a841f 100644 --- a/ui/src/pages/Admin/index.tsx +++ b/ui/src/pages/Admin/index.tsx @@ -36,20 +36,20 @@ const Index: FC = () => { }); return (
-
-
+
-
+
-
+