Merge : 메인 페이지 css 및 중복 컴포넌트화 #33
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- largeMobile과 desktop 사이즈 css 완성
- 톱니바퀴 클릭시 modal창 나오게 구현 - largeMobile 반응형까지 구현 - loadin 페이지 height 100vh로 수정
- desktop 400px, largeMobile 300px, smallMobile 200px을 기준으로 반응형구현
- modal 컴포넌트를 따로 폴더를 만들어서 정리했습니다
- 재사용이 많이 되는 navigationbar를 컴포넌트화 했습니다
Zamoca42
reviewed
Feb 6, 2024
| import styled from 'styled-components'; | ||
| import { media } from '../../style/mediaQuery'; | ||
|
|
||
| const Wapper = styled.div` |
There was a problem hiding this comment.
Wapper는 무슨 뜻인가요?
Wrapper와 역할과 변수명이 비슷한거같습니다.
오타라면 수정이 필요할거같네요
There was a problem hiding this comment.
오타네요 컨벤션에 따라서 고쳐보겠습니다
- 함수명과 변수명은 카멜 케이스, jsx는 파스칼 케이스 - props를 보기 편하게 고쳤습니다.
Zamoca42
approved these changes
Feb 9, 2024
|
넵 styled-component라는 라이브러리 인데 React 애플리케이션에서 스타일을 작성하기 위한 CSS-in-JS 라이브러리라고 하네요 클래스 네임도 자동으로 설정해줘서 똑같은 클래스명으로 충돌할 위험이 없고 ex) class="sc-fsYfdN eujpIp" 제 기준에는 가독성도 편해요 |
sheepdog13
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR 체크리스트
아래 항목을 확인해 주세요:
PR 유형
이 PR은 어떤 종류의 변경을 가져오나요?
관련 이슈
이슈 번호: #10
현재 동작은 무엇인가요?
새로운 동작은 무엇인가요?
이 PR은 호환성 변경을 도입하나요?
기타 정보
main 페이지 css 반응형 구현
중복되는 부분 컴포넌트화 했습니다 밑 커밋을 보시고 궁금하시면 물어봐주세요
Refact : NavigationBar 컴포넌트