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

Space 화면 Header UI 구현 #144

Merged
merged 8 commits into from
Nov 28, 2024
Merged

Space 화면 Header UI 구현 #144

merged 8 commits into from
Nov 28, 2024

Conversation

hoqn
Copy link
Collaborator

@hoqn hoqn commented Nov 26, 2024

✏️ 한 줄 설명

이 PR의 주요 변경 사항이나 구현된 내용을 간략히 설명해 주세요.

Space 페이지의 헤더 UI를 구현했습니다.

✅ 작업 내용

  • Header UI를 구현했습니다.
  • Breadcrumb UI를 구현했습니다.
  • Header에서도 yjs connection에 접근할 수 있도록, yjs context provider를 끌어올렸습니다.

🏷️ 관련 이슈

📸 스크린샷/영상

이번 PR에서 변경되거나 추가된 뷰가 있는 경우 이미지나 동작 영상을 첨부해 주세요.

브레드크럼은 아래와 같이 동적으로 작동하도록 하였습니다.

둘 이하

스크린샷 2024-11-26 오후 2 13 29

스크린샷 2024-11-26 오후 2 13 21

넷 이상

스크린샷 2024-11-26 오후 2 13 10

헤더는 다음과 같이 구성하였습니다.

스크린샷 2024-11-27 오전 11 56 25

📌 리뷰 진행 시 참고 사항

리뷰 코멘트 작성 시 특정 사실에 대해 짚는 것이 아니라 코드에 대한 의견을 제안할 경우, 강도를 함께 제시해주세요! (1점: 가볍게 참고해봐도 좋을듯 ↔ 5점: 꼭 바꾸는 게 좋을 것 같음!)

5점: 우선은 Space 페이지 헤더 UI만 구현했습니다. 백엔드 연동으로 브레드크럼 실제로 띄우고, 공유 버튼 구현하는 부분은 별도 PR로 올리겠습니다!

@hoqn hoqn force-pushed the feature/breadcrumb branch from ce6cb15 to 35244ba Compare November 26, 2024 16:29
@hoqn hoqn self-assigned this Nov 27, 2024
@hoqn hoqn added the 2. Space label Nov 27, 2024
@hoqn hoqn added this to the 스페이스의 상단바 UI milestone Nov 27, 2024
@hoqn hoqn linked an issue Nov 27, 2024 that may be closed by this pull request
@hoqn hoqn changed the title Space 화면 상의 Breadcrumb UI 구현 Space 화면 Header UI 구현 Nov 27, 2024
@hoqn hoqn marked this pull request as ready for review November 27, 2024 02:59
@hoqn hoqn requested a review from a team as a code owner November 27, 2024 02:59
@hoqn hoqn requested review from heegenie and CatyJazzy and removed request for a team November 27, 2024 03:00
Copy link
Collaborator

@CatyJazzy CatyJazzy left a comment

Choose a reason for hiding this comment

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

빠르게 구현해서 올려주신 PR이었는데 .. 늦은 리뷰 죄송합니다.
코멘트는 사소한 것들이라 가볍게 체크해주셔도 될 것 같습니다 👍

Copy link
Collaborator

@heegenie heegenie left a comment

Choose a reason for hiding this comment

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

다른 부분은 지후님이 이미 리뷰를 잘 남겨 주셔서..
간단하게만 코멘트 남기고 Approve 하겠습니다 🐝

packages/frontend/src/components/SpaceBreadcrumb.tsx Outdated Show resolved Hide resolved
@hoqn hoqn merged commit aa044d8 into dev Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Breadcrumb UI 및 공유 버튼을 구현한다.
3 participants