Skip to content

fix(ui): pin pagination dot dimensions to prevent flex-col stretch on mobile#168

Merged
BilgeGates merged 1 commit into
masterfrom
fix/pagination-dot-layout-mobile
Jun 26, 2026
Merged

fix(ui): pin pagination dot dimensions to prevent flex-col stretch on mobile#168
BilgeGates merged 1 commit into
masterfrom
fix/pagination-dot-layout-mobile

Conversation

@BilgeGates

Copy link
Copy Markdown
Member

Closes #140

Summary

  • Pagination dots were rendering as tall vertical bars on mobile inside flex-col containers (board theme preset and piece set pagers)
  • Root cause: <button> elements inside a flex-col parent inherited align-self: stretch, overriding Tailwind h-*/w-* classes and making dots stretch to fill the container height
  • Fix: pin all dot dimensions (width, height, min/maxWidth, min/maxHeight, borderRadius, padding, border) via inline style props — immune to any parent flex context
  • Active dot restored to 20×8px accent pill; normal dots 8×8px; faded dots 6×6px

Test plan

  • Open Export page → Board Style tab on mobile — pagination dots appear as horizontal row of circles/pill
  • Open Export page → Board Style tab on desktop — same
  • Piece set pager dots render correctly on both mobile and desktop
  • Active dot shows as wider accent pill, inactive as small circles
  • Clicking dots navigates pages correctly

@github-actions github-actions Bot added Bug Something is not working correctly area: ui Shared UI components, layout, and theming labels Jun 26, 2026
@BilgeGates BilgeGates self-assigned this Jun 26, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in ChessViewer Jun 26, 2026
@BilgeGates BilgeGates moved this from Backlog to In review in ChessViewer Jun 26, 2026
@BilgeGates BilgeGates added this to the v6.0.2 – Mobile responsive milestone Jun 26, 2026
@BilgeGates BilgeGates merged commit b170bb8 into master Jun 26, 2026
10 of 11 checks passed
@BilgeGates BilgeGates deleted the fix/pagination-dot-layout-mobile branch June 26, 2026 18:20
@github-project-automation github-project-automation Bot moved this from In review to Done in ChessViewer Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui Shared UI components, layout, and theming Bug Something is not working correctly

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Bug]: Mobile navbar dropdown spacing and general mobile UI spacing issues

1 participant