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

boxSizing:border-box 일괄 적용 작업 #389

Merged
merged 5 commits into from
Sep 12, 2023
Merged

boxSizing:border-box 일괄 적용 작업 #389

merged 5 commits into from
Sep 12, 2023

Conversation

na-reum
Copy link
Contributor

@na-reum na-reum commented Sep 10, 2023

🚩 관련 이슈

📋 작업 내용

  • boxSizing:border-box를 글로벌하게 적용하고 크기 보정작업을 했어요.

📌 PR Point

  • 원래 Box 컴포넌트에만 있던 속성을 일괄로 적용했어요
  • 처음에는 box 컴포넌트를 활용해서 부분적으로만 적용하자는 생각이었어요.
  • 하지만 저희가 stitches가 아닌 다른 라이브러리를 사용하게 되었을때 이부분이 많이 헷갈려할것 같았어요.
  • 일단 제가 눈에 보이는 부분은 전부 교체를 하였어요.
  • 단, tablet 환경에서부터 모바일 버전으로 보여주자는 이야기를 했어서, 약간 다를 수 있어요
  • 추가적으로 Select 컴포넌트들의 tablet 환경에서 bottomSheet가 안나오는 에러를 해결했어요. ( 아무도 안썻나봐요.)
  • 그리고 bottomSheet에 overlay를 적용해서 세세한 오류를 잡았어요.

📸 스

스크린샷 2023-09-11 오전 12 27 28
크린샷

@na-reum na-reum temporarily deployed to development September 10, 2023 15:31 — with GitHub Actions Inactive
@cloudflare-workers-and-pages
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 32a78e4
Status: ✅  Deploy successful!
Preview URL: https://852b7f07.sopt-crew-dev-legacy.pages.dev
Branch Preview URL: https://feat--388.sopt-crew-dev-legacy.pages.dev

View logs

@na-reum na-reum self-assigned this Sep 10, 2023
Copy link
Contributor

@borimong borimong left a comment

Choose a reason for hiding this comment

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

수정 감사합니닷!!!
처음에는 box 컴포넌트를 활용해서 부분적으로만 적용하자는 생각이었어요.
혹시 처음부터 border-box 를 글로벌하게 적용하지 않았던 이유가 무엇이었을지 알 수 있을까요..?? ٩( °ꇴ °)۶

@na-reum
Copy link
Contributor Author

na-reum commented Sep 11, 2023

수정 감사합니닷!!! 처음에는 box 컴포넌트를 활용해서 부분적으로만 적용하자는 생각이었어요. 혹시 처음부터 border-box 를 글로벌하게 적용하지 않았던 이유가 무엇이었을지 알 수 있을까요..?? ٩( °ꇴ °)۶

불필요한 속성 정의의 최소화를 하고 싶었어요 !

Copy link
Member

@eunsukimme eunsukimme left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 👍

Copy link
Member

@100Gyeon 100Gyeon left a comment

Choose a reason for hiding this comment

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

🚀🚀🚀🚀🚀

@na-reum na-reum merged commit 08fddf6 into develop Sep 12, 2023
@na-reum na-reum deleted the feat/#388 branch September 17, 2023 05:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

boxSizing:border-box 일괄 적용
4 participants