Skip to content

feat(v2): collapsible navigation rail with frequency-based prioritization#89

Merged
godie merged 6 commits intomasterfrom
migration/v2
Apr 24, 2026
Merged

feat(v2): collapsible navigation rail with frequency-based prioritization#89
godie merged 6 commits intomasterfrom
migration/v2

Conversation

@godie
Copy link
Copy Markdown
Owner

@godie godie commented Apr 24, 2026

Summary

V2 navigation redesign with collapsible menu and user behavior tracking.

Changes

  • Collapsible Nav Rail: Top 4 items by usage + 2 fixed (Inicio, Práctica). Rest in drawer.
  • Frequency Tracking: localStorage persistence, auto-sorts visible items
  • Responsive Fixes: Removed legacy CSS padding-left conflicts (App.css, index.css)
  • Landing Page: Better card separation (32px gap) in specialties grid
  • Tests: Added test files for V2Navi, V2Examen, V2FlashcardCreator components

Commits

  • fix(v2): remove legacy padding-left conflicts affecting landing page
  • feat(v2): implement collapsible navigation rail with frequency-based prioritization
  • fix(v2): improve a11y and theme-awareness in CSS and dialogs

Files Changed

  • 59 files, +9,962 / -1,800 lines

Reviewer Notes

  • No critical blockers
  • localStorage errors handled gracefully
  • Mobile detection via resize listener (could add debounce in future)
  • V1 admin pages should render correctly without legacy padding

Closes navigation accessibility improvement

godie added 3 commits April 24, 2026 07:34
- prefers-reduced-motion: fix skeleton class names (.skeleton-title vs .v2-shimmer)
- V2SubscriptionManagement: separate scrim (role=presentation) from dialog (role=dialog)
- V2FlashcardCreator: add v2-bg-secondary-container utility for theme consistency
…prioritization

- Add navFrequency.js utility for tracking user navigation patterns
- Create V2NavDrawer component with desktop drawer and mobile bottom sheet
- Update V2Navi to show fixed items + top 4 by frequency
- Add 'Ver más' button to access all navigation options
- Fix responsive issues for screens 1024px-1400px
- Add .v2-hero-icon, .v2-mock-exam-grid, .v2-specialties-grid utilities
- Fix landing page responsive layout for desktop viewports
- Update mock exam setup and practica landing with responsive grids
- Add tests for V2 components (V2Navi, V2Examen, V2FlashcardCreator, etc.)
- Fix lint errors across V2 pages (unused imports, escape characters)

Closes navigation accessibility improvement
Legacy App.css and index.css applied 300px padding-left starting at 993px,
conflicting with V2's 80px nav rail layout. Removed conflicting rules.
Also added gap: 32px to v2-landing-specialties-grid for better card separation.
- Add post-deploy verification step to check .htaccess upload
- Fix permissions to 644 on remote server
- Add local test script for deploy workflow validation
- Ignore test artifacts in .gitignore
@godie godie merged commit 59cec93 into master Apr 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant