[SP2] 프로젝트 탭 레이아웃 스타일 반영 #237
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
f0rever0
approved these changes
Oct 25, 2023
| width: 100%; | ||
| margin-top: 36px; | ||
| margin-bottom: 36px; | ||
| @include mobile { |
There was a problem hiding this comment.
question;
scss에서 사용하는 미디어쿼리값이 궁금해 확인해봤는데, breakpoint가 피그마랑 조금 다른것 같이 보여요..!
맞게 동작하는건가요?
There was a problem hiding this comment.
일단 다른 값들이 다 저 값을 보고 있어서, 변경하기 두려운 면이 있습니다 ..
그리고 양옆에 패딩이 넉넉해서 일단 100px 정도 차이는 괜찮을 것 같다는 나이브한 생각도 있어요 ..!!
scss를 다 드러내거나, project page 용도의 breakpoint를 만들어서 쓰는 방법을 택해야 할 것 같습니다
solar3070
reviewed
Oct 28, 2023
f0rever0
added a commit
that referenced
this pull request
Oct 28, 2023
* fix : 삼항연산자 추가 * feat : blog 탭 생성 * style : change header style * feat : 블로그 탭 네비게이션 * style : 전체적인 컬러 코드 변경 * chore: Update issue templates * chore: Update issue templates * refactor: utils 폴더 통합 * refactor: styles 폴더 통합 * fix: 변경된 경로 반영 * style: 메이커스 디자인 시스템 컬러 추가 * feat : blog 탭 생성 * style : change header style * feat : 블로그 탭 네비게이션 * style : 전체적인 컬러 코드 변경 * feat: implement useOutsideClickListener * feat: add Select component for dropdown selections * feat: implement Select component and modify some styles in ProjectPage * refactor: apply color systems * fix: do not close dropdown after selection * fix: fix color * feat : 프로젝트 디테일 페이지 getServerSideProps 사용 * refactor : change to inline export * refactor: apply color systems * style : header 선택한 탭에 밑줄 * fix : change constant color * style: 회색 컬러 코드 수정 * chore: 레거시 컬러 코드 제거 * style: 404 페이지 스타일 수정 * fix: add image domains in next.config.js * fix : blog tab mobile boder style * fix : blog tab description * chore: rename St to S * chore: remove ProjectFilter component * feat : blog 탭 생성 * style : change header style * feat : 블로그 탭 네비게이션 * style : 전체적인 컬러 코드 변경 * feat : blog 탭 생성 * refactor : change to inline export * refactor: apply color systems * style : header 선택한 탭에 밑줄 * fix : change constant color * fix : blog tab mobile boder style * fix : blog tab description * style : change tab style * fix : color 상수 코드 사용 * style : footer color 상수로 바꾸기 * refactor : change folder name * feat : tab description 어절 단위 줄바꿈 * chore: makers-frontend 컬러 시스템 설치 * chore: 기존 컬러 상수 라이브러리로 대체 * chore: 라이브러리 업데이트 * chore : sopt color libaray 적용 * style : header 디자인 변경 * feat: 블로그 글 목록 컴포넌트 구현 * asset: 빈 하트 svg 추가 * feat: 블로그 글 컴포넌트 구현 * feat: 기본 프로필 이미지 컴포넌트 구현 * feat: 블로그 글 리스펀스 타입 선언 * feat : add blog tab title hover style * chore : blogTab 변수명 수정 * feat: 블로그 글 헤더, 좋아요 컴포넌트 분리 * feat: 선택한 탭 props 받기 * feat: 블로그 글 공통 타입 선언 * chore: 컴포넌트명 및 위치 수정 * fix: re-export용 index.ts 파일 제거 * fix: 기본 내보내기 코드 컨벤션에 맞춰 수정 * chore: colors 파일 제거 * feat: implement carousel component * feat: implement carousel component in Project Page * chore: misc styles and utils * fix: fix detail of recent project list style * fix: change type of moreStyle property * feat: implement page indicator dot in carousel * chore: replace lib/colors to sopt-makers/colors * chore: remove console.log * fix: 메이커스 프론트엔드 컬러 라이브러리 적용 * style: 네비게이션 클릭 범위 수정 * fix: alter moreStyle prop with css prop * fix: apply convention of component default export * refactor: make magic number into const * fix: fix responsive related style values * feat: add interaction to try link button * asset: 채용 공고 기업 로고 이미지 추가 * chore: 후원 탭 채용 공고 추가 * feat: implement styling updates in Project page (#237) * fix : 삼항연산자 추가 * feat : 프로젝트 디테일 페이지 getServerSideProps 사용 * chore : rebase develop --------- Co-authored-by: Hyejun Lee <[email protected]> Co-authored-by: solar3070 <> Co-authored-by: SeojinSeojin <[email protected]>
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.
카드 디자인 변경은 #234 이슈에서!
Summary
Screenshot
Comment