[fix] 예매 일정 호버 잘림 및 OG URL 버그 수정, 브랜드 키워드 노출 개선 - #143
Merged
Merged
Conversation
.ticketingPanelList가 overflow-y:auto인데 padding-top이 없어 .ticketCard:hover의 translateY(-1px)+box-shadow가 스크롤 컨테이너 상단 경계에 잘리던 문제. padding으로 여유 공간을 만들고 동일한 크기의 음수 margin으로 바깥 레이아웃은 그대로 유지했다. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
콘서트/아티스트/발매 상세 페이지를 공유해도 og:url 등이 항상 홈으로 고정 노출되던 문제(PR #141 self-review 지적)를 해소한다. react-helmet-async는 index.html의 정적 meta 태그를 덮어쓰지 못하고 별도로 추가만 해서 title 요소가 중복되는 문제가 있어(첫 title 우선 규칙 때문에 오히려 반영이 안 됨) 채택하지 않았다. 대신 기존 정적 meta 노드를 직접 갱신하고 언마운트 시 기본값으로 복원하는 usePageMeta 훅을 추가했다. SSR/프리렌더링이 없는 CSR 환경이라 JS를 실행하지 않는 카카오톡/ 페이스북 등 공유 미리보기 봇에는 여전히 반영되지 않는다. 브라우저 탭 제목과 JS를 렌더링하는 검색엔진(Googlebot 등) 대상 개선. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
"coming"은 사전 단어라 검색 상위 노출이 사실상 불가능하지만 "커밍"은 브랜드성 한글 키워드라 경쟁이 적어 색인 시 상위 노출 가능성이 높다. index.html 메타 태그, 브라우저 탭 타이틀, 이용약관/개인정보처리방침, Footer, 회원가입 안내 문구를 "커밍"으로 전환했다. 좌측 상단 로고 워드마크(Logo.jsx)와 "COMING" 배지/탭(내한 예정 아티스트를 뜻하는 도메인 라벨이지 브랜드명이 아님)은 변경 대상에서 제외했다. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
모든 라우트에서 렌더링되는 schema.org/WebSite JSON-LD를 추가해 구글이 "커밍" 브랜드 키워드 검색 시 사이트 정체성을 명확히 인식하도록 한다. name/alternateName으로 "커밍"/"Coming" 둘 다 매핑했다. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
PR #141 self-review 지적사항. 빌드 시점 자동 생성 인프라가 없는 상태에서 고정 날짜를 유지하면 시간이 지날수록 실제 갱신 시점과 어긋나 오히려 신뢰도를 떨어뜨린다. lastmod는 sitemap 스펙상 선택 필드라 제거했다. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- index.html <title>을 "커밍 - Jpop 아티스트 내한 공연 정보"에서 "커밍"으로 변경 - usePageMeta document.title 복원 기본값도 동일하게 축소 - og:title 복원 기본값은 DEFAULT_OG_TITLE로 분리해 기존 키워드 문구 유지 (공유 미리보기·SEO 영향 최소화) Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (18)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThe change standardizes the brand name as “커밍”, adds shared page and website metadata, removes sitemap ChangesBranding and metadata
Ticketing panel spacing
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant DetailPage as Detail page
participant PageMeta as usePageMeta
participant Document as Document head
DetailPage->>PageMeta: provide dynamic page metadata
PageMeta->>Document: set title and Open Graph tags
PageMeta->>Document: restore site defaults on cleanup
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
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.
관련 이슈
Closes #142
변경 개요
홈 화면 예매 일정 패널의 호버 잘림 버그, PR #141 self-review에서 지적된 og:url 고정 문제를 수정하고, "coming"(사전 단어라 검색 상위 노출이 사실상 불가능)이 아닌 "커밍"(경쟁이 적은 브랜드성 한글 키워드) 중심으로 사이트 전반의 브랜드 표기를 전환해 검색 노출 가능성을 높인다.
변경사항
.ticketingPanelList에 padding/margin 추가해 호버 시 카드 상단 잘림 수정주요 구현 내용
.ticketingPanelList가overflow-y:auto인데 padding-top이 없어.ticketCard:hover의translateY(-1px)+box-shadow가 스크롤 컨테이너 상단 경계에 잘리던 문제. padding으로 여유 공간을 만들고 동일 크기의 음수 margin으로 바깥 레이아웃은 그대로 유지react-helmet-async도입을 시도했으나, 이 라이브러리가index.html의 정적 meta 태그를 덮어쓰지 않고 별도로 추가만 해서<title>이 중복되고(브라우저는 첫 번째 title을 우선하므로 오히려 반영이 안 됨) og:url도 중복되는 문제를 발견해 제거함. 대신 기존 정적 meta 노드를 직접 갱신·복원하는usePageMeta훅으로 구현테스트
npm run lint,npm run build통과 — 기존에 존재하던 무관한 18개 lint 에러 외 신규 에러 없음)리뷰어 참고사항
AdminPage.jsx의 "Coming 관리자 콘솔" 문구는 관리자 전용(robots.txt 차단) 페이지라 브랜드 전환 범위에서 제외함코드 리뷰
변경사항 요약
예매 일정 패널 호버 잘림 버그 수정, 상세 페이지 OG URL 동적화(usePageMeta 훅 신규), "Coming"→"커밍" 브랜드 표기 전환, 사이트 전역 WebSite JSON-LD 추가, sitemap.xml lastmod 정리. 총 18개 파일 변경, 신규 훅 1개(usePageMeta.js) 추가.
검토 결과
🔵 suggestion
src/hooks/usePageMeta.js/src/hooks/usePageTitle.js: 두 훅이 title 설정 로직을 일부 중복하고 있음(하나는 title만, 하나는 title+OG meta)→ 현재는 상세 페이지 3곳만 OG meta가 추가로 필요해 분리했지만, 대상 페이지가 늘어나면 하나로 통합하는 것을 고려할 만함
✅ 그 외 특이사항 없음.
Summary by CodeRabbit
Branding
SEO & Metadata
Visual Improvements