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

[Feat] 대시보드 timeline date 이슈 및 css 수정 #325

Closed
wants to merge 2 commits into from

Conversation

Bowoon1216
Copy link
Contributor

@Bowoon1216 Bowoon1216 commented Nov 12, 2024

해당 이슈 번호

closed #324


체크리스트

  • 🔀 PR 제목의 형식을 잘 작성했나요? e.g. [feat] PR을 등록한다.
  • 💯 테스트는 잘 통과했나요?
  • 🏗️ 빌드는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요?
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 💻 git rebase를 사용했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

💎 PR Point

  • 타임라인부분에서 달이 넘어가지 않는 이슈를 타임라인의 내용(Day 컴포넌트 아래)과 헤더부분을 분리함으로 해결했습니다.

  • 내용부분에 useDateContext를 사용해야 했는데, 그러기 위해서는 헤더에 필요한 DateProvider 컴포넌트와 분리되어야 하기 때문에 내용 컴포넌트를 분리했습니다.

  • 인수인계 부분에서는 첫번째 태그의 글자수가 200픽셀 너비를 넘었을 때 아예 표시되지 않고 숫자만 (ex. '+4') 표시되는 문제가 있어서

  • 보여주는 태그를 선정하는 로직(getVisibleTags)에 첫번째 태그일 경우 말줄임표와 함께 표시하는 로직을 구현했습니다.


📌스크린샷 (선택)

2024-11-12.101518.mp4

Copy link

🚀 Storybook 확인하기 🚀

@Bowoon1216 Bowoon1216 changed the title [Hotfix] 대시보드 timeline date 이슈 및 css 수정 [Feat] 대시보드 timeline date 이슈 및 css 수정 Nov 12, 2024
@Bowoon1216 Bowoon1216 closed this Nov 12, 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.

대시보드 타임라인 날짜 이슈 및 자잘한 css 수정
1 participant