-
Notifications
You must be signed in to change notification settings - Fork 9
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] main 페이지 디자인 구현 #282
Conversation
# Conflicts: # src/pages/index.tsx
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Bundle SizesCompared against d4bc7f9
|
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.
반응형 작업 감사합니다👍
스토리북으로 확인했을 때는 잘 모르겠는데 어떤 이슈예요?? |
@@ -65,4 +66,8 @@ const tabCss = (theme: Theme, isActive: boolean) => css` | |||
color: ${isActive ? theme.colors.yellow500 : theme.colors.white}; | |||
cursor: pointer; | |||
padding: 16px 24px; | |||
|
|||
${mediaQuery('mobile')} { | |||
font-size: 14px; |
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.
rem 이나 윤호님이 만들어주신 typo 사용하면 좋을 것 같아요 !
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.
이부분은 다른 부분에서도 이런식으로 사용해서, 다른 브랜치에서 한번에 바꾸겠습니다!
다시 시도해보니 안뜨네요,./? 또 같은 현상을 발견하면 그때 다시 말씀드리겠습니다! |
# Conflicts: # src/pages/index.tsx
* style: gnb pc design 수정 * feat: mobile menu icon 추가 * refactor: pc footer theme 적용 * style: footer 모바일 반응형 추가 --------- Co-authored-by: byun sumi <sumi@adenasoft.com>
# Conflicts: # src/components/SectionTitle/SectionTitle.tsx
📝 Changed routes:
Commit 1bbf8a5 (https://depromeet-7ooa1ef2q-depromeet-official-site.vercel.app). |
작업 내용
main 디자인 PC, Mobile 남은 부분 작업
main에서만 사용하는 컴포넌트는
components/Main
으로 옮겼습니다. (스토리북 추가 예정)FAQ 반응형 작업
스크린샷