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

Design: 마이페이지 설정으로 이름 변경 및 font, theme ver3.0 적용 #266

Merged
merged 4 commits into from
Oct 28, 2024

Conversation

seoyoung-min
Copy link
Contributor

@seoyoung-min seoyoung-min commented Oct 27, 2024

개요

  • 디자인이 되지 않은 페이지라, 일단 ver3.0 색과 폰트 중 골라서 적용했습니다. 하지만 영 별로여서 리디자인 곧 할 예정입니다!

작업 사항

  • 마이페이지 > 설정 이름변경
  • ver3.0 공용헤더 적용
  • 서비스 배경 background color 변경
  • 폰트, hover color ver 3.0 시스템 중 적용

참고 사항 (optional)

  • TODO
    • 공지 및 안내 메뉴 추가해야하지만, 미개발된 페이지로 추후 추가할 예정입니다!
    • 다시 보기 좋게 디자인 해보겠습니다. (벤토 디자인 적용, 언어선택 드롭다운 대신 segmented control로 변경, 카테고리별 리스트 분류)
    • language 상관없이 한국어는 한국어 영어는 'ENG'으로 고정되게끔 변경

관련 이슈 (optional)


스크린샷

설정

⬇︎ 다시 디자인 한 버전

Screenshot 2024-10-28 at 21 38 07

리뷰어에게

  • 어떤 부분에 리뷰어가 집중하면 좋을지

Copy link

vercel bot commented Oct 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
listywave ❌ Failed (Inspect) Oct 28, 2024 0:42am

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~✨🥹💕
+. PR 머지하면 해당 브랜치는 삭제해야 될 것 같아요..! 👀

Comment on lines 27 to 31
<h1 className={`${styles.headerTitle} ${styles.flexChild}`}>{title}</h1>

<div className={`${styles.flexChild} ${styles.rightChild}`}>{right}</div>
{right === null ? <></> : <div className={`${styles.flexChild} ${styles.rightChild}`}>{right}</div>}
</div>
);
Copy link
Contributor

Choose a reason for hiding this comment

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

서영님, right이 null일때 Fragment를 추가해 둔 이유가 궁금합니다~!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

소현님 저도 제가 왜그랬는지 궁금합니다..! 이 점 반영하여 리팩토링 하였고, 페이지 디자인 다시 하여 수정하였습니다~!
그리고 git push도 upstream으로 곧바로 해버렸네요ㅠㅠ 이번에는 머지 후 지우고, 다음부터는 origin push 후에 PR 올리겠습니다~!

Copy link
Contributor

Choose a reason for hiding this comment

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

서영님 새로 반영한 디자인 너무 보기 좋아요!! 👍 또, 드롭다운이 아닌 토글모양 선택도 직관적으로 옵션을 보여주는 것 같아요✨ 역시 최고입니다!!🥹

@seoyoung-min seoyoung-min merged commit 7c238e3 into dev Oct 28, 2024
1 of 3 checks passed
@seoyoung-min seoyoung-min deleted the feature/account branch October 28, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design UI 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants