-
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: 멤버 프로필 등록/수정 mds 적용 #1717
base: main
Are you sure you want to change the base?
Conversation
|
🚀 프리뷰 배포 확인하기 🚀 |
0ebf32e
to
d9a78c2
Compare
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.
넘 고생하셨습니다!!! 코멘트 남긴 내용 한 번 확인 부탁드려용
🤫 쉿, 나한테만 말해줘요. 이슈넘버
🧐 어떤 것을 변경했어요~?
멤버 수정 폼 mds 적용
🤔 그렇다면, 어떻게 구현했어요~?
❤️🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.
files changed가 많은데, 자잘한 css 수정과 mds Select 적용을 위한 상수 수정 등이 대부분입니다. 이런 부분은 가볍게 봐주시고,
pc/mobile 반응형 UI에 문제가 없는지, 폼 수정 동작이 문제 없이 잘 동작하는지를 위주로 확인해주시면 좋을 것 같습니다!!
코드상에서 가장 영향이 있는 부분은 mds TextField/TextArea/Select 교체 부분일듯합니다. 이 또한 반복적인 코드여서 모두 확인하실 필요는 없어보입니다.
pc/모바일에서 각각 select/bottomsheet로 달라져야하는 부분과, pc에서는 textField인데 mobile에서는 textArea인 경우가 많아서 이는 전부
<Responsive>
로 처리했습니다.이 외 레이아웃 반응형에 대해서는 최대한 media query로 해결했습니다.
📸 스크린샷, 없으면 이것 참,, 섭섭한데요?