-
Notifications
You must be signed in to change notification settings - Fork 3
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
[Feat/#89] 반려동물 성별 뷰 #92
Conversation
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.
PR #85 랑 유사해서 어푸하겠습니다 -! 고생하셨어요!!
{data.map((item) => ( | ||
<div | ||
key={item.id} | ||
className={styles.selector({ | ||
state: selectedOption === item.id ? "selected" : "unselected", | ||
})} | ||
onClick={() => handleOptionClick(item.id, item.label)} | ||
> | ||
{item.label} | ||
<img src={item.image} alt={item.label} /> | ||
</div> | ||
))} |
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.
p5) 아하 #85 -!
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.
어푸할게요~
나중에 기회되면 Prograss Bar 에 애니메이션도 넣어봅시다!
야미 빨리 뷰 끝내고 해보겠습니당! |
🔥 Related Issues
✅ 작업 리스트
🔧 작업 내용
전 85번 pr에
dualOptionSelector
컴포넌트 리팩토링 한거 반영해서 그거에 맞게 새로 상수 파일 만들었습니다.하지만 성별 뷰는 별도의 api가 연결되는 것은 아닌데요... 공유하는 컴포넌트가 api 연결하는 컴포넌트라 부득이 하게 목데이터 파일 만들었습니다.
📣 리뷰어에게 어떠신가요?
#85번 pr과 매우!!!! 유사합니다 한 번에 코리하시면 리소스 낭비 없으셔요 ^,^
넋두리 주석은 컴포넌트 조립시 삭제할 예정입니다.
progressbar
나 이동 버튼 역시 조립 시 구현 예정입니다. (고려해서 구현해두었으니 돈워리)📸 스크린샷 / GIF / Link
default.mov