[Fix] material 의존성 제거 #423
Conversation
Summary of ChangesHello @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은 프로젝트의 의존성을 간소화하기 위해 레거시 Material Design 라이브러리를 제거하는 것을 목표로 합니다. 프로젝트가 Material 3를 독점적으로 사용함에 따라, 불필요한 Material 의존성을 제거하여 코드베이스를 정리하고 최신 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
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| android:fillColor="#00000000" | ||
| android:pathData="M18,6L6,18M6,6L18,18" | ||
| android:strokeWidth="2" | ||
| android:strokeColor="#9D9D9D" |
There was a problem hiding this comment.
새로 추가된 아이콘의 strokeColor에 특정 색상(#9D9D9D)이 하드코딩되어 있습니다. 이 아이콘은 Icon 컴포저블의 tint 파라미터를 통해 색상이 지정되므로, 벡터 드로어블 자체에는 특정 테마 색상을 하드코딩하지 않는 것이 좋습니다. 이렇게 하면 아이콘의 재사용성이 높아지고, 테마 색상이 변경될 때 발생할 수 있는 불일치를 방지할 수 있습니다. 일반적으로 틴팅을 적용할 아이콘은 검은색 (#FF000000)으로 정의하는 것이 표준적인 방법입니다. 이 방식을 다른 아이콘(ic_arrow_left.xml 등)에도 일관되게 적용하는 것을 고려해 보세요.
| android:strokeColor="#9D9D9D" | |
| android:strokeColor="#FF000000" |
kangyuri1114
left a comment
kangyuri1114
left a comment
There was a problem hiding this comment.
엄청 빠르게 해주셨네요! 감사합니다
슬쩍 봤을 때는 TopBar IconButton의 아이콘도 material 사용중인줄 알았는데 material3 사용 중이었네요
x 아이콘만 material 사용 중인거라 제거하는게 확실히 좋은 방향이네요 👍
* feat: 제휴지도 학과/전체 토글 디폴트값 변경 및 토글 UI 위치 변경 * feat: PartnershipFilterToggle에서 toggleItem 생성방식 변경 및 UI수정, 불필요 코드 삭제 * delete: 사용하지 않는 위치 권한 코드 삭제 * feat: compose 라이브러리 버전 업데이트 * feat: compose router-screen 분리 * feat: 학과 선택을 하지 않은 경우, default 토글이 "전체"로 수정. 학과 토글 선택 시 바텀시트 보여주기 * feat: 제휴지도 학과/전체 토글 디폴트값 변경 및 토글 UI 위치 변경 * feat: PartnershipFilterToggle에서 toggleItem 생성방식 변경 및 UI수정, 불필요 코드 삭제 * delete: 사용하지 않는 위치 권한 코드 삭제 * feat: compose 라이브러리 버전 업데이트 * feat: compose router-screen 분리 * feat: 학과 선택을 하지 않은 경우, default 토글이 "전체"로 수정. 학과 토글 선택 시 바텀시트 보여주기 * feat: compose 버전 업데이트 롤백 * feat: 학과 정보가 업데이트될 때마다 토글 상태 없데이트하도록 key 변경 * feat: init 내부에서 전체 제휴정보 load하는 코드 제거(compose LaunchedEffect로 이동) * feat: BottomSheet 표시는 View의 SheetState로만 관리 (ViewModel은 데이터만 제공), 제휴정보, 학과 입력 바텀시트 상태 변수 분리 * feat: Domain 모델(RestaurantType)을 UI 모델(PlaceType)로 변환 로직을 뷰모델로 이동 * feat: 토글 필터 상태, 이벤트 로거를 ViewModel로 이동. departmentId, collegeId flow 방출로 수정 * [Hotfix] Release에서 발생하던 문제 해결 및 3.1.8 릴리즈 (#418) * fix: DTO 파일 이동에 따른 Proguard Rule 수정 * fix: Generics 관련 정보가 유실되는 TypeToken 대신 Java Class 사용 * release: 3.1.8(46) * refactor: unused import 삭제 * fix: 제미니 대응 * chore: material 의존성 제거 (#423) * feat: 코드 리뷰 반영 (네이밍 변경 및 scope 전달 -> 람다 전달로 수정, 최초 정보 load 시 state copy가 아닌 mapState 객체 생성 및 초기화) * feat: 제휴정보 토글 변경 시 선택했던 식당의 제휴정보 state 초기화 * feat: MapScreen 접근 제어자 private -> internal 변경 --------- Co-authored-by: 정제훈 <[email protected]> Co-authored-by: 유진 <[email protected]>
Summary
material3만을 사용하기에 material 의존성을 제거합니다.
Issue