Skip to content

[Chore] : UI 조정 및 검색 투어 종료 버그 수정 (#311) - #312

Open
GulSam00 wants to merge 5 commits into
developfrom
chore/311-uiAdjustment
Open

[Chore] : UI 조정 및 검색 투어 종료 버그 수정 (#311)#312
GulSam00 wants to merge 5 commits into
developfrom
chore/311-uiAdjustment

Conversation

@GulSam00

Copy link
Copy Markdown
Owner

📌 PR 제목

[Chore] : UI 조정 및 검색 투어 종료 버그 수정

📌 변경 사항

인기 차트 — 장르 필터 뱃지 아이콘 제거

  • ChartGenreFilter의 칩에서 이모지 아이콘을 제거하고 라벨 텍스트만 남김
  • 더 이상 참조되지 않는 STR_TYPE_EMOJI 상수(types/tjChart.ts)도 함께 삭제

검색 투어 — X 버튼으로 투어가 종료되지 않던 문제 수정

  • react-joyride 3.x의 closeButtonAction 기본값 'close'는 continuous 모드에서 인덱스만 +1 하고 statusRUNNING으로 유지해, X를 눌러도 다음 단계로 넘어갈 뿐 투어가 끝나지 않았음
  • 옵션에 closeButtonAction: 'skip'을 지정해 controls.skip()이 호출되도록 변경. 상태가 SKIPPED가 되면서 기존 TOUR_END 핸들러의 '다시 보지 않기' 저장까지 정상 동작함
  • 투어 예시 곡 데이터를 실제 곡 정보로 수정 ('밤이 깊었네' / 크라잉넛)

이달의 아티스트 — 비로그인 안내를 오버레이로 변경

  • 기존에는 early return으로 투표 화면 전체를 안내 문구로 대체했으나, 이제 투표 UI를 그대로 렌더링하고 그 위에 반투명 레이어(bg-background/70 + backdrop-blur)로 안내 문구만 덮음
  • 아래 UI에는 inert를 적용해 클릭·키보드 포커스·접근성 트리 노출을 차단 (pointer-events-none만으로는 탭 이동이 막히지 않음)

기타

  • 빌드 postbuild(next-sitemap)로 sitemap-0.xml 재생성

💬 추가 참고 사항

  • pnpm build / pnpm lint / pnpm format 통과 (테스트 스위트 없음)
  • ESC 키(dismissKeyAction)와 오버레이 클릭(overlayClickAction)도 기본값이 'close'라 동일하게 "다음 단계로 이동"합니다. 다만 ESC는 옵션에 'skip' 값이 없어('close' | 'next' | 'replay' | false) 종료시키려면 비활성화하는 수밖에 없어 이번 범위에서는 제외했습니다.
  • close UI 조정 작업 #311

@GulSam00

Copy link
Copy Markdown
Owner Author

/describe

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
singcode Ready Ready Preview Aug 28, 2026 4:44pm

@GulSam00

Copy link
Copy Markdown
Owner Author

/review

@qodo-code-review

Copy link
Copy Markdown

ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing

@GulSam00

Copy link
Copy Markdown
Owner Author

/improve

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.

UI 조정 작업

1 participant