Skip to content

fix: 큐레이션 hidden input 페이지 스크롤 확장 수정#87

Merged
hyejj19 merged 1 commit into
mainfrom
fix/curation-hidden-file-input-scroll
Jul 8, 2026
Merged

fix: 큐레이션 hidden input 페이지 스크롤 확장 수정#87
hyejj19 merged 1 commit into
mainfrom
fix/curation-hidden-file-input-scroll

Conversation

@bottlenote-app

@bottlenote-app bottlenote-app Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

PR 제목 (Title)

  • 큐레이션 이미지 파일 입력으로 인한 html 페이지 스크롤 확장 수정

변경 사항 (Changes)

  • 체크리스트
    • 큐레이션 위스키 직접 입력 이미지 파일 input을 sr-only에서 hidden으로 변경했습니다.
    • 위스키 페어링 음식 이미지 파일 input을 sr-only에서 hidden으로 변경했습니다.
    • 페어링 음식명/설명 보조 label을 hidden 처리하고 기존 input aria-label 접근성 경로를 유지했습니다.

변경 이유 (Reason for Changes)

  • dev 배포 화면에서 위스키 직접 입력 또는 페어링 음식 이미지 등록 UI가 렌더링되면 sr-only 파일 input/label이 스크롤 컨테이너 내부의 절대 위치 요소로 계산되어 document.documentElement.scrollHeight가 viewport보다 크게 늘어났습니다.
  • 실제로 콘텐츠 스크롤은 <main> 내부에 있어야 하지만, html 영역에도 별도 하단 스크롤이 생기는 문제가 발생했습니다.
  • 화면에 표시하지 않는 파일 input은 접근성용 sr-only보다 레이아웃에 전혀 참여하지 않는 hidden이 적합합니다.

테스트 방법 (Test Procedure)

  • dev 배포 화면에서 재현 확인
    • https://admin.development.bottle-note.com/dashboard/curations/pairings/new
    • 위스키 추가 → 직접 입력 후 document.documentElement.scrollHeight577 → 2001로 증가하는 것 확인
    • 원인 요소: 수동 위스키 이미지 파일 input .sr-only, getBoundingClientRect().top ≈ 1531.5
  • 수정 브랜치 로컬 dev 서버에서 재검증
    • 위스키 추가 → 직접 입력 후 document.documentElement.scrollHeight = 577, document.body.scrollHeight = 577 유지
    • main.scrollHeight만 실제 폼 높이에 맞게 증가
    • 파일 input 3개 모두 className: hidden, rect 0x0 확인
  • pnpm test:run -- src/pages/curation/whisky-tasting-event/__tests__/CurationWhiskyTastingEventCreate.test.tsx src/pages/curation/whisky-curation/__tests__
    • 37 files / 268 tests passed
  • pnpm build:dev
    • 성공
  • pnpm lint
    • 실패: 기존 e2e 및 기존 컴포넌트 lint 오류 11건/경고 7건 확인, 이번 변경 파일 신규 오류 없음
  • git diff --check
    • 성공

참고 사항 (Additional Information)

  • 현재 배포 이슈는 body가 아니라 html/documentElement scrollHeight가 커지는 케이스로 확인했습니다.
  • 기존 접근성 이름은 각 input의 aria-label로 유지됩니다.

@hyejj19 hyejj19 merged commit 87da827 into main Jul 8, 2026
2 checks passed
@hyejj19 hyejj19 deleted the fix/curation-hidden-file-input-scroll branch July 8, 2026 14:16
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