Repository navigation
fix(seasons): preview scene photos on tap instead of hover - #392
Conversation
Hover previews made the background photo flicker while moving across
the cards. Tapping or clicking a place now swaps the photo at every
screen size.
- Remove the hover and focus preview handlers from place cards and
nearby items; the nearby list buttons still switch on activation.
- Show a one-line hint ("카드를 눌러 사진 보기") in the nearby panel
until the user dismisses it; the dismissal is stored in localStorage
with try/catch and read after mount.
- Shorten the nearby item reveal delay (was up to ~0.8s before text
appeared; now ~0.2–0.4s).
Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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. Comment |
📋 코드 리뷰 체크리스트
|
|
✅ PR 검증 통과
|
|
✅ 코드 품질 검사가 완료되었습니다. 위 출력 결과에서 이슈를 확인하세요. |
|
♿ 접근성 감사가 완료되었습니다. 아티팩트의 a11y-report를 확인하세요. |
|
📦 번들 크기 분석 요약: 번들 크기 분석 결과Next.js 빌드 출력 크기257M .next 가장 큰 파일 상위 20개35M .next/cache/turbopack/v16.3.5-ca2c75eb/00000014.sst |
|
⚡ Lighthouse 성능 감사가 완료되었습니다. 아티팩트에서 상세 리포트를 확인하세요. |
|
📦 번들 크기 분석 요약: 번들 크기 분석 결과Next.js 빌드 출력 크기257M .next 가장 큰 파일 상위 20개35M .next/cache/turbopack/v16.3.5-ca2c75eb/00000004.sst |
|
🔍 CodeQL 보안 분석이 완료되었습니다. 세부 검출 결과는 리포지토리의 Security 탭을 확인하세요. |
|
📊 테스트 커버리지 리포트 테스트 커버리지 요약사용 가능한 커버리지 데이터가 없습니다. |
|
✅ E2E 테스트가 완료되었습니다. 아티팩트에서 Playwright 리포트를 확인하세요. |
Summary
Test plan
npx tsc --noEmit -p .(no new errors)npx vitest run src/features/seasons src/shared/components/Layout/seasonsat 1440px and 390px: hover does not change the preview, tap does, the hint is dismissed and stays hidden after reload, no page errors🤖 Generated with Claude Code