-
Notifications
You must be signed in to change notification settings - Fork 1
[#362] 사용하지 않는 파일 제거 및 디자인 시스템 적용 #363
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
- 모듈화된 디자인 시스템의 접근 제어 문제를 해결 - 러 뷰에 남아있던 레거시 폰트 및 색상 사용을 최신 디자인 시스템 표준에 맞게 수정
Summary of ChangesHello @Hrepay, 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은 앱의 UI 디자인 시스템을 표준화하고 기존 UI 코드를 리팩토링하는 데 중점을 둡니다. 주요 변경 사항으로는 커스텀 폰트 및 색상 정의를 중앙 집중식 디자인 시스템으로 대체하고, 불필요한 UI 요소를 제거하며, 여러 유틸리티 확장 기능을 Highlights
Using Gemini Code AssistThe 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 by creating a comment using either
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 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
|
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은 디자인 시스템(EATSSUDesign)을 도입하여 UI 전반에 걸쳐 일관성을 확보하고, 사용하지 않는 코드를 정리하는 대규모 리팩토링을 성공적으로 수행했습니다.
주요 변경 사항은 다음과 같습니다:
- 디자인 시스템 적용: 하드코딩된 폰트와 색상을 디자인 시스템 토큰으로 교체하여 코드의 가독성과 유지보수성을 크게 향상시켰습니다.
- 모듈화: 기존
Utility/Extension에 있던 코드들을EATSSUDesign모듈로 이전하여 프로젝트 구조를 개선했습니다. - 코드 정리:
ReviewViewController에서 불필요한 UI 요소를 제거하는 등 코드를 깔끔하게 정리했습니다.
전반적으로 매우 훌륭한 리팩토링 작업입니다. 몇 가지 사소한 개선점을 제안드렸으니 확인 부탁드립니다. 고생 많으셨습니다!
Funital
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.
표준화된 폰트 스타일을 사용하니까 코드가 더 깔끔해지네요
#️⃣ 관련 이슈
Resolved #362
💡작업 내용
디자인 시스템 표준화 및 UI 리팩토링
폰트 및 색상 표준화
.bold(size:)와 같은 커스텀 폰트 초기화 코드를.header1,.subtitle1,.caption2,.button2등의 표준화된 폰트 스타일로 전면 교체했습니다.EATSSUDesignAsset.Color의 중앙 집중식 색상 토큰(.gray700,.primary등)으로 변경하여 텍스트와 배경 색상을 통일했습니다.코드 정리 (Clean-Up)
ReviewViewController에서 더 이상 사용하지 않는noReviewImageView속성과 관련 레이아웃 코드를 삭제했습니다.UI 일관성 및 유지보수성 향상
EATSSUDesignAsset과UIFont확장을 활용함으로써 코드의 유지보수성과 향후 확장성을 개선했습니다.ReviewViewController.swift에EATSSUDesign모듈을 import 했습니다.💬리뷰 요구사항(선택)