-
Notifications
You must be signed in to change notification settings - Fork 5
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
[FE] 마이페이지 화면 퍼블리싱 #839
base: fe-dev
Are you sure you want to change the base?
[FE] 마이페이지 화면 퍼블리싱 #839
Conversation
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.
멋집니댱, 쿠키! 🍪
<TextButton textColor="onTertiary" textSize="body"> | ||
닉네임 설정하기 | ||
</TextButton> | ||
<TextButton textColor="onTertiary" textSize="body"> | ||
기본 계좌 번호 설정하기 | ||
</TextButton> | ||
<TextButton textColor="onTertiary" textSize="body"> | ||
내가 만든 행사 목록 보기 | ||
</TextButton> | ||
<TextButton textColor="onTertiary" textSize="body"> | ||
탈퇴하기 | ||
</TextButton> |
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.
나중에 기능 추가할 때, 해당 항목들과 이동 Url을 배열로 정리해서 렌더링시키는 것도 좋을 것 같아요~
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.
탈퇴하기 부분은 충분히 destructive
한 기능이니까 color를 destructive로 적용해 줘도 좋을 것 같아요
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.
좋아요좋아요~ 고생많았습니다. 쿠키~ 저도 남은 기능을 이번 주에 많이 개발해볼게요!
<Flex | ||
flexDirection="column" | ||
width="100%" | ||
gap="0.5rem" | ||
backgroundColor="white" | ||
padding="1rem" | ||
css={{borderRadius: '0.75rem'}} |
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.
Flex 컴포넌트가 사용되는 모습을 보니 어색하네요. 간편하게 쓰기 위해서 여러 props를 열어두었는데, css prop도 받는걸 보니 누군가는 '어라 그냥 css에 다 집어넣으면 되지 왜 따로갈비지?'라고 생각할 수도 있을 것 같아서요.. 다양한 사용 방법을 모두 수용하다보니 이렇게 된 것 같기도 하구요. 어떻게 만드는게 좋았을 지 고민이 되네요. 고쳐달라는 뜻은 아니고 아마 저와 같은 생각을 하는 사람이 있을 것 같아 적어봤써요 😭
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.
저도 어느정도 동감해요! css를 인자로 받아서, 다른 component처럼 css를 넘겨줄 수 있도록 하는게 제일 깔끔해 보이기는 해요!
좋은 방법이 있을지 고민해 보도록 하겠습니다 :)
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.
고생많았습니다~~! :)
<Flex | ||
flexDirection="column" | ||
width="100%" | ||
gap="0.5rem" | ||
backgroundColor="white" | ||
padding="1rem" | ||
css={{borderRadius: '0.75rem'}} |
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.
저도 어느정도 동감해요! css를 인자로 받아서, 다른 component처럼 css를 넘겨줄 수 있도록 하는게 제일 깔끔해 보이기는 해요!
좋은 방법이 있을지 고민해 보도록 하겠습니다 :)
<TextButton textColor="onTertiary" textSize="body"> | ||
닉네임 설정하기 | ||
</TextButton> | ||
<TextButton textColor="onTertiary" textSize="body"> | ||
기본 계좌 번호 설정하기 | ||
</TextButton> | ||
<TextButton textColor="onTertiary" textSize="body"> | ||
내가 만든 행사 목록 보기 | ||
</TextButton> | ||
<TextButton textColor="onTertiary" textSize="body"> | ||
탈퇴하기 | ||
</TextButton> |
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.
탈퇴하기 부분은 충분히 destructive
한 기능이니까 color를 destructive로 적용해 줘도 좋을 것 같아요
issue
구현 사항
마이페이지 퍼블리싱 작업 수행
로그인 후 사용자가 혜택을 누릴 수 있는 화면, 마이페이지 화면 퍼블리싱 작업만 수행했습니다.
이 페이지에서 로그아웃, 닉네임 설정, 계좌번호 설정, 내가 만든 행사 목록, 탈퇴 기능을 수행할 수 있습니다.
피그마에 있는 레이아웃 대로 작업했으며 기능은 추후에 추가할 예정입니다.
🫡 참고사항