Skip to content

design:자료 수정 입력과 근무자 UI 개선(#93) - #94

Merged
seongjinss555 merged 1 commit into
developfrom
design/#93/resource-edit-and-schedule-avatar-layout
Jul 23, 2026
Merged

design:자료 수정 입력과 근무자 UI 개선(#93)#94
seongjinss555 merged 1 commit into
developfrom
design/#93/resource-edit-and-schedule-avatar-layout

Conversation

@seongjinss555

@seongjinss555 seongjinss555 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Pull Request

작업 내용

자료 수정 다이얼로그의 입력 항목을 명확히 구분하고, 요일별 근무자 카드의 아바타가 인원 수에 따라 깨지지 않도록 개선했습니다.

작업 결과

  • 링크 자료 수정 시 URL, 제목, 설명 입력 항목을 라벨로 구분합니다.
  • 근무자 아바타는 32px 크기를 유지하며 카드 폭을 넘으면 다음 줄로 배치됩니다.

변경 사항

Added

  • 자료 수정 입력 필드 라벨

Changed

  • 근무자 아바타 목록의 줄바꿈 레이아웃

Fixed

  • 근무자 수 증가 시 아바타 원형 축소 현상

실행화면

  • 자료 수정 다이얼로그 라벨링 추가
스크린샷 2026-07-23 오후 4 39 39
  • 근무 표 인원이 많을 시 원 픽셀 깨짐 이슈 해결
스크린샷 2026-07-23 오후 4 39 57

테스트

  • 로컬 실행 확인
  • 주요 시나리오 확인
  • 영향 범위 확인
  • npm run lint

리뷰 체크리스트

  • PR base branch가 올바릅니다. (feature/* -> develop, 배포 시 develop 또는 release/* -> main)
  • 브랜치명이 Type/#issue-number/description 형식을 따릅니다.
  • 커밋 메시지가 컨벤션을 따릅니다.
  • 불필요한 console.log, 주석, 임시 코드를 제거했습니다.
  • 타입 에러와 린트 에러를 확인했습니다.
  • CodeRabbit 1차 리뷰를 확인했습니다.
  • CodeRabbit 리뷰 반영 후 Discord에 공유했습니다.
  • 최소 1명 이상의 approve 후 merge합니다.

리뷰 요청사항

근무자 아바타의 고정 크기와 자동 줄바꿈이 카드 레이아웃에 자연스럽게 적용되는지 확인 부탁드립니다.

관련 이슈

Closes #93

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
syncly Ready Ready Preview, Comment Jul 23, 2026 7:42am

@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

자료 수정 다이얼로그의 URL·제목·설명 입력에 시각적 라벨을 추가하고, 근무자 아바타 목록이 여러 줄로 표시되도록 레이아웃을 조정했습니다.

Changes

UI 레이아웃 업데이트

Layer / File(s) Summary
자료 수정 입력 라벨 개선
src/features/manage-resources/ui/ResourceEditDialog.tsx
URL, 제목, 설명 입력을 각각 라벨과 함께 표시하고 설명 플레이스홀더를 변경했습니다.
근무자 아바타 줄바꿈
src/features/manage-work-schedule/ui/WorkScheduleBoard.tsx
아바타 목록에 줄바꿈을 적용하고 각 아바타가 축소되지 않도록 조정했습니다.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • TeampleRun/syncly#12: WorkScheduleBoard의 근무자 아바타 레이아웃 변경과 직접적으로 연결됩니다.

Suggested reviewers: kwon812

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed #93의 입력 라벨 추가와 아바타 고정 크기·줄바꿈 개선 요구를 코드 변경이 모두 반영합니다.
Out of Scope Changes check ✅ Passed 변경 범위는 두 UI 개선 항목에 한정되어 있고, 요구사항과 무관한 추가 수정은 보이지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed 제목이 자료 수정 입력 라벨 추가와 근무자 아바타 UI 개선이라는 핵심 변경을 간결하게 요약합니다.
Description check ✅ Passed 템플릿의 작업 내용, 결과, 변경 사항, 실행화면, 테스트, 리뷰 체크리스트, 이슈 항목을 대부분 충족합니다.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch design/#93/resource-edit-and-schedule-avatar-layout

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.

@JiWoongE JiWoongE left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다~

@seongjinss555
seongjinss555 merged commit c4cbbbf into develop Jul 23, 2026
3 checks passed
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.

자료 수정 입력 라벨 및 근무자 아바타 레이아웃 개선

2 participants