Feature/#282 qa 모달및유저프로필 #283
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
horororok
requested review from
crazyupinc-design,
haksoo0918,
junghwaYang and
solprime
junghwaYang
requested changes
Dec 30, 2024
junghwaYang
reviewed
Dec 30, 2024
haksoo0918
approved these changes
Dec 30, 2024
Collaborator
haksoo0918
left a comment
haksoo0918
left a comment
There was a problem hiding this comment.
정화님이 꼼꼼하게 봐주셔서 제가 더 얹을 말이 없네요. 😄
수고하셨습니다. 👍
Collaborator
Author
|
수정했습니다. |
Collaborator
@horororok mo,ta일때 가이드 문구가 사라지는것 같 습니다 |
Collaborator
Author
|
해결했습니다. |
crazyupinc-design
approved these changes
Dec 30, 2024
Collaborator
crazyupinc-design
left a comment
crazyupinc-design
left a comment
There was a problem hiding this comment.
양 사이드 여백, 안내문구, 프로필 이미지 크기, 간격 확인했습니다
모바일에서 모달이 전체 영역을 사용함-> 양 사이드 여백이 필요
Collaborator
양 사이드 여백, 안내문구, 프로필 이미지 크기, 간격 확인했습니다 |
junghwaYang
approved these changes
Dec 30, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
이슈 번호
close #282
변경 사항 요약
모바일에서 모달이 전체 영역을 사용함-> 양 사이드 여백이 필요
위키 퀴즈 모달에서 타이틀과 안내문구가 다릅니다
[pc]프로필 이미지 크기 다름 200x200 으로 수정
"[pc] UserProfile.tsx:65
패딩값이 다름 프로필 위아래 패딩 60px 가져가는거면 메인 컨테이너는 px:30px, pb:47 로 맞춰야함"
[pc] UserProfile 수정하기전 거주 도시 .. 리스트 간격 12에서 16으로 수정
[ta, mo] 태블릿 간격 및 화살표 위치 등 전체 다시
Doc
컴포넌트 인 경우 props를 입력해주세요hooks 및 utils 인 경우 input,output을 입력해주세요테스트 결과
베이스(develop) 브랜치에 포함되기 위한 코드는 모두 정상적으로 작동이 되어야 합니다.컴포넌트의 경우 스크린샷을 포함해주세요.