Skip to content

fix: 방 큐와 모달 UI 표시 오류 교정 - #61

Merged
aryu1217 merged 2 commits into
mainfrom
dev
Sep 8, 2026
Merged

aryu1217 merged 2 commits into
mainfrom
dev

Conversation

@aryu1217

@aryu1217 aryu1217 commented Sep 5, 2026

Copy link
Copy Markdown
Member

변경 내용

  • 내 노래 타임라인에서 현재 재생곡과 같은 entryId의 history 항목을 제외해 중복 표시를 막았습니다.
  • 노래 신청 폼의 URL/재생목록 선택 영역과 선정 이유 사이를 40px로 맞췄습니다.
  • 방 내부 Settings/Friends 모달에만 dim 배경을 적용하고, 두 모달의 데스크톱 기본 높이를 540px로 맞췄습니다.
  • Friends 목록에서 연 프로필 상세의 누락된 상단 패딩을 복원했습니다.

변경 이유

  • 현재 재생곡이 history 응답에도 포함되면 내 노래 탭에 같은 곡이 두 번 보였습니다.
  • 신청 폼과 Friends/Settings 모달이 요청된 기본 간격·높이·배경 명도와 달랐고, Friends 프로필 상세는 본문이 drag header에 붙어 보였습니다.

영향 범위

  • UI / 사용자 흐름
  • API 요청·응답 계약
  • React Query 캐시
  • WebSocket / 실시간 상태
  • 인증 / 보안
  • 문서 / 개발 도구

검증

  • npm run lint
  • npm run test
  • npm run build
  • 관련 수동 시나리오 확인
  • 실패·로딩·빈 상태 확인

검증 결과 및 재현 방법:

  • targeted Vitest: 5 files / 26 tests pass
  • full Vitest: 155 files / 688 tests pass
  • lint: pass
  • build: pass
  • git diff --check: pass
  • 브라우저 세션을 사용할 수 없어 실제 픽셀 시각 QA는 미실시

리뷰 포인트

  • 현재곡 entryId dedupe가 표시되는 현재곡에만 적용되어 자동 재생 history를 보존하는지
  • dimBackdrop이 방 호출부에서만 opt-in되어 홈/검색 모달 배경을 바꾸지 않는지
  • normal density에서 540px Friends 모달이 72px 행 약 6.5개를 표시하는지

기능별 커밋

  • c0734f2 fix(room): 큐와 모달 UI 표시 오류를 교정 — 현재곡 dedupe, 폼 간격, 방 modal dim/높이, Friends 상세 패딩 및 회귀 테스트

위험 및 후속 작업

  • compact density에서는 57.6px 행이 적용되어 Friends 목록 노출 행 수가 normal density와 다릅니다.
  • 매우 낮은 데스크톱 viewport의 세부 반응형과 실제 픽셀 결과는 후속 브라우저 시각 QA가 필요합니다.

추적 정보

  • Linked issue: 없음
  • Execution plan: docs/exec-plans/active/2026-09-05-room-queue-modal-polish/
  • Selected skills: queuing-feature-delivery, queuing-orchestrator, queuing-ui-flow, frontend-architecture-guardrails, queuing-qa-reviewer
  • QA result: pass

@vercel

vercel Bot commented Sep 5, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
queuing Ready Ready Preview Sep 5, 2026 2:39am UTC

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@aryu1217
aryu1217 marked this pull request as ready for review September 8, 2026 10:11
@aryu1217
aryu1217 merged commit b7d680a into main Sep 8, 2026
3 of 4 checks passed
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-08T10:13:05.908711Z b84fbb3 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

This branch was successfully deployed

1 active deployment
Preview — b84fbb37 Deployed Sep 5, 2026 by vercel[bot]
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