-
Notifications
You must be signed in to change notification settings - Fork 1
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
편집 취소버튼 눌러도 저장 #103
편집 취소버튼 눌러도 저장 #103
Conversation
✔️ Deploy Preview for todaytheylearned ready! 🔨 Explore the source changes: 183acc2 🔍 Inspect the deploy log: https://app.netlify.com/sites/todaytheylearned/deploys/62160cd36760c100081a43da 😎 Browse the preview: https://deploy-preview-103--todaytheylearned.netlify.app/ |
pr은 프로젝트 연결은 안해도 된답니단 |
base브랜치 바꿔야해요 ㅠㅠ 지금 main 브랜치에 병합중이어서 제가 develop으로 고쳐놓음.. |
@@ -107,7 +107,8 @@ const ProfileEdit = () => { | |||
/> | |||
</Styled.AvatarFormContainer> | |||
<Styled.ActionContainer> | |||
<Styled.BackButton>취소</Styled.BackButton> | |||
<Styled.BackButton onClick={props.onChangeMode}>취소</Styled.BackButton> | |||
|
|||
<Styled.EditButton>저장</Styled.EditButton> |
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.
p2: 저장버튼은 form submit 용이면 option에 type="submit" 이런걸 넣어주는게 좋습니다
back button은 반대로 form submit 을 안하니까 type="button" 으로 지정해줍시다.
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.
댑악 넹
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.
확인했습니다 👍
버그라 하기도 애매한... 기능 추가가 맞는 듯