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

✨ 프로필 UI 구현 #77

Merged
merged 27 commits into from
May 23, 2024
Merged

✨ 프로필 UI 구현 #77

merged 27 commits into from
May 23, 2024

Conversation

Legitgoons
Copy link
Member

@Legitgoons Legitgoons commented May 23, 2024

작업 이유

  • 사용자 프로필 UI 구현

작업 사항

1️⃣ Profile UI 구현

  • 유저 관련 정보를 보여주는 ProfileUser와 해당 유저가 작성한 Feed를 보여주는 ProfileFeed로 분리해서 구현했습니다.
  • 우선 구현해야하는 사항인 내 프로필 UI를 기준으로 구현하였습니다.
    • 내 프로필에서만 사용되는 Header인 ProfileHeader를 따로 구현했습니다.
    • 추후 API 구현 과정에서 id에 따라 내 프로필상대방 프로필로 이동할 수 있도록 추가할 예정입니다.
  • 🐛 Params URL에 Icon Sprite 미 적용 버그 #75 로 인해 우선 '/user:id' 대신 '/user'를 경로로 설정하였습니다.

2️⃣ 기타 수정

  • Feed footer 수정
    • UI 변경으로 likeCount와 commnetCount의 위치 및 font를 수정했습니다.
  • PageHeader 수정
    • div태그에서 header태그로 수정
    • p태그 부분 h4md 폰트 적용
  • FollowButton 개선
    • Follow에만 사용된다는 점을 고려 isFollow만을 Props로 받고, 이에 따라 동작하도록 개선

리뷰어가 중점적으로 확인해야 하는 부분

  • 구현된 UI가 Figma의 UI 기획과 일치하나요?
  • 수정된 컴포넌트들이 적절하게 수정되었나요?
  • 이슈에 대해서 인지하였나요?

발견한 이슈

#75 로 인해 우선 '/user:id' 대신 '/user'를 경로로 설정하였습니다.

@Legitgoons Legitgoons self-assigned this May 23, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-77.d37mn03xh3qyyz.amplifyapp.com

Copy link
Collaborator

@BangDori BangDori left a comment

Choose a reason for hiding this comment

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

PR 확인 부탁드릴게용~

src/app/routers/index.tsx Show resolved Hide resolved
src/features/profile-header/index.ts Outdated Show resolved Hide resolved
src/features/profile-header/ui/ProfileHeader.scss Outdated Show resolved Hide resolved
src/shared/ui/button/ConfirmModalButton.tsx Show resolved Hide resolved
src/shared/ui/button/FollowButton.tsx Outdated Show resolved Hide resolved
src/widgets/feed-main-list/ui/Feed.scss Show resolved Hide resolved
src/widgets/profile-feed-list/ui/ProfileFeedList.scss Outdated Show resolved Hide resolved
src/widgets/profile-feed-list/ui/ProfileFeedList.tsx Outdated Show resolved Hide resolved
src/widgets/profile-user/ui/ProfileUser.scss Show resolved Hide resolved
src/widgets/profile-user/ui/ProfileUser.scss Outdated Show resolved Hide resolved
Copy link
Collaborator

@BangDori BangDori left a comment

Choose a reason for hiding this comment

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

확인했습니다! 병합하셔도 좋습니다 ㅎㅎ

@Legitgoons Legitgoons merged commit f533a73 into main May 23, 2024
2 checks passed
@Legitgoons Legitgoons deleted the feat/PW-344-profile-ui branch May 23, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants