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

Feature: 팔로워 팔로잉 페이지 디자인 수정 및 확인 기능 추가 #288

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

seoyoung-min
Copy link
Contributor

@seoyoung-min seoyoung-min commented Dec 29, 2024

개요

  • 팔로워, 팔로잉 페이지 ver3.0에 맞추어 업데이트 했습니다.

작업 사항

  • ver3.0 디자인에 맞추어 팔로워 팔로잉 페이지 디자인을 일부 수정했습니다. (폰트, 색상, border 적용)
    -기존에는 팔로워 삭제 버튼 클릭하면 바로 삭제가 되었습니다. 실수 발생 가능성이 있어 '정말 삭제하시겠습니까?' 모달로 한번 확인할 수 있도록 변경 했습니다.
  • '총 n명' 표시를 제거했습니다. 중요한 정보가 아니라고 판단했기 때문입니다.

스크린샷

Screenshot 2024-12-29 at 20 54 00 Screenshot 2024-12-29 at 20 53 53

리뷰어에게

  • 총 n명 삭제한 것, 삭제 모달 공용 컴포넌트로 뺀 것 다른 의견 있으시면 편하게 말씀해 주세요!
  • @ParkSohyunee 삭제 모달 공용 컴포넌트 만들어서 마이피드 리스트 삭제 기능에도 적용했습니다! 확인 부탁드려요🙇‍♀️

Copy link

vercel bot commented Dec 29, 2024

@seoyoung-min is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel.

A member of the Team first needs to authorize it.

Comment on lines +96 to +106
<DeleteModal
handleClose={handleModalOff}
handleDelete={(e) => {
e.stopPropagation();
deleteListMutation.mutate();
}}
handleCancel={(e) => {
e.stopPropagation();
handleModalOff();
}}
/>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ParkSohyunee 이 부분입니다! 확인 부탁드립니다🙇‍♀️

Copy link
Contributor

@ParkSohyunee ParkSohyunee left a comment

Choose a reason for hiding this comment

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

서영님~ 팔로우, 팔로잉 페이지 디자인 너무 맘에들어요! 🎨✨ 또, 삭제 모달도 함께 적용해 주셔서 감사합니다. 👍🥹 LGTM~~💕💕

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design UI 수정 Feat 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants