-
Notifications
You must be signed in to change notification settings - Fork 2
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
Gmmq 690 style: 메인 페이지 UI 리터칭 #242
Conversation
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.
좋아요 고생하셨습니다 !!
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.
수고하셨습니다!
{positions.slice(MAX_LABELS_TO_DISPLAY).map((position) => ( | ||
<Label |
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.
positions를 직접 자르는것 보다 splice를 사용하는 것도 좋아보여요!
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.
splice를 사용하면 원본 배열을 건드는 것으로 알고 있어서 slice를 사용했습니다.
혹시나 같은 컴포넌트에서 positions를 사용할 일이 있을 수도 있으니 허허... :)
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.
반대로 알고 있었네요 ㅜㅜ
…-690-style/Main-page-ui-redesign
💻 스크린샷 (선택사항)
[메인 페이지 리터치]
📃 PR 설명
캐치프레이즈
이력서를 작성하는 당신에게는, 최대한의 많은 피드백이 필요합니다.
에서커리어를 시작하는 당신의 이력서에는 최대한 많은 피드백이 필요합니다.
로 수정했습니다.안내사항
안내사항 탭이 추가되었습니다. 데이터는 목업 데이터가 들어가 있어요.
UI적인 요소로만 존재합니다. 추후 기능을 추가해봐도 괜찮을 것 같아요.
상세 페이지 없이 한 줄로만 안내사항을 알려주는 한 줄 알림 컨셉입니다.
이벤트 그리드 아이템
디자인이 전체적으로 수정되었습니다.
차크라UI의 Card 컴포넌트를 사용했고, Tooltip으로 대부분의 데이터 항목 전체 길이를 확인할 수 있습니다.
썸네일 이미지
포토샵으로 썸네일 이미지 6종 (프론트, 백, 풀스택, 데브옵스, 모바일, 머신러닝/인공지능)을 assets에 추가했습니다.
📃 참고사항
🔎 PR포인트 및 궁금한 점