Skip to content
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] 세팅 페이지 취소, 삭제 버튼 추가 #156

Merged
merged 3 commits into from
Jul 17, 2024

Conversation

seong-hui
Copy link
Member

작업 내용 🧑‍💻

  • 세팅 페이지에서 계정 삭제 버튼을 누를 때 취소, 삭제 버튼이 나타나도록 했습니다.
  • 기존에 만들어두었던 ModalDeleteDetail 컴포넌트를 사용하였습니다.
  • 기존에는 버튼에 onClick이 없어서 props에 onClick을 추가하고 취소 버튼에는 close 함수를 연결하였습니다.

알게된 점 🚀

기록하며 개발하기!

리뷰 요구사항 💬

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 확인해보시고 추가 수정할 부분 말씀해주세요!

관련 이슈

close #153

스크린샷 (선택)

Jul-16-2024 22-34-18

Copy link
Member

@jeeminyi jeeminyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다! 빌드 에러만 잡으면 될 것 같습니당 ㅎ.ㅎ

return (
<AccountDeleteBtnContainer>
<SettingCheckBtn size="small" type="close" isHover isPressed={false} isActive onClick={handleClickBtn} />
{isModalOpen && <ModalDeleteDetail onClose={handleCloseModal} top={321} left={482} />}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오호 요런 식으로 모달 위치를 지정해주었군요!

@wrryu09
Copy link
Member

wrryu09 commented Jul 17, 2024

버튼에 props로 넘겨주는 onClose 함수 타입을 (e: React.MouseEvent) => void; 로 변경해 빌드에러 수정했습니다.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

세팅 페이지 정리까지 해주셨네요 ㅎㅎ 감사합니당

@wrryu09 wrryu09 merged commit bd60ab9 into develop Jul 17, 2024
2 checks passed
@wrryu09 wrryu09 deleted the feat/#153/setting-BtnADd branch July 17, 2024 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FEAT] 세팅 페이지 취소, 삭제 버튼 추가
3 participants