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

[view] px단위 rem으로 수정 #751

Merged
merged 5 commits into from
Oct 6, 2024
Merged

Conversation

joonwonBaek
Copy link
Contributor

Related issue

#732

Result

  • px 단위를 rem으로 수정하는 작업을 했습니다.
  • px을 계산하는 부분은 pxToRem 함수를 구현해서 계산된 px을 rem으로 바꿔주었습니다.
    • 16px은 1rem이므로 계산된 px에 16을 나누어서 계산하도록 구현했습니다.

Work list

  • scss 파일 px 단위 rem으로 수정
  • ts 파일 계산된 px 단위 rem으로 수정
    • px을 rem으로 수정하기 위해 pxToRem 함수를 만들어서 수정했습니다.

Discussion

  • 추가적으로 수정하면 좋을 부분이나 더 좋은 방식이 있으시면 편히 말씀해주세요!!!

@joonwonBaek joonwonBaek self-assigned this Oct 4, 2024
@joonwonBaek joonwonBaek requested review from a team as code owners October 4, 2024 09:50
pcwadarong
pcwadarong previously approved these changes Oct 4, 2024
}

.top-container {
height: 220px;
height: 13.75rem;
}

.middle-container {
display: flex;
height: calc(100vh - 200px);
Copy link
Contributor

Choose a reason for hiding this comment

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

엇 여기는 혹시 발견 못하신건가요?!?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 발견못한 것 같아요..!! 이 부분 추가로 수정하겠습니다 ㅠㅠ

Copy link
Contributor

Choose a reason for hiding this comment

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

👍👍👍

ytaek
ytaek previously approved these changes Oct 6, 2024
Copy link
Contributor

@ytaek ytaek left a comment

Choose a reason for hiding this comment

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

👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍
👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍
👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍

@joonwonBaek joonwonBaek dismissed stale reviews from ytaek and pcwadarong via 93c5aa0 October 6, 2024 11:41
@joonwonBaek joonwonBaek merged commit ef89f96 into githru:main Oct 6, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants