-
Notifications
You must be signed in to change notification settings - Fork 0
[Feat] Image Input(모임 생성 페이지 Thumbnail Field) 제작 #84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
🎨 Storybook Reportℹ️ Story 변경사항이 감지되지 않았습니다 이 PR에는 Story 변경이 없어서 빌드를 스킵했습니다.
|
📊 Coverage Report
📉 #84을 main에 병합하면 coverage가 Coverage 요약@@ Coverage Diff @@
## main #84 +/- ##
===========================================
- Coverage 36.85% 34.91% -1.94%
===========================================
Files 80 81 +1
Lines 2491 2692 +201
Branches 158 158 0
===========================================
+ Hits 918 940 +22
+ Misses 1573 1752 +179 영향받은 파일
|
🚀 PR Preview Report✨ Build가 성공적으로 완료되었습니다. Preview에서 변경사항을 확인하세요.
|
| className={cn( | ||
| 'size-6 text-gray-600', // 기본 스타일 | ||
| 'group-hover:scale-120', // hover 스타일 | ||
| 'transition-all duration-300', // animation 스타일 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
스타일 별로 묶어서 작성하니까 보기 좋아요!👍
|
|
||
| // append 모드면 이미지 계속 쌓임 | ||
| // replace 모드면 이미지 교체 | ||
| const nextImages = mode === 'append' ? { ...images, ...newImages } : newImages; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replace 모드는 이미 추가되어 있는 이미지를 누르면 적용되는건가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이렇게 말씀하시니까 그렇게 보이기도 하네요 ...
제 의도는 예를들어 이미지가 이미 선택되어있는 상태에서 추가로 이미지를 선택하면 이게 교체될꺼냐(replace), 아니면 쌓일거냐(append)를 결정하려고 했어요!
wooktori
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!
📝 변경 사항
프로필 Image Input은 추후 제작 예정입니다.
💡
components/ui/imageinput/index.tsximage input Headless(core 로직) 컴포넌트입니다.
💡
components/pages/post-meetup/fields/thumbnails-field/index.tsximageinput 컴포넌트에 디자인 적용된 컴포넌트입니다.
아래와 같이 사용 가능합니다.
ImageRecord의 Type은 아래와 같습니다.
즉, 아래와 같이 url(blob 포함)이 key가 되고 File 객체가 값이 됩니다.
🔗 관련 이슈
Closes #
🧪 테스트 방법
테스트와 스토리는 추후 추가하겠습니다.
📸 스크린샷 (선택)
📋 체크리스트
💬 추가 코멘트
CodeRabbit Review는 자동으로 실행되지 않습니다.
Review를 실행하려면 comment에 아래와 같이 작성해주세요