-
Notifications
You must be signed in to change notification settings - Fork 0
[feat] 비교 분석 페이지 UI 구현 #28
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
base: main
Are you sure you want to change the base?
Conversation
dohy-eon
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.
늦은 시간까지 수고 많으셨습니다! 🙌
먼저 px-5 py-8 bg-white border-b-[8px] border-neutral-5 이런 부분이 자주 보이는데 컴포넌트로 빼서 작업해도 좋을 것 같습니당
추가로 스켈레톤 UI가 들어가면 좋을 것 같은데 어떻게 생각하실까요?? 하단에 참고 링크 첨부해드릴게용
https://tech.kakaopay.com/post/skeleton-ui-idea/
[작업 내용]레이아웃 공통 컴포넌트화로 중복 코드 제거
Layout Shift 방지를 위한 스켈레톤 UI 도입
날짜 객체 참조 고정(useMemo)으로 무한 로딩 이슈 해결
skeleton-test.mp4 |
dohy-eon
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.
@seunghee0321
Merge branch 'feat/sector-analysis' into feat/compare-analysis
4aa2055 이부분 커밋 메시지 수정 부탁드리겠습니다. chore: 분야별 분석 및 비교 분석 기능 통합 이런 식으로 작성하면 좋을 것 같아요!
e60b2cf to
3fde399
Compare
dohy-eon
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.
lgtm 🙌
14ef0d3 to
7f8b598
Compare
|
커밋 메시지 수정 후 누락된 코드 추가 커밋했습니다. |
Pull Request
📝 변경 내용
🔗 관련 이슈
🎯 변경 사항
📱 테스트
📸 스크린샷 (UI 변경시)
test.mp4
📋 체크리스트
npm run format)npm run lint:fix)💻 코드 품질 확인
PR 제출 전에 다음 명령어를 실행하여 코드 품질을 확인해주세요: