Skip to content

fix: 팝업이 화면 밖에 배치되고 스크롤 영역에 잘리던 문제 - #132

Merged
lee-sihun merged 7 commits into
masterfrom
fix/popup-viewport-clamp
Aug 18, 2026
Merged

fix: 팝업이 화면 밖에 배치되고 스크롤 영역에 잘리던 문제#132
lee-sihun merged 7 commits into
masterfrom
fix/popup-viewport-clamp

Conversation

@eun-yeon

Copy link
Copy Markdown
Member

개요

팝업이 내용 없는 상태에서 잰 크기로 배치돼 화면 밖으로 나가던 문제 해소. 화면 경계 계산을 공용 유틸로 통합하고 피커의 잘림을 구조적으로 차단

변경 내용

  • 내용이 채워진 뒤 크기를 다시 재도록 전환, 빈 상태 측정값으로 첫 배치가 화면 밖에 놓이던 문제 해소
  • 뷰포트 크기 구독과 목록 스크롤 지표 계산을 전용 모듈로 분리
  • 화면 경계 보정을 공용 유틸로 통합, 드롭다운·플로팅 팝업·패널 앵커 팝업·플러그인 드롭다운이 같은 규칙 사용
  • 피커가 기본으로 창 최상위에 그려지도록 전환, 스크롤·contain 조상 안에서 잘리던 경로 제거
  • 색상 형식 메뉴를 공용 팝업으로 교체, 자체 배치 코드 152줄 제거
  • 색상 형식 메뉴가 트리거에서 떨어지고 피커 아래로 깔리던 문제 해소
  • 신규 테스트 5개 파일로 경계 보정 커버리지 확보

검증

  • npx tsc --noEmit / npm run lint 오류 0, 경고는 base와 동일 / npm run format:check
  • vitest 3,015 통과
  • 실기: 화면 가장자리에서 드롭다운·색상 형식 메뉴·멀티 키 피커 열기 확인 예정

리뷰 포인트

  • 커밋 단위 리뷰 권장. 결함 수정 두 건이 앞, 공용화가 뒤
  • 멀티 키 피커에 넣은 portalToBody 프롭은 뒤 커밋에서 기본 동작 전환으로 대체돼 브랜치 순변화 0. 중간 커밋에만 존재

@lee-sihun
lee-sihun merged commit 839eb01 into master Aug 18, 2026
@lee-sihun
lee-sihun deleted the fix/popup-viewport-clamp branch August 18, 2026 00:56
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