Skip to content

[FIX] 아이콘 추가 및 MediaUpload max height 제거#106

Merged
yougyung merged 4 commits intomainfrom
fix/#97
Nov 17, 2025
Merged

[FIX] 아이콘 추가 및 MediaUpload max height 제거#106
yougyung merged 4 commits intomainfrom
fix/#97

Conversation

@yougyung
Copy link
Copy Markdown
Collaborator

@yougyung yougyung commented Nov 17, 2025

🔥 연관 이슈

🚀 작업 내용

  • 동아리 점수 아이콘 추가
  • 통계 아이콘 추가
  • MediaUpload max height 제거를 통한 스크롤 방지

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 7개의 새로운 아이콘 추가: 차트, 점, 청소용품, 사람들, 더하기/빼기, 보고서, 단축 보고서
  • 스타일

    • 미디어 업로드 컴포넌트 레이아웃 개선 및 높이 제한 제거

@yougyung yougyung linked an issue Nov 17, 2025 that may be closed by this pull request
3 tasks
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Nov 17, 2025

Caution

Review failed

The pull request is closed.

개요

미디어 업로드 컴포넌트의 스크롤 제약을 제거하고 새로운 아이콘 7개(chart, dots, dustpan, peoples, plusMinus, report, shortReport)를 아이콘 라이브러리에 추가했습니다.

변경 사항

코호트 / 파일(들) 변경 요약
UI 레이아웃 개선
src/shared/ui/MediaUpload/MediaUpload.tsx
래퍼 div의 클래스를 max-h-[500px] w-full overflow-auto에서 w-full로 변경하여 고정 높이와 오버플로우 스크롤 동작 제거
아이콘 추가
src/shared/ui/assets/index.tsx
7개 새로운 아이콘(chart, dots, dustpan, peoples, plusMinus, report, shortReport) 임포트 및 public Icons 매핑에 추가

예상 코드 리뷰 난이도

🎯 1 (간단함) | ⏱️ ~5분

  • MediaUpload 레이아웃 변경이 다른 컴포넌트에 미치는 영향 확인
  • 새로 추가된 아이콘들이 모두 올바르게 임포트되고 매핑되었는지 검토

연관된 가능성 있는 PR

제안 라벨

fix

제안 리뷰어

  • keemsebin
  • ujinsim
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#97

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 70cde93 and e27c6cd.

⛔ Files ignored due to path filters (7)
  • src/shared/ui/assets/chart.svg is excluded by !**/*.svg
  • src/shared/ui/assets/dots.svg is excluded by !**/*.svg
  • src/shared/ui/assets/dustpan.svg is excluded by !**/*.svg
  • src/shared/ui/assets/peoples.svg is excluded by !**/*.svg
  • src/shared/ui/assets/plusminus.svg is excluded by !**/*.svg
  • src/shared/ui/assets/report.svg is excluded by !**/*.svg
  • src/shared/ui/assets/shortReport.svg is excluded by !**/*.svg
📒 Files selected for processing (2)
  • src/shared/ui/MediaUpload/MediaUpload.tsx (1 hunks)
  • src/shared/ui/assets/index.tsx (2 hunks)

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 and usage tips.

@yougyung yougyung merged commit efd46ca into main Nov 17, 2025
@yougyung yougyung deleted the fix/#97 branch November 17, 2025 09:00
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 17, 2025

Update: 2025년 11월 17일 18시 02분 25초
Storybook 미리보기: https://677a9a76c58a295e9421d3ef-apzomlzwoj.chromatic.com/

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.

[FIX] icon 추가 및 이미지업로드 스크롤 개선

1 participant