diff --git a/docs/plans/20260505-ui-feedback/plan.md b/docs/plans/20260505-ui-feedback/plan.md new file mode 100644 index 0000000..68b7029 --- /dev/null +++ b/docs/plans/20260505-ui-feedback/plan.md @@ -0,0 +1,128 @@ +# UI Feedback - Mobile Navbar Restructure & Size Fixes + +Date: 2026-05-05 + +## Problem + +1. **Mobile navbar layout** — The hamburger menu currently sits in `navbar-start` (left side), leaving the mobile layout as: `[hamburger + logo/name] ... [spinner | theme | user]`. The desired order is: `[logo/name] ... [spinner | theme | user | hamburger]`. + +2. **Dropdown menu sizes too small** — After the Tailwind v4 / DaisyUI v5 upgrade, the mobile navigation dropdown and user profile dropdown render with noticeably smaller text and icons, making tap targets difficult to hit. + +3. **Duplicated inline SVGs** — The mobile nav dropdown in `spa.html` contains inline SVGs duplicated from the `icons.js` icon functions. These should use the existing JS icon functions to eliminate duplication. + +## Files to Change + +| File | Change | +|------|--------| +| `src/meshcore_hub/web/templates/spa.html` | Restructure navbar: move hamburger to `navbar-end`, remove inline SVGs from mobile dropdown, render mobile nav via JS instead | +| `src/meshcore_hub/web/static/js/spa/app.js` | Add mobile nav rendering (inject icon + link items into the mobile dropdown via JS) | +| `src/meshcore_hub/web/static/js/spa/components.js` | Increase user profile dropdown sizes (`menu-sm` → normal or custom, icon `h-4 w-4` → `h-5 w-5`) | +| `src/meshcore_hub/web/static/css/app.css` | Add CSS rules for increased mobile dropdown font size, icon size, and tap target padding | + +## Step-by-step Plan + +### Step 1: Restructure `spa.html` navbar layout + +**Current structure:** +``` +navbar-start: [hamburger dropdown] [logo/name link] +navbar-center: [desktop nav menu] (hidden on mobile) +navbar-end: [spinner] [theme toggle] [auth section] +``` + +**Target structure:** +``` +navbar-start: [logo/name link] +navbar-center: [desktop nav menu] (hidden on mobile) +navbar-end: [spinner] [theme toggle] [auth section] [hamburger dropdown] +``` + +Changes in `spa.html`: +- **Remove** the entire `