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] 잔디 활동 로그 카드 제작 #401

Merged
merged 29 commits into from
Dec 4, 2023
Merged

Conversation

42sungwook
Copy link
Contributor

@42sungwook 42sungwook commented Nov 11, 2023

Summary

잔디 활동 로그 카드 제작
아직 못끝냈습니다,,, api만 붙이면 되는데 시간이 조금 더 걸릴 것 같아서 진행상황 공유드립니다

Describe your changes

  • 대쉬보드 -> 저번에 제작했던 임시 대쉬보드 사용 (row 2칸짜리랑 1칸짜리가 기존 대쉬보드에서는 적용안되기 때문)
  • 뷰 제작
  • 팀페이지 링킹
스크린샷 2023-11-11 오후 5 12 07 스크린샷 2023-11-11 오후 5 12 34 스크린샷 2023-11-11 오후 5 14 49

남은 일

  • atom에 상태 저장 (잔디의 날짜를 클릭한게 아니면 최신 날짜를 받으려 했는데 로직을 보니 어떻게 받을지 좀 막막하네요)
  • 로그별 합계 구하기 (요것도 할라면 금방 하는데 계산을 두번 돌면서 해야될 것 같아 고민이에요)
  • 대쉬보드 반응형 (위에서 바꾸느라 개고생했기 때문에 요건 금방 합니다)
  • dailyActivity 세로 구분선(?) 이게 컴포넌트 오버플로우가 생기니까 밑에까지 선이 안이어지네요 ㅡ.ㅡ 고쳐보려했는데 잘 안되서 더 늦어졌습니다
스크린샷 2023-11-11 오후 5 13 50

Issue number and link

@42sungwook 42sungwook self-assigned this Nov 11, 2023
@42sungwook 42sungwook linked an issue Nov 11, 2023 that may be closed by this pull request
@yoopark
Copy link
Contributor

yoopark commented Nov 11, 2023

아하 하드코딩된 상황이군요. 제가 밤에 볼 수 있으면 후속 커밋으로 올릴게요 ㅎㅎ

@42sungwook
Copy link
Contributor Author

혹시 작업하시면 말해주세여 저도 새벽에 보려고 해요!

@yoopark
Copy link
Contributor

yoopark commented Dec 3, 2023

image

Overflow에 대하여 고민하다가 이렇게 표현하는게 나을 것 같아 뷰를 변경... 합니다.

@yoopark
Copy link
Contributor

yoopark commented Dec 3, 2023

image

일별 활동 내역 카드 크기를 축소하였습니다.
뭔가 잔디 오른 편에 이 내역이 있으면 보기 깔끔할 것 같긴 한데.. 고민 중입니다!

@yoopark
Copy link
Contributor

yoopark commented Dec 3, 2023

선택한 연도 내에 아무런 활동도 없는 경우 (최근 활동일자 없음) : 일단 귀찮아서 오늘 날짜를 띄우도록 하였습니다.
image

if (latestDailyActivity === undefined) {
setSelectedDailyActivity({
date: new Date().toString(), // FIXME: 해당 Year 구간의 마지막 날짜로 변경
records: [],
});

@yoopark
Copy link
Contributor

yoopark commented Dec 3, 2023

웬만한 건 다 고친 듯한데, 추가적인 버그 수정과 자세한 코드 리뷰는 오늘 오후에 성욱님과 직접 만나서 진행하겠습니다 😀

[현재 진행상황 캡처]
image
image
image

- 현재일이 있는 경우 해당 x 좌표로 scrollTo
- 클릭 시 상세가 나오므로, hover 시 효과 추가
- 약간 정렬이 안 맞는 문제 등
Copy link
Contributor

@yoopark yoopark left a comment

Choose a reason for hiding this comment

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

만나서 코드 리뷰하였습니다.
이후 date-fns 도입하여 리팩토링하겠습니다. #404

@42sungwook 42sungwook merged commit b18a5bf into develop Dec 4, 2023
1 check passed
@42sungwook 42sungwook deleted the feat/grass-activity branch December 4, 2023 10:38
@42sungwook 42sungwook added the enhancement New feature or request label Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Feat] 잔디 활동내역 카드 제작
2 participants