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

마이페이지 구현 #46

Merged
merged 24 commits into from
Jun 8, 2024
Merged

마이페이지 구현 #46

merged 24 commits into from
Jun 8, 2024

Conversation

yeeZinu
Copy link
Contributor

@yeeZinu yeeZinu commented Jun 6, 2024

Description

마이페이지에 공통 컴포넌트를 붙여 기본적인 틀을 구현하였습니다.
mock data로 구현하였으며 api를 붙일 예정입니다.

Changes Made

  • 유저정보에서 설명글을 작성할 때 자동줄바꿈을 하기위해 textfiled를 이용할 계획입니다. 줄바꾼내용도 string으로 들어가는것으로 알고있고 해당 내용을 받아와서 html로 변환시켜
    태그를 달아주려합니다.
  • 리뷰, 생성한물품, 찜한물품에 대한 api는 기본적으로 review를 기준으로 하고있으며 추후 선택된 내용에 따라 api를 불러오려합니다.
  • 뷰포트의 사이즈에따라 드롭다운 컴포넌트를 사용해야했습니다. useBrowserSize훅을 만들어 대응하였습니다.
  • layout.tsx에서 유저저 활동 내용(찜, 별점 평균, 관심카테고리 등)의 내용을 받아오고 유저프로필만 하위페이지에서 쓰는 방향으로 바꿔보려고합니다. 찾는중인데 잘 모르겠습니다...

Screenshots

image
image
image

IssueNumber

[#35]

@yeeZinu yeeZinu added ✨ Feature 기능 개발 ✨ Component 컴포넌트 작업 📜 Page 페이지 관련 작업 labels Jun 6, 2024
@yeeZinu yeeZinu self-assigned this Jun 6, 2024
@yeeZinu yeeZinu requested review from aoooec and suzinxix as code owners June 6, 2024 18:55
@yeeZinu yeeZinu requested review from yb3143 and zoonyoung as code owners June 6, 2024 18:55
Copy link
Contributor

@aoooec aoooec left a comment

Choose a reason for hiding this comment

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

유저 설명글이 프로필 desc 부분인가요? width 설정 외에 자동 줄바꿈이 필요한 이유가 뭘까용??
혹시 유저가 입력한 개행문자가 표시되게 하기 위해서일까요?! 찾아보니까 textarea로 받아온 stringwhite-space: pre-wrap 스타일 적용해도 되는거같아요! 직접 해본게 아니라서 확신은 없지만 ㅎㅎ;;
드롭다운 관련해서 말씀해주셨던거 혹시 해결 안되시면 팀미팅때 한 번 더 얘기해주세요~! 고생하셨습니다 👍

Copy link
Contributor

@suzinxix suzinxix left a comment

Choose a reason for hiding this comment

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

드롭다운 수정시켜 놓을게요! 수고하셨습니닷!

@yeeZinu yeeZinu merged commit bc7809a into develop Jun 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Component 컴포넌트 작업 ✨ Feature 기능 개발 📜 Page 페이지 관련 작업
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants