Conversation
There was a problem hiding this comment.
Code Review
This pull request updates multiple layout files to standardize text styling using the textAppearance attribute and updated color resources. The changes affect the calendar selection, onboarding, permission, and calendar account item screens. Feedback focuses on critical build errors caused by referencing non-existent styles (e.g., BodyMd.Regular and ModalSm.Regular), as well as several instances of redundant or conflicting attributes where textStyle or textSize are used alongside textAppearance. There are also concerns regarding potential font size regressions in the permission and account item layouts.
| android:layout_marginTop="50dp" | ||
| android:gravity="center" | ||
| tools:text="이제는 일정과 경로를 하나로\n출발까지의 흐름을 Pace가 이어드려요." | ||
| android:textAppearance="@style/TextAppearance.App.BodyMd.Regular" |
There was a problem hiding this comment.
TextAppearance.App.BodyMd.Regular 스타일이 styles.xml에 정의되어 있지 않아 빌드 오류가 발생합니다. styles.xml에 BodyMd.Regular 스타일을 추가하거나, TextAppearance.App.BodyMd.Medium과 같은 기존 스타일을 사용하도록 수정해주세요.
| android:textAppearance="@style/TextAppearance.App.BodyMd.Regular" | |
| android:textAppearance="@style/TextAppearance.App.BodyMd.Medium" |
| android:textAppearance="@style/TextAppearance.App.ModalSm.Regular" | ||
| android:textColor="@color/text_secondary" |
| android:gravity="center" | ||
| tools:text="일정부터 출발까지 \n하루의 흐름을, 나의 페이스로" | ||
| android:textColor="@color/secondary_900" | ||
| android:textAppearance="@style/TextAppearance.App.BodyMd.Regular" |
There was a problem hiding this comment.
TextAppearance.App.BodyMd.Regular 스타일이 styles.xml에 정의되어 있지 않아 빌드 오류가 발생합니다. styles.xml에 BodyMd.Regular 스타일을 추가하거나, TextAppearance.App.BodyMd.Medium과 같은 기존 스타일을 사용하도록 수정해주세요.
| android:textAppearance="@style/TextAppearance.App.BodyMd.Regular" | |
| android:textAppearance="@style/TextAppearance.App.BodyMd.Medium" |
| android:layout_marginTop="50dp" | ||
| android:gravity="center" | ||
| tools:text="하루의 일정과 이동 사이에서 \n당신의 페이스는 어땠나요?" | ||
| android:textAppearance="@style/TextAppearance.App.BodyMd.Regular" |
There was a problem hiding this comment.
TextAppearance.App.BodyMd.Regular 스타일이 styles.xml에 정의되어 있지 않아 빌드 오류가 발생합니다. styles.xml에 BodyMd.Regular 스타일을 추가하거나, TextAppearance.App.BodyMd.Medium과 같은 기존 스타일을 사용하도록 수정해주세요.
| android:textAppearance="@style/TextAppearance.App.BodyMd.Regular" | |
| android:textAppearance="@style/TextAppearance.App.BodyMd.Medium" |
| android:layout_marginTop="50dp" | ||
| android:gravity="center" | ||
| tools:text="일정은 캘린더로, 경로는 지도로\n출발 타이밍까지 신경 쓰다 보면\n하루의 흐름과 내 페이스를 놓치곤 해요." | ||
| android:textAppearance="@style/TextAppearance.App.BodyMd.Regular" |
There was a problem hiding this comment.
TextAppearance.App.BodyMd.Regular 스타일이 styles.xml에 정의되어 있지 않아 빌드 오류가 발생합니다. styles.xml에 BodyMd.Regular 스타일을 추가하거나, TextAppearance.App.BodyMd.Medium과 같은 기존 스타일을 사용하도록 수정해주세요.
| android:textAppearance="@style/TextAppearance.App.BodyMd.Regular" | |
| android:textAppearance="@style/TextAppearance.App.BodyMd.Medium" |
| android:textAppearance="@style/TextAppearance.App.BodySm.Semibold" | ||
| android:textColor="@color/text_primary" | ||
| android:textStyle="bold" |
There was a problem hiding this comment.
TextAppearance.App.BodySm.Semibold 스타일은 이미 pretendard_semibold 폰트를 사용하여 굵은 글씨를 표현하므로, android:textStyle="bold" 속성은 불필요합니다. 스타일 가이드의 일관성을 위해 제거하는 것이 좋습니다.
| android:textAppearance="@style/TextAppearance.App.BodySm.Semibold" | |
| android:textColor="@color/text_primary" | |
| android:textStyle="bold" | |
| android:textAppearance="@style/TextAppearance.App.BodySm.Semibold" | |
| android:textColor="@color/text_primary" |
| android:textAppearance="@style/TextAppearance.App.BodySm.Medium" | ||
| android:textColor="@color/text_primary" | ||
| android:textStyle="bold" /> |
There was a problem hiding this comment.
textAppearance를 통해 폰트 스타일을 지정하면서 textStyle을 별도로 설정하면 일관성이 떨어지고 혼란을 줄 수 있습니다. TextAppearance.App.BodySm.Medium은 pretendard_medium 폰트를 사용하는데, textStyle="bold"와 충돌합니다.
굵은 글씨가 의도된 것이라면, pretendard_semibold 폰트를 사용하는 TextAppearance.App.BodySm.Semibold 스타일을 사용하고 textStyle 속성을 제거하는 것을 권장합니다.
| android:textAppearance="@style/TextAppearance.App.BodySm.Medium" | |
| android:textColor="@color/text_primary" | |
| android:textStyle="bold" /> | |
| android:textAppearance="@style/TextAppearance.App.BodySm.Semibold" | |
| android:textColor="@color/text_primary" /> |
| android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular" | ||
| android:textColor="@color/text_secondary" /> |
There was a problem hiding this comment.
이 설명 텍스트 또한 12sp에서 10sp로 크기가 줄었습니다. 12sp 크기를 유지하고 싶다면, TextAppearance.App.CaptionLg.Regular 스타일을 사용하는 것을 고려해 보세요.
| android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular" | |
| android:textColor="@color/text_secondary" /> | |
| android:textAppearance="@style/TextAppearance.App.CaptionLg.Regular" | |
| android:textColor="@color/text_secondary" /> |
| android:textAppearance="@style/TextAppearance.App.BodySm.Semibold" | ||
| android:textColor="@color/white" | ||
| android:textSize="16sp" | ||
| android:textStyle="bold" |
There was a problem hiding this comment.
버튼에 textAppearance를 사용하면서 textSize와 textStyle을 별도로 지정하면 일관성이 떨어집니다.
TextAppearance.App.BodySm.Semibold는 14sp인데,textSize는 16sp로 오버라이드하고 있습니다.BodySm.Semibold는 이미pretendard_semibold폰트를 사용하므로textStyle="bold"는 불필요합니다.
16sp 굵은 글씨가 의도된 것이라면, TextAppearance.App.BodyMd.SemiBold 스타일을 사용하고 textSize와 textStyle 속성을 제거하는 것을 권장합니다.
| android:textAppearance="@style/TextAppearance.App.BodySm.Semibold" | |
| android:textColor="@color/white" | |
| android:textSize="16sp" | |
| android:textStyle="bold" | |
| android:textAppearance="@style/TextAppearance.App.BodyMd.SemiBold" | |
| android:textColor="@color/white" |
| android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular" | ||
| android:textColor="@color/gray_500" | ||
| android:textSize="12sp" /> |
There was a problem hiding this comment.
textAppearance를 통해 폰트 스타일을 지정하면서 textSize를 별도로 설정하면 일관성이 떨어지고 혼란을 줄 수 있습니다. TextAppearance.App.CaptionMd.Regular는 10sp로 정의되어 있지만, textSize는 12sp로 오버라이드하고 있습니다.
12sp가 의도한 크기라면, 12sp로 정의된 TextAppearance.App.CaptionLg.Regular 스타일을 사용하고 textSize 속성을 제거하는 것을 권장합니다. 10sp가 맞다면 textSize 속성을 제거해주세요.
| android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular" | |
| android:textColor="@color/gray_500" | |
| android:textSize="12sp" /> | |
| android:textAppearance="@style/TextAppearance.App.CaptionLg.Regular" | |
| android:textColor="@color/gray_500" /> |
PR템플릿
변경 사항 요약
권한허용창을 피그마에 있는대로 했지만 글씨가 너무 작아서 물어봐야함
체크리스트
사진올리는곳