Skip to content
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

레이아웃에 헤더 높이 만큼 padding-top 적용, Input style rem값 변경 #234

Merged

Conversation

Jaymyong66
Copy link
Contributor

⚡️ 관련 이슈

close #233

📍주요 변경 사항

  1. 레이아웃에 헤더 높이 만큼 padding-top을 일괄적으로 적용한다.
  • 현재는 각 페이지마다 적용해야함.
  • 적용하는 값도 헤더의 값이 아닌 10rem으로 주고 있었음
  1. Input의 style 파일의 rem 값 변경
  • 루트 폰트의 0.625 제거로 인한 변경
  • 파일 이름을 sytle.ts -> Input.style.ts로 변경

@Jaymyong66 Jaymyong66 added FE 프론트엔드 refactor 요구사항이 바뀌지 않은 변경사항 labels Aug 1, 2024
@Jaymyong66 Jaymyong66 added this to the 3차 스프린트 🐤 milestone Aug 1, 2024
@Jaymyong66 Jaymyong66 self-assigned this Aug 1, 2024
margin-left: 8px;
height: 1rem;
margin-left: 0.5rem;
font-size: 1rem;
color: red;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분 red 아니고 디자인 토큰에 있는 빨간색 중 하나로 써주셔도 좋을 것 같아요!

@Hain-tain Hain-tain merged commit 49ecb89 into woowacourse-teams:dev/fe Aug 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 refactor 요구사항이 바뀌지 않은 변경사항
Projects
Status: Weekend Done
Development

Successfully merging this pull request may close these issues.

3 participants