Skip to content

feat(ui): harden responsive layout and typography for mobile and narrow viewports (#144) - #157

Merged
Dyu20705 merged 1 commit into
devfrom
issue/144-mobile-responsive-hardening
Aug 31, 2026
Merged

feat(ui): harden responsive layout and typography for mobile and narrow viewports (#144)#157
Dyu20705 merged 1 commit into
devfrom
issue/144-mobile-responsive-hardening

Conversation

@Dyu20705

Copy link
Copy Markdown
Owner

Summary

Hardens layout and typography for viewport widths below 768px (mobile devices and split-screen desktop windows), ensuring touch-friendly tap targets and responsive layout adaptations while preserving the existing desktop-first architecture.

Fixes #144 Part of #143

Changes

  • styles.css: Added @media (max-width: 767px) responsive rules for app shell, topbar, header actions, search box, buttons (44x44px minimum touch targets), and iOS auto-zoom prevention (16px input font-size).
  • styles/editor.css: Added 767px mobile rules adapting note editor overlay to full-viewport modal on narrow screens, compacting editor header, sizing toolbar/action buttons to >=44x44px, and adjusting drawing region max-height.
  • styles/japanese.css: Fixed 3-button workspace switcher grid bug (repeat(3, 1fr)), added 767px rules for touch-friendly filters, presets, subview buttons, full-width review dialog, and iOS zoom prevention on Japanese input fields.
  • styles/kanji-ink.css: Added 767px rules for full-viewport kanji ink dialog, flexible grid layout, wrapping toolbar, and >=44x44px icon buttons.
  • styles/command.css: Added 767px rules for iOS zoom prevention on command input and touch-friendly command items.
  • styles/state-recovery.css: Added 767px rules for responsive recovery banner and dialog sizing.
  • tests/e2e/mobile-viewport.spec.mjs: Added 24 Playwright E2E tests covering 3 viewport configurations (393×852, 412×915, 600×800) validating no horizontal document overflow, workspace navigation, note CRUD workflow, editor overlay containment, Japanese workspace usability, touch target bounding boxes, and long content handling.

Verification

  • npm run test:content -> PASS (3/3)
  • npm run lint -> PASS (0 errors, 0 warnings)
  • npm run test:unit -> PASS (317/317)
  • npm run test:integration -> PASS (111/111)
  • npm run test:e2e -> PASS (164/164)
  • git diff --check -> PASS

Copilot AI lite review requested due to automatic review settings August 31, 2026 05:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Dyu20705
Dyu20705 merged commit 0f0d866 into dev Aug 31, 2026
1 check passed
@Dyu20705
Dyu20705 deleted the issue/144-mobile-responsive-hardening branch August 31, 2026 14:23
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.

2 participants