-
Notifications
You must be signed in to change notification settings - Fork 0
[Chore] caption1 semibold로 변경 #333
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary of Changes
Hello @HI-JIN2, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!
이 PR은 디자인 시스템의 변경사항을 반영하여 caption1 텍스트 스타일의 폰트 두께를 업데이트합니다. 이는 사용자 인터페이스의 일관성을 유지하고 최신 디자인 가이드라인을 따르기 위함입니다.
Highlights
- 디자인 시스템 업데이트:
caption1스타일의 폰트가bold에서semibold로 변경되었습니다. - 적용 파일:
themes.xml(주간/야간 모드) 및Type.kt파일에 해당 변경사항이 반영되었습니다.
Using Gemini Code Assist
The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.
Invoking Gemini
You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
Customization
To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.
Limitations & Feedback
Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.
You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.
Footnotes
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
PR의 설명대로 caption1 타이포그래피 스타일의 폰트를 bold에서 semibold로 일관성 있게 변경한 것을 확인했습니다. XML 테마와 Jetpack Compose 테마 양쪽 모두에 잘 적용되었습니다. 변경 사항 자체는 정확하지만, Compose 테마의 폰트 관리 방식에 대한 개선 제안을 댓글로 남겼습니다. 해당 제안을 별도의 PR에서 논의해보는 것을 권장합니다.
| ), | ||
| caption1 = EatssuTextStyle( | ||
| fontFamily = pretendardBold, | ||
| fontFamily = pretendardSemiBold, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 변경 자체는 올바르지만, 현재 디자인 시스템에서 폰트 두께를 처리하는 방식은 개선될 수 있습니다. 각 두께에 대해 별도의 FontFamily 변수를 선언하는 대신, 모든 관련 폰트 파일을 포함하는 단일 FontFamily를 정의하는 것이 좋습니다. 그런 다음 EatssuTextStyle을 호출할 때 fontWeight 매개변수를 사용하여 원하는 두께를 지정할 수 있습니다.
예를 들어, 다음과 같이 pretendard FontFamily를 정의할 수 있습니다:
val pretendard = FontFamily(
Font(R.font.pretendard_regular, FontWeight.Normal),
Font(R.font.pretendard_medium, FontWeight.Medium),
Font(R.font.pretendard_semibold, FontWeight.SemiBold),
Font(R.font.pretendard_bold, FontWeight.Bold)
)그리고 caption1 스타일은 다음과 같이 정의할 수 있습니다:
caption1 = EatssuTextStyle(
fontFamily = pretendard,
fontWeight = FontWeight.SemiBold,
fontSize = 12.sp,
lineHeight = 18.sp
)이러한 리팩토링은 코드의 가독성과 유지보수성을 향상시키고, Jetpack Compose의 타이포그래피 시스템을 더 효과적으로 활용하는 방법입니다. 이 작업은 별도의 PR에서 진행하는 것을 고려해볼 수 있습니다.
PeraSite
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
푸릅푸릅
Summary
디자인 시스템 caption1이 bold에서 semibold로 바뀜에 따라 이를 적용합니다.
Describe your changes
None
Issue
To reviewers