Skip to content

fix: 상점주 예약 목록 높이를 상세 패널에 맞춤 - #324

Merged
xodn09 merged 1 commit into
developfrom
fix/merchant-reservation-panel-height
Oct 20, 2026
Merged

xodn09 merged 1 commit into
developfrom
fix/merchant-reservation-panel-height

Conversation

@xodn09

@xodn09 xodn09 commented Oct 20, 2026 •

Copy link
Copy Markdown
Member

변경 요약

fix/merchant-reservation-panel-height → develop

상점주 예약 신청 조회 화면에서 왼쪽 목록이 오른쪽 상세 패널보다 일찍 끝나던 높이 제한을 수정합니다. 브라우저 주석의 목록 높이 요청 한 건만 반영합니다.

  • 넓은 화면에서는 두 패널의 위·아래 경계를 맞추고, 목록의 스크롤 영역이 상세 높이에 따라 확장됩니다.
  • 목록 내용이 전체 화면 높이를 무한히 늘리지 않도록 크기 계산을 분리하고, 넘치는 예약은 목록 안에서 스크롤합니다. 카드 자체는 압축하지 않습니다.
  • 980px 이하에서는 기존 세로 배치와 65vh 목록 스크롤 상한을 유지합니다. 페이지네이션은 스크롤 영역 밖에 유지합니다.
  • 예약 API·승인·반려·취소·필터·페이지 로직은 변경하지 않습니다.

검증

  • 변경 범위에 맞는 lint, type check, build 또는 직접 검증을 수행했습니다.

  • 수행하지 않은 검증과 이유를 적었습니다.

  • npm test: 1,123개 통과, 실패·skip 없음.

  • npm run lint, TypeScript 포함 npm run build, git diff --check 통과.

  • node tests/browser/merchant-reservation-query.mjs: 1920×1080·1366×768·390×844 통과. 긴 상세 이력에서 목록 높이 증가·패널 경계 일치·내부 스크롤·카드 비압축·마지막 카드 선택·페이지 이동을 추가 검증합니다. 기존 필터·계정 범위 전환·로딩·빈 결과·조회 실패·권한 실패 검사도 유지합니다.

  • 실제 페이지/스타일과 합성 GET 응답으로 검증했습니다. 운영 예약의 승인·반려·취소 등 쓰기나 실기기/Safari/Firefox 검사는 수행하지 않았습니다.

  • 이 PR 최신 HEAD b21b750의 원격 CI verify·ec2-image 모두 성공했습니다. 전체 브라우저 23개 스위트를 포함합니다.

  • 실제 로컬 상점주 화면에서도 두 패널 높이 약 763px·하단 차이 0px, 목록 내부 스크롤을 DOM으로 확인했습니다.

Web API 계약

  • Web API 영향 없음
  • 영향 operation을 아래에 METHOD /path 형식으로 모두 기록했습니다.
  • 계약 변경 시 계약표 갱신 여부 검토: endpoint/schema/payload 변경이 없어 해당 없음입니다.
  • 제거·legacy API를 구현 완료로 표시하지 않았습니다.
  • 파괴적 mutation의 경고·확인 수준 검토: 처리 로직을 변경하지 않았으며 운영 쓰기를 실행하지 않습니다.

영향 operation:

  • 없음. 목록/상세의 기존 GET 동작을 유지합니다.

관련 이슈

  • 별도 이슈 없음. 사용자 브라우저 주석의 예약 목록 높이 개선 요청입니다.

릴리스 반영 결과

  • 이 PR은 develop eeb807d로 병합됐고 release PR #325에서 release 5af0623으로 병합했습니다.
  • release CI와 Docker 배포가 성공했습니다. 새 공개 JS·예약 화면 코드·배치 스타일의 반영을 확인했습니다.
  • 로그인 세션 만료로 운영 재로그인 후 실제 패널 높이 측정만 merge: develop into release #325 체크리스트에 남겨둡니다. 운영 예약의 승인·반려·취소 등은 실행하지 않았습니다.

@xodn09
xodn09 merged commit eeb807d into develop Oct 20, 2026
2 checks passed
@xodn09 xodn09 mentioned this pull request Oct 20, 2026
10 of 11 tasks
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