Skip to content

fix(#744): polish MethodChip layout for narrow (≤375px) viewports - #807

Closed
legalienn wants to merge 1 commit into
CalloraOrg:mainfrom
legalienn:fix/744-methodchip-mobile
Closed

fix(#744): polish MethodChip layout for narrow (≤375px) viewports#807
legalienn wants to merge 1 commit into
CalloraOrg:mainfrom
legalienn:fix/744-methodchip-mobile

Conversation

@legalienn

Copy link
Copy Markdown
Contributor

Overview

This PR audits the MethodChip component for narrow mobile viewports (≤375px). Tap targets are enlarged to meet WCAG 2.5.5 minimum 44×44px, padding and margins are increased for better touch interaction, and the chip layout now wraps gracefully when multiple chips overflow the viewport.

Related Issue

Closes #744

Changes

  • src/components/MethodChip.css — Add @media (max-width: 375px) breakpoint with enlarged padding (10px 14px), 44px min-height, increased font-size, larger icon gap, and adjusted tooltip styling. Base styles also improved with increased min-height (32px) and padding.
  • src/components/MethodChip.test.tsx — Add 9 focused tests covering render, focus, tooltip, CSS class contract for responsive breakpoints, design-token color usage, and fallback colors for unknown methods.

Verification Results

npx vitest run src/components/MethodChip.test.tsx
✅ 9/9 passed
Acceptance Criteria Status
Tap targets ≥ 44×44px on narrow viewports ✅ CSS @media breakpoint enforces min-height: 44px, min-width: 44px
Chips wrap without overflow ✅ Increased margin/gap allows natural flex-wrap
Tooltip remains accessible on mobile ✅ Larger padding, font-size, and repositioned on narrow screens
Tests added for new behavior ✅ 9 new tests

Timeline

  • legalienn committed

…wports (Closes CalloraOrg#744)

Audit MethodChip for narrow mobile viewports (≤375px): enlarged tap targets (44×44px, WCAG 2.5.5), responsive padding/margin wrapping, fix pre-existing class name mismatch (.method-icon → .method-chip-icon), and add comprehensive test coverage with 9 focused tests.
@greatest0fallt1me

Copy link
Copy Markdown
Contributor

Closing this as invalid — it doesn't link to a DripsWave-assigned issue or the PR author isn't the current assignee.

To contribute: apply on DripsWave, wait for assignment, then open a PR with Closes #<issue-number> in the body. Thanks!

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.

Polish MethodChip layout for narrow (mobile) viewports [b#067]

2 participants