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] 캘린더 v2 변경 #328

Open
wants to merge 19 commits into
base: develop
Choose a base branch
from
Open

[FEAT] 캘린더 v2 변경 #328

wants to merge 19 commits into from

Conversation

Kjiw0n
Copy link
Contributor

@Kjiw0n Kjiw0n commented Jan 3, 2025

작업 내용 🧑‍💻

  • 캘린더 v2 디자인 반영
  • 기존 일/주/월 -> 주/월 으로 캘린더 옵션 축소
  • size에 따라 small -> 주 5일 캘린더, big -> 주 7일 캘린더로 변경
  • 타임그리드 범위 조정 (30분 -> 15분)

알게된 점 🚀

기록하며 개발하기!

  • fullcalendar 사용법에 대해 알게되었습니닷.

리뷰 요구사항 💬

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 디자인 이슈 있는 지 확인 부탁드립니다 🙇‍♀️

관련 이슈

close #322

스크린샷 (선택)

size = 'small'
스크린샷 2025-01-03 16 26 18

size = 'big'
스크린샷 2025-01-03 16 26 44

@Kjiw0n Kjiw0n force-pushed the feat/#322/calendar-v2 branch from 3900c2f to 3c0f862 Compare January 3, 2025 11:10
@TEAM-DAWM TEAM-DAWM deleted a comment from github-actions bot Jan 3, 2025
@TEAM-DAWM TEAM-DAWM deleted a comment from github-actions bot Jan 3, 2025
Copy link

github-actions bot commented Jan 3, 2025

@Kjiw0n Kjiw0n force-pushed the feat/#322/calendar-v2 branch from fccdb2f to 345bc24 Compare January 3, 2025 12:51
@TEAM-DAWM TEAM-DAWM deleted a comment from github-actions bot Jan 3, 2025
Copy link

github-actions bot commented Jan 3, 2025

@wrryu09
Copy link
Member

wrryu09 commented Jan 3, 2025

Screenshot 2025-01-03 at 10 59 24 PM 이쪽 정렬 한번만 확인해주세요!!

@wrryu09
Copy link
Member

wrryu09 commented Jan 3, 2025

Jan-03-2025 22-57-54
누를 때 버튼이 살짝 덜그럭거리는 것 같은데 혹시 없앨 수 있나요??

@wrryu09
Copy link
Member

wrryu09 commented Jan 3, 2025

image 월간간뷰에서 스크롤이 안생기도록 조정할 수도 잇을까요?

Copy link
Member

@wrryu09 wrryu09 left a comment

Choose a reason for hiding this comment

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

와 캘린더 고생많으셧어요 ㅜ.ㅜ👍 커스텀 힘드네요 ....
위 코멘트들 간단하게 확인 부탁드리고 이후 머지해도 될 것 같습니당

@Kjiw0n Kjiw0n force-pushed the feat/#322/calendar-v2 branch 3 times, most recently from d92e1f6 to f9d9cb1 Compare January 5, 2025 13:10
@TEAM-DAWM TEAM-DAWM deleted a comment from github-actions bot Jan 5, 2025
@TEAM-DAWM TEAM-DAWM deleted a comment from github-actions bot Jan 5, 2025
@TEAM-DAWM TEAM-DAWM deleted a comment from github-actions bot Jan 5, 2025
Copy link

github-actions bot commented Jan 5, 2025

@Kjiw0n
Copy link
Contributor Author

Kjiw0n commented Jan 5, 2025

와 캘린더 고생많으셧어요 ㅜ.ㅜ👍 커스텀 힘드네요 .... 위 코멘트들 간단하게 확인 부탁드리고 이후 머지해도 될 것 같습니당

반영 완료했습니다! 꼼꼼하게 봐주셔서 감사합니다~~

Copy link

github-actions bot commented Jan 5, 2025

Copy link
Member

@jeeminyi jeeminyi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!! 미세한 디자인 요소만 달았는데요,, 첫번째 정렬 문제는 왜그런지 모르겠네요 😿
어푸어푸 🌊

Copy link
Member

Choose a reason for hiding this comment

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

제 pc 상에서만 깨지는 거일 가능성이 높은데요..(pr에는 잘 보여서)
요거 정렬 혹시 왜 이럴까유,, 👀
스크린샷 2025-01-06 오후 10 01 44

Copy link
Contributor Author

Choose a reason for hiding this comment

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

스크린샷 2025-01-08 22 30 12

오잉 이 부분은 SubDate 컴포넌트 사용한건데, 혹시 이전 pr 확인하실때에도 이런 문제 있으셨을까요????
스토리북 확인해도 중앙에 맞게 떠서.. 글씨체 먹었는 지 확인 부탁드립니다

.fc-toolbar-chunk .fc-button:active {
background: none;
}

Copy link
Member

Choose a reason for hiding this comment

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

요기 보더값이 미세하게 안맞는 것 같아요!
스크린샷 2025-01-06 오후 10 03 18

Copy link
Member

Choose a reason for hiding this comment

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

으앙 여기 제가한건데 라디어스가 이상하네욤,, 돌아가면 고치겠슴다

Copy link
Member

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋ 바쁜 리드

Copy link
Contributor Author

Choose a reason for hiding this comment

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

요거 FullCalendar에서 제공해주는 버튼을 다른 컴포넌트로 바꿔서 연결하는 게 기존버튼 스타일링보다 코드가 복잡해져서, 기존 버튼 스타일링하는 방식으로 했습니다 -> 제가했다는 소리...

FullCalnedar 자체 스타일링이 radius에 먹혀있어서 지우고 스타일링 조정했습니다~

Copy link

github-actions bot commented Jan 9, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

[FEAT] 캘린더 디자인 수정
3 participants