-
Notifications
You must be signed in to change notification settings - Fork 0
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
[#7] 공통 컴포넌트(Header, Footer, ScrollFloatBtn) 구현, [#9] 랜딩페이지 구현 #12
Conversation
…R 기법을 위한 Global CSS 추가
+ consistent-type-definitions : 타입스크립트 type 키워드로 타입을 선언해야할 상황이 있기 때문 + strict-boolean-expressions : 명확한 nullish 가 아니 falsy 한 값들에 대한 삼항 연산자를 사용하기 위해
오키 ~ 이해는 다 했는데 일단 저 contents 관련 부분은, 적용해봐야 알 것 같고 이거 merge 되면 Typography는 해당 사항대로 수정해놓겠습니당 |
|
||
// Web Accessibility Styles - IR | ||
.visually-hidden { | ||
position: absolute !important; | ||
width: 1px; | ||
height: 1px; | ||
padding: 0; | ||
margin: 0; | ||
border: 0; | ||
overflow: hidden; | ||
clip: rect(1px 1px 1px 1px); /* IE6, IE7 - a 0 height clip, off to the bottom right of the visible 1px box */ | ||
clip: rect(1px, 1px, 1px, 1px); /*maybe deprecated but we need to support legacy browsers */ | ||
clip-path: inset(50%); /*modern browsers, clip-path works inwards from each corner*/ | ||
white-space: nowrap; /* added line to stop words getting smushed together (as they go onto seperate lines and some screen readers do not understand line feeds as a space */ | ||
} |
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.
IR 기법에 대해서는 생각해본 적이 없었는데, 새롭네요,, 배워갑니다 💨 아래는 내가 참고한 자료들,, 나중에 필요하면 다시 살펴보려구용
https://snupi.tistory.com/109
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.
웹 접근성에 대해서는 나도 아직 공부해야할 게 더 많아용 ㅋㅋ,,
이런 존재부터 아는 게 중요하다고 생각해요 ~ 👍
@@ -33,6 +33,6 @@ export default RootLayout; | |||
|
|||
const ChildrenContainer = styled.main` | |||
position: relative; | |||
min-height: calc(100vh - 160px); | |||
font-size: 2rem; | |||
min-height: calc(100vh - 258px); |
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.
음, 어디까지나 랜딩페이지
만 구현되었다는 전제하지만
현재 디자인 시안을 기준으로 랜딩페이지의 Header, Footer 를 제외한 Contents 영역의 컨텐츠가 많지가 않아서, 컨텐츠를 기준으로만 height 를 설정하면
Footer 의 경우, 현재 디자인 시안처럼 원페이지 형태로 딱 이쁘게 안떨어져서 나오더라구요 ~
Footer 가 원페이지에서 하단에 딱 걸치는 형태가 아니라, Contents 가 끝나는 시점 바로 밑에 배치된다는 !
이해가 가시나요 ~ ?
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.
음, 어디까지나
랜딩페이지
만 구현되었다는 전제하지만현재 디자인 시안을 기준으로 랜딩페이지의 Header, Footer 를 제외한 Contents 영역의 컨텐츠가 많지가 않아서, 컨텐츠를 기준으로만 height 를 설정하면
Footer 의 경우, 현재 디자인 시안처럼 원페이지 형태로 딱 이쁘게 안떨어져서 나오더라구요 ~ Footer 가 원페이지에서 하단에 딱 걸치는 형태가 아니라, Contents 가 끝나는 시점 바로 밑에 배치된다는 !
이해가 가시나요 ~ ?
아하 아이갓잇..! 못난이 UI를 방지하기 위한 거였군뇨.. 신기하네욥~
<div | ||
style={{ | ||
display: 'flex', | ||
alignItems: 'center', | ||
justifyContent: 'center', | ||
width: '240px', | ||
height: '50px', | ||
borderRadius: '8px', | ||
backgroundColor: '#0064FF', | ||
color: '#FFFFFF', | ||
fontSize: '16px', | ||
fontWeight: 'medium', | ||
}} | ||
> |
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.
아 알았어 준다고요 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 진짜 정성이네..
📎 Related Issues
📋 작업 내용
🔎 PR Point
styled-components 환경설정을 위한 babelrc.json 을 제거 👉 next.config.js 에서 experimental 에서 설정 가능 확인 후 마이그레이션
package.json
실행 스크립트 중dev
커맨드가 현재next dev
인데next dev --turbo
로 변경해서 실행해보면 알 수 있음Typography 공통 컴포넌트 구현
Typography
공통 컴포넌트를 구현함랜딩페이지 - 컨텐츠 영역 버튼 부분
컨텐츠 영역 배경색에 대한 예측
흰색
이다.옅은 회색
이다.layout.tsx
에서 다음과 같이Header
,Footer
를 제외한Contents
영역에 대해컨테이너 컴포넌트
가 존재하는데, 이 컴포넌트에 background-color 를 현재는흰색
으로 고정해서 사용 중Contents
영역에 대해 Root 단계에서 컨테이너를 감싸지 않고, 각 페이지 별로 필요에 따라 컨테이너 컴포넌트를 구성할 것인지는 문제가 발생할 경우 조취해볼 것😡 Trouble Shooting
👀 스크린샷 / GIF / 링크
랜딩페이지 1차 결과물
➕ 추가적인 태스크
입정 신청하기
실제Button
컴포넌트 적용Header
에서 네비게이션 아이템들(로그아웃
,마이페이지
) UI 추가📚 Reference
웹 접근성 - IR
NextJS 13 - Font
NextJS 13 - SWC (related. styled-compoments 설정)
→ babelrc 설정에서 next.config.js 에서 설정하는 방법
Next13 Turbopack(alpha, 200.12.22 기준)
✅ PR Submit 이전에 확인하세요 !
체크리스트