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

button size 조정 #52

Merged
merged 3 commits into from
Mar 29, 2024
Merged

button size 조정 #52

merged 3 commits into from
Mar 29, 2024

Conversation

yougyung
Copy link
Member

@yougyung yougyung commented Mar 27, 2024

📌 작업 내용

구현 내용 및 작업 했던 내역

  • 버튼 사이즈 리사이징

🤔 고민 했던 부분

  • button size의 기준은 기존 프로젝트의 1500px과 figma가 동일한 것을 확인하고, 해당 기준에 맞춰서 진행했습니다.
  • 버튼 사이즈 내의 반응형은 기존의 논의대로 고려하지 않았습니다.
  • 사이즈는 기존의 lg, md, sm, xs을 기준으로 변경을 진행했으며, 대표적으로 쓰임이 되는 예시를 사진으로 첨부합니다.
    • lg
    스크린샷 2024-03-27 오후 7 20 00
  • md
    스크린샷 2024-03-27 오후 7 22 22
  • sm
    스크린샷 2024-03-27 오후 7 22 53
  • xs
    스크린샷 2024-03-27 오후 7 23 17

@yougyung yougyung added the task label Mar 27, 2024
@yougyung yougyung self-assigned this Mar 27, 2024
@yougyung yougyung linked an issue Mar 27, 2024 that may be closed by this pull request
Copy link

@yougyung yougyung changed the title fix: button size 조정 button size 조정 Mar 27, 2024
Copy link

seonghunYang
seonghunYang previously approved these changes Mar 27, 2024
Copy link
Collaborator

@seonghunYang seonghunYang left a comment

Choose a reason for hiding this comment

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

  • Chromatic을 확인해 보니, Form 컴포넌트의 submit 버튼의 크기가 작아지면서 부자연스러워 보여 크기를 조정하여 커밋했습니다.
  • 약간 sm - md의 중간 사이즈가 없어서 조금 크게 보이는 것 같지만, 크게 문제는 없어보여 괜찮을 것 같습니다.
  • Chromatic 좋네요

gahyuun
gahyuun previously approved these changes Mar 28, 2024
Copy link
Member

@gahyuun gahyuun left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~!

@yougyung yougyung dismissed stale reviews from gahyuun and seonghunYang via 3f1727a March 28, 2024 12:29
Copy link

@yougyung
Copy link
Member Author

가현이가 전달해줘서 기존 md size의 경우text-md로 font-size를 조절했는데, tailwind의 font-size기준으로 존재하지않더라고요 ㅎㅎ..
이 부분을 text-base로 변경했습니다.
text-base가 1rem 16px을 띄고 있어서, 브라우저 기본 글꼴크기와 동일하므로 디자인 변화는 없을 것으로 예상됩니다!

@yougyung yougyung merged commit 34af5a6 into main Mar 29, 2024
2 of 3 checks passed
@yougyung yougyung deleted the button-resize/#48 branch March 29, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

button size 재설정
3 participants