-
Notifications
You must be signed in to change notification settings - Fork 1
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
[공통] 헤더, 사이드 패널 컴포넌트화 #272
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.
700줄 보고 기절할 뻔 했지만, 힘내서 리뷰했습니다 ㅎㅎ
수고하셨습니다
LABEL1: 'mega-menu-label-1', | ||
LABEL2: 'mega-menu-label-2', |
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.
아 사용하지 않는 변수군요? 그렇다면 지워도 될 것 같아요!
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.
제 추측입니다만 추후 확장을 고려해서 예시 라벨을 넣어놓은 것 같습니다.
@@ -0,0 +1,264 @@ | |||
.mobileheader { |
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.
.mobileheader { | |
.mobile-header { |
mobile-header 아닌가요?
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.
수정하였습니다!
What is this PR? 🔍
Changes 📝
ScreenShot 📷
Test CheckList ✅
Precaution
✔️ Please check if the PR fulfills these requirements
develop
branch, not themain
branchdevelop
branch?