-
Notifications
You must be signed in to change notification settings - Fork 7
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
[SP2] blog tab navigation #228
Changes from 29 commits
7145e8a
8bbcfe5
94a89ee
12efe7d
78c2717
80adbca
c10ea97
9c116aa
d9ea44c
88ae4c1
4c2e5f6
babab4f
85aa5ef
5c178a1
8966d63
7dc7385
43776ca
8f2e09a
dc3d174
51d8a5e
190d379
c02989e
2064b1e
51d8f9a
d0557d7
5c31a0d
bb6d10a
6058f69
59cc555
7f2037a
74553a9
3f1bfb4
afe94b6
dc96bc5
5423574
1ed8aaf
173801f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,18 +1,18 @@ | ||
import { useIsDesktop, useIsMobile, useIsTablet } from '@src/hooks/useDevice'; | ||
import DesktopHeader from './Desktop/DesktopHeader'; | ||
import MobileHeader from './Mobile/MobileHeader'; | ||
import styles from './header.module.scss'; | ||
import * as S from './style'; | ||
|
||
export function Header() { | ||
const isDesktop = useIsDesktop('992px'); | ||
const isTablet = useIsTablet('766px', '991.9px'); | ||
const isDesktop = useIsDesktop('940px'); | ||
const isTablet = useIsTablet('768px', '939px'); | ||
const isMobile = useIsMobile(); | ||
|
||
return ( | ||
<header className={styles.wrapper}> | ||
<S.Wrapper> | ||
{isDesktop && <DesktopHeader />} | ||
{isTablet && <MobileHeader />} | ||
{isMobile && <MobileHeader />} | ||
</header> | ||
</S.Wrapper> | ||
); | ||
} |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,20 +12,9 @@ export const menuTapList: MenuTapList = [ | |
href: '/project', | ||
}, | ||
{ | ||
type: MenuTapType.Parent, | ||
type: MenuTapType.Router, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 좋네용.. 사실 이제 MenuTapType.Anchor 도 사라지긴 해서 type 이라는 친구도 없어도 되긴 혀요 다른 것이 추가될 수도 있으니 살려두어도 괜찮을 것 같습니당 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. MenuTapType.Anchor 도 지울까 했는데,,, 일단 살려뒀습니다! |
||
title: '블로그', | ||
children: [ | ||
{ | ||
type: MenuTapType.Router, | ||
title: '활동후기', | ||
href: '/review', | ||
}, | ||
{ | ||
type: MenuTapType.Router, | ||
title: '솝티클', | ||
href: '/sopticle', | ||
}, | ||
], | ||
href: '/blog', | ||
}, | ||
{ | ||
type: MenuTapType.Router, | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
import styled from '@emotion/styled'; | ||
import { colors } from '@src/lib/styles/colors'; | ||
|
||
export const Wrapper = styled.header` | ||
width: 100%; | ||
min-height: 80px; | ||
margin: 0 auto; | ||
display: flex; | ||
justify-content: center; | ||
align-items: center; | ||
position: fixed; | ||
background-color: ${colors.gray900}; | ||
backdrop-filter: blur(20px); | ||
z-index: 100; | ||
padding: 0 20px; | ||
top: 0; | ||
|
||
/* 태블릿 + 데스크탑 뷰 */ | ||
@media (min-width: 768px) { | ||
height: 48px; | ||
} | ||
|
||
/* 모바일 뷰 */ | ||
@media (max-width: 767px) { | ||
height: 48px; | ||
min-height: 48px; | ||
|
||
justify-content: space-between; | ||
} | ||
`; |
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.
요 컬러는 mds에 없어서...! 색상코드를 넣었습니다! pd에게 다시 확인하고 커밋해둘게요!