feat(ui): mobile hamburger nav, PWA support, 40 unit tests#307
Open
natiixnt wants to merge 2 commits intoruvnet:mainfrom
Open
feat(ui): mobile hamburger nav, PWA support, 40 unit tests#307natiixnt wants to merge 2 commits intoruvnet:mainfrom
natiixnt wants to merge 2 commits intoruvnet:mainfrom
Conversation
…oggle, and WCAG accessibility - Keyboard shortcuts overlay (press ? for help, 1-8 for tabs, T for theme, P for perf) - Real-time performance monitor with FPS, memory, latency sparklines (draggable) - Enhanced toast notification system with stacking, auto-dismiss, progress bars - Dark/light theme toggle with localStorage persistence and system preference detection - WCAG accessibility: skip-to-content link, ARIA roles/attributes on tabs and panels, arrow key navigation in tab bar, focus-visible outlines - ESLint config for UI directory with security and quality rules
- Mobile hamburger navigation: slide-out drawer replacing tab bar on <768px, swipe-to-close, animated hamburger icon, auto-sync with tab manager - PWA manifest + service worker: installable dashboard, offline shell caching (cache-first for static, network-first for API), auto-cleanup of old caches - 40 unit tests for ToastManager, ThemeToggle, KeyboardShortcuts, PerfMonitor, TabManager - browser-based test runner at ui/tests/unit-tests.html - PWA meta tags: theme-color, apple-mobile-web-app-capable, manifest link - Icon generator page for creating PWA icons (ui/icons/generate.html)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR addresses three significant gaps in the web UI:
1. Mobile Navigation (hamburger menu)
The 10-tab navigation bar wraps awkwardly on mobile devices. This adds a proper hamburger menu:
2. PWA Support (Progressive Web App)
Turns the dashboard into an installable, offline-capable app:
manifest.jsonwith app metadata, theme colors, icon referencessw.js) with dual caching strategy:index.html(theme-color, apple-mobile-web-app)ui/icons/generate.html) for creating PNG icons3. Unit Tests (40 tests, 5 suites)
The web UI previously had zero tests (mobile/ has 30+, but web/ had none). This adds a browser-based test runner:
Open
ui/tests/unit-tests.htmlin browser to run all tests.Test plan
ui/tests/unit-tests.html- all 40 tests pass (green)