Skip to content

fix(cinema): mobile seat-map zoom, teal selection and center-zone overlay - #9

Draft
hoangsvit wants to merge 14 commits into
mainfrom
fix/mobile-seatmap-zoom-selection-center-zone
Draft

hoangsvit wants to merge 14 commits into
mainfrom
fix/mobile-seatmap-zoom-selection-center-zone

Conversation

@hoangsvit

@hoangsvit hoangsvit commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

Theo ảnh đối chiếu trên iPhone

  • Zoom: nút −/+ (45–180%), Vừa khung và 100%. Hỗ trợ pinch hai ngón, giữ trọng tâm cuộn khi thay đổi tỷ lệ.
  • Màu ghế: chuyển sang xanh ngọc khi chọn, kể cả ghế VIP đỏ; trạng thái seat--selected + aria-pressed và CSS ưu tiên cho :hover/:focus/:active trên Mobile Safari. Bỏ chọn thì trở lại màu loại ghế từ API.
  • Vùng trung tâm: vẽ viền xanh mảnh có bo góc quanh vùng giữa sơ đồ dựa trên vị trí ghế thực tế (không phủ lên nút và không làm lệch hàng/lối đi), thêm chú thích "Vùng trung tâm (tham khảo)". Đây là chỉ dẫn hình học, không tự nhận là vùng ưu tiên do MoMo trả về.
  • Modal mobile: chuyển popup trực tiếp vào document.body, tăng z-index vượt sticky header và chỉnh chiều cao theo safe-area iPhone, footer vẫn neo bên dưới.
  • Regression test: scripts/seatmap-ui-test.mjs cùng bộ test màu ghế và build/smoke hiện hành.

QA

  • CI + Pages staging thành công (CI, Staging)
  • Kiểm tra trên iOS Safari: scroll ngang, zoom, chọn/bỏ chọn ghế VIP, footer và legend
  • Preview publish thành công (Pages)

Không chạm workflow chống trùng ở PR #8; không merge tự động.

@coderabbitai

coderabbitai Bot commented Nov 10, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

This branch has not been deployed

No deployments
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