-
Notifications
You must be signed in to change notification settings - Fork 1
✨ feat: 알림 모달창 구현 #23
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
Conversation
Moon-ju-young
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저희 jsDoc 사용하기로 합의했나요?
minimo-9
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 잘하셨네요 굿입니다!
| } else if (months <= 11) { | ||
| return `${months}달 전`; | ||
| } else { | ||
| return `${years}년 전`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
~~년 까지 표기로 만드시다니! 좋습니다!!
음 따로 정하지는 않았는데, 이게 뭐 라이브러리나 패키지가 아니라 그냥 주석 형식의 일종이다보니, 멘토님 말씀대로 주석을 좀 더 자세히 작성하기 위해 이런 형식으로 사용했습니다~! |
Moon-ju-young
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다 jsDoc 같은 경우 일반 주석과는 달리 쓰냐마냐에 따라 파일 스타일이 많이 달라지는 거 같아서 멘토님이 말하시는 한 사람이 쓴 거 같은 코드를 지향한다면 팀적으로 쓸지 말지 정하는 게 좋을 것 같습니다~
📌 변경 사항 개요
📝 상세 내용
알림 모달창 구현 -
NotificationModal알림 모달창 내 카드 구현 -
NotificationCardformatWorkTime함수 활용)calculateTimeDifference함수 활용)모달창 내에서 스크롤바 숨기기
tailwind-scrollbar-hide패키지 추가추후 구현 예정 기능
🔗 관련 이슈
🖼️ 스크린샷(선택사항)
PC, Tablet용 모달창

Mobile용 모달창

💡 참고 사항
부족한 부분이 있으면 말씀해주시면 감사하겠습니다 (_ _)