[Fix] : 곡 표기 한국어 우선 통일 및 UI 수정 (#315) - #319
Open
GulSam00 wants to merge 5 commits into
Open
Conversation
로그인 안내 오버레이가 bg-background/70 + blur 2px라 뒤의 투표 패널이 거의 안 보였다. 무엇에 참여하는지 보여야 로그인 동기가 생기므로 /10 + blur 1px로 낮춘다. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V3Qg1DFPBGjtDjhBN5CuAn
"전체 문장보다는 단어 단위로 검색해보세요"를 "단어 단위로 검색해주세요"로 줄인다. 안내 문구가 길면 정작 읽지 않는다. 함께 주석 처리된 채 남아 있던 ChatBot 위젯 관련 코드(import, 토글 상태, localStorage 핸들러, 렌더 블록)를 삭제한다. 되살릴 계획이 없어 git 이력에만 남기면 된다. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V3Qg1DFPBGjtDjhBN5CuAn
검색 화면만 title_ko를 큰 줄에 올리고 부를곡·즐겨찾기·재생목록·홍보는 원어를 큰 줄에 두고 있었다. 검색에서 "요네즈 켄시"를 보고 담은 곡이 부를곡 목록에선 "米津玄師"로 떠, 한자를 못 읽는 사용자가 자기가 담은 곡을 눈으로 찾지 못했다. 일본곡 번역을 채워둔 의미가 검색에서만 살고 정작 노래방에서 보는 화면에서는 죽던 셈이다. 화면마다 복붙돼 있던 규칙을 utils/songDisplay.ts의 splitDisplay 하나로 모으고 한국어 우선으로 통일한다. 우선순위를 다시 바꿀 일이 생기면 이 함수만 고치면 된다. title_ko가 비어 있는 한국곡·영미곡은 예전과 똑같이 한 줄로 그려진다. 이미 한국어 우선이던 SearchResultCard·ReportFieldCard·SongPromotionModal은 각자 갖고 있던 같은 로직을 유틸로 교체만 했다(동작 변화 없음). 내 홍보 화면의 취소 다이얼로그는 `title_ko ?? title`이라 title_ko가 원어와 같은 값일 때를 거르지 못했는데, 이 과정에서 같은 규칙으로 맞춰졌다. 이달의 아티스트 배지 판정은 원어(getPrimaryArtistName(artist))를 그대로 쓴다. artists 마스터가 TJ 표기라 번역된 이름으로 비교하면 배지가 빠진다. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V3Qg1DFPBGjtDjhBN5CuAn
최상위 이동·삭제·최하위 이동 세 버튼의 aria-label이 전부 "삭제"라 스크린리더로는 어느 것이 삭제인지 구분할 수 없었다. 복붙 실수다. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V3Qg1DFPBGjtDjhBN5CuAn
빌드 검증 과정에서 next-sitemap postbuild가 lastmod를 갱신했다. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V3Qg1DFPBGjtDjhBN5CuAn
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Owner
Author
|
/describe |
Owner
Author
|
/review |
Owner
Author
|
/improve |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 PR 제목
[Fix] : 곡 표기 한국어 우선 통일 및 UI 수정
📌 변경 사항
곡 제목·아티스트 표기를 한국어 우선으로 통일 (
utils/songDisplay.ts신설)title_ko를 큰 줄에 올리고 부를곡·즐겨찾기·재생목록·홍보는 원어를 큰 줄에 두고 있었다. 검색에서 "요네즈 켄시"를 보고 담은 곡이 부를곡 목록에선 "米津玄師"로 떠, 한자를 못 읽는 사용자가 자기가 담은 곡을 눈으로 찾지 못했다splitDisplay(translated, original)하나로 모으고 한국어 우선으로 통일SongSummary(부를곡·최신곡·인기곡), 즐겨찾기, 재생목록 폴더, 부를곡 추가 모달, 재생목록 저장 모달, 홍보 전광판, 내 홍보 목록SearchResultCard,ReportFieldCard,SongPromotionModaltitle_ko가 비어 있는 한국곡·영미곡은 예전과 똑같이 한 줄로 그려진다접근성
tosing/SongCard.tsx— 최상위 이동·삭제·최하위 이동 세 버튼의aria-label이 전부"삭제"였던 것을 수정. 스크린리더로 어느 것이 삭제인지 구분할 수 없었다UI 다듬기
bg-background/70→/10, blur 2px → 1px) — 무엇에 참여하는지 보여야 로그인 동기가 생긴다next-sitemappostbuild 결과 반영💬 추가 참고 사항
덤으로 고쳐진 것 — 내 홍보 화면의 취소 다이얼로그가
title_ko ?? title이라title_ko가 원어와 같은 값일 때를 거르지 못했다. 같은 파일 안에서 목록과 다이얼로그의 규칙이 달랐는데 이번에 맞춰졌다.의도적으로 유지한 것 — 이달의 아티스트 배지 판정은 원어(
getPrimaryArtistName(artist))를 그대로 쓴다.artists마스터가 TJ 표기라 번역된 이름으로 비교하면 배지가 빠진다.리스크 — 오역된
artist_ko가 있으면 큰 글씨로 승격돼 더 눈에 띈다.#313에서artistAlias사전으로 중국곡 혼입을 막았지만, 그 이전에 잘못 박힌 데이터가 남아 있다면 도드라질 수 있다. 머지 후title_ko is not null인 곡을 한 번 훑어보면 좋다.검증:
tsc --noEmit✅ /pnpm lint✅ (건드린 파일 경고 0) /pnpm build✅. 테스트 스위트는 없어 일본곡 표시는 실데이터 QA 필요.