Skip to content

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 삭제 모달 공용 컴포넌트 만들어서 마이피드 리스트 삭제 기능에도 적용했습니다! 확인 부탁드려요🙇‍♀️

@vercel
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~~💕💕

@Eugene-A-01
Copy link
Contributor

새로운 팔로우 팔로잉 페이지 너무 트렌디해요 ㅎㅎ
정말 알잘딱깔센.. 재확인 모달이랑 인원없앤것도 넘 좋네요 감사합니다 서영님 LGTM!
(로티 컴포넌트에서 CI 빌드 에러 났네요!! :0

@seoyoung-min seoyoung-min merged commit 24c79df into 8-Sprinters:dev Jan 14, 2025
1 of 2 checks passed
@youdame
Copy link

youdame commented Jan 14, 2025

안녕하세요.. 저도 ,,, 리스티웨이브 개발자 하고싶어요..... @seoyoung-min @Eugene-A-01 @ParkSohyunee

@seoyoung-min
Copy link
Contributor Author

안녕하세요.. 저도 ,,, 리스티웨이브 개발자 하고싶어요..... @seoyoung-min @Eugene-A-01 @ParkSohyunee

어맛 유담님!!>0<// 너무 반가워요...🥹🤍🤍

@seoyoung-min seoyoung-min deleted the design/followings branch January 15, 2025 07:22
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.

4 participants