Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions app/src/main/res/layout/fragment_calendar_select.xml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@
android:layout_height="wrap_content"
android:text="어떤 캘린더에 일정을 담아 드릴까요?"
android:gravity="center"
android:textAppearance="@style/TextAppearance.App.BodySm.Regular"
android:textColor="@color/text_primary"
android:layout_marginTop="40dp"
app:layout_constraintTop_toTopOf="parent"
app:layout_constraintEnd_toEndOf="parent"
Expand All @@ -24,6 +26,8 @@
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:text="설정에서 기본 캘린더를 변경할 수 있습니다."
android:textAppearance="@style/TextAppearance.App.ModalSm.Regular"
android:textColor="@color/text_secondary"
Comment on lines +29 to +30
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

TextAppearance.App.ModalSm.Regular 스타일이 styles.xml에 정의되어 있지 않아 빌드 오류가 발생합니다. 의도한 스타일이 TextAppearance.App.CaptionMd.Regular 또는 다른 기존 스타일인가요? 혹은 새로운 스타일을 추가해야 한다면 styles.xml에 정의해주세요.

android:gravity="center"
app:layout_constraintTop_toBottomOf="@id/tv_description"/>

Expand Down Expand Up @@ -65,6 +69,7 @@
android:layout_marginBottom="24dp"
android:background="@drawable/btn_main_round"
android:text="Pace 시작하기"
android:textAppearance="@style/TextAppearance.App.BodySm.Semibold"
android:textColor="@color/white"
android:textSize="16sp"
Comment on lines +72 to 74
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

textAppearance를 통해 폰트 스타일을 지정하면서 textSize를 별도로 설정하면 일관성이 떨어지고 혼란을 줄 수 있습니다. TextAppearance.App.BodySm.Semibold는 14sp로 정의되어 있지만, textSize는 16sp로 오버라이드하고 있습니다.
16sp가 의도한 크기라면, 16sp로 정의된 TextAppearance.App.BodyMd.SemiBold 스타일을 사용하고 textSize 속성을 제거하는 것을 권장합니다.

Suggested change
android:textAppearance="@style/TextAppearance.App.BodySm.Semibold"
android:textColor="@color/white"
android:textSize="16sp"
android:textAppearance="@style/TextAppearance.App.BodyMd.SemiBold"
android:textColor="@color/white"

android:elevation="0dp"
Expand Down
7 changes: 4 additions & 3 deletions app/src/main/res/layout/fragment_onboarding1.xml
Original file line number Diff line number Diff line change
Expand Up @@ -25,16 +25,17 @@
app:layout_constraintBottom_toTopOf="@+id/centerLine"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
android:layout_marginBottom="20dp"/>
android:layout_marginBottom="13dp"/>

<TextView
android:id="@+id/tvDescription"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="30dp"
android:layout_marginTop="50dp"
android:gravity="center"
tools:text="일정부터 출발까지 \n하루의 흐름을, 나의 페이스로"
android:textColor="@color/secondary_900"
android:textAppearance="@style/TextAppearance.App.BodyMd.Regular"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

TextAppearance.App.BodyMd.Regular 스타일이 styles.xml에 정의되어 있지 않아 빌드 오류가 발생합니다. styles.xmlBodyMd.Regular 스타일을 추가하거나, TextAppearance.App.BodyMd.Medium과 같은 기존 스타일을 사용하도록 수정해주세요.

Suggested change
android:textAppearance="@style/TextAppearance.App.BodyMd.Regular"
android:textAppearance="@style/TextAppearance.App.BodyMd.Medium"

android:textColor="@color/text_primary"
android:lineSpacingExtra="4dp"
android:alpha="1"
app:layout_constraintEnd_toEndOf="parent"
Expand Down
4 changes: 3 additions & 1 deletion app/src/main/res/layout/fragment_onboarding2.xml
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,11 @@
android:id="@+id/tvDescription"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="30dp"
android:layout_marginTop="50dp"
android:gravity="center"
tools:text="하루의 일정과 이동 사이에서 \n당신의 페이스는 어땠나요?"
android:textAppearance="@style/TextAppearance.App.BodyMd.Regular"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

TextAppearance.App.BodyMd.Regular 스타일이 styles.xml에 정의되어 있지 않아 빌드 오류가 발생합니다. styles.xmlBodyMd.Regular 스타일을 추가하거나, TextAppearance.App.BodyMd.Medium과 같은 기존 스타일을 사용하도록 수정해주세요.

Suggested change
android:textAppearance="@style/TextAppearance.App.BodyMd.Regular"
android:textAppearance="@style/TextAppearance.App.BodyMd.Medium"

android:textColor="@color/text_primary"
android:lineSpacingExtra="4dp"
android:alpha="1"
app:layout_constraintEnd_toEndOf="parent"
Expand Down
4 changes: 3 additions & 1 deletion app/src/main/res/layout/fragment_onboarding3.xml
Original file line number Diff line number Diff line change
Expand Up @@ -66,9 +66,11 @@
android:id="@+id/tvDescription"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="30dp"
android:layout_marginTop="50dp"
android:gravity="center"
tools:text="일정은 캘린더로, 경로는 지도로\n출발 타이밍까지 신경 쓰다 보면\n하루의 흐름과 내 페이스를 놓치곤 해요."
android:textAppearance="@style/TextAppearance.App.BodyMd.Regular"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

TextAppearance.App.BodyMd.Regular 스타일이 styles.xml에 정의되어 있지 않아 빌드 오류가 발생합니다. styles.xmlBodyMd.Regular 스타일을 추가하거나, TextAppearance.App.BodyMd.Medium과 같은 기존 스타일을 사용하도록 수정해주세요.

Suggested change
android:textAppearance="@style/TextAppearance.App.BodyMd.Regular"
android:textAppearance="@style/TextAppearance.App.BodyMd.Medium"

android:textColor="@color/text_primary"
android:lineSpacingExtra="4dp"
android:alpha="1"
app:layout_constraintEnd_toEndOf="parent"
Expand Down
4 changes: 3 additions & 1 deletion app/src/main/res/layout/fragment_onboarding4.xml
Original file line number Diff line number Diff line change
Expand Up @@ -49,9 +49,11 @@
android:id="@+id/tvDescription"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="30dp"
android:layout_marginTop="50dp"
android:gravity="center"
tools:text="이제는 일정과 경로를 하나로\n출발까지의 흐름을 Pace가 이어드려요."
android:textAppearance="@style/TextAppearance.App.BodyMd.Regular"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

TextAppearance.App.BodyMd.Regular 스타일이 styles.xml에 정의되어 있지 않아 빌드 오류가 발생합니다. styles.xmlBodyMd.Regular 스타일을 추가하거나, TextAppearance.App.BodyMd.Medium과 같은 기존 스타일을 사용하도록 수정해주세요.

Suggested change
android:textAppearance="@style/TextAppearance.App.BodyMd.Regular"
android:textAppearance="@style/TextAppearance.App.BodyMd.Medium"

android:textColor="@color/text_primary"
android:lineSpacingExtra="4dp"
android:alpha="1"
app:layout_constraintEnd_toEndOf="parent"
Expand Down
32 changes: 21 additions & 11 deletions app/src/main/res/layout/fragment_permission.xml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@
android:layout_marginTop="24dp"
android:gravity="center"
android:text="PACE 이용을 위해\n아래 권한을 허용해주세요."
android:textAppearance="@style/TextAppearance.App.BodyMd.Medium"
android:textColor="@color/text_primary"
android:textStyle="bold"
Comment on lines +28 to 30
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

textAppearance를 통해 폰트 스타일을 지정하면서 textStyle을 별도로 설정하면 일관성이 떨어지고 혼란을 줄 수 있습니다. TextAppearance.App.BodyMd.Mediumpretendard_medium 폰트를 사용하는데, textStyle="bold"와 충돌합니다.
굵은 글씨가 의도된 것이라면, pretendard_semibold 폰트를 사용하는 TextAppearance.App.BodyMd.SemiBold 스타일을 사용하고 textStyle 속성을 제거하는 것을 권장합니다.

Suggested change
android:textAppearance="@style/TextAppearance.App.BodyMd.Medium"
android:textColor="@color/text_primary"
android:textStyle="bold"
android:textAppearance="@style/TextAppearance.App.BodyMd.SemiBold"
android:textColor="@color/text_primary"

app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
Expand All @@ -44,8 +46,8 @@
android:layout_height="wrap_content"
android:layout_marginTop="24dp"
android:text="필수 접근 권한"
android:textColor="@color/black"
android:textSize="14sp"
android:textAppearance="@style/TextAppearance.App.BodySm.Semibold"
android:textColor="@color/text_primary"
android:textStyle="bold"
Comment on lines +49 to 51
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

TextAppearance.App.BodySm.Semibold 스타일은 이미 pretendard_semibold 폰트를 사용하여 굵은 글씨를 표현하므로, android:textStyle="bold" 속성은 불필요합니다. 스타일 가이드의 일관성을 위해 제거하는 것이 좋습니다.

Suggested change
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"

app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toBottomOf="@id/divider" />
Expand Down Expand Up @@ -73,21 +75,24 @@
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:text="위치"
android:textColor="@color/black"
android:textAppearance="@style/TextAppearance.App.BodySm.Medium"
android:textColor="@color/text_primary"
android:textStyle="bold" />
Comment on lines +78 to 80
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

textAppearance를 통해 폰트 스타일을 지정하면서 textStyle을 별도로 설정하면 일관성이 떨어지고 혼란을 줄 수 있습니다. TextAppearance.App.BodySm.Mediumpretendard_medium 폰트를 사용하는데, textStyle="bold"와 충돌합니다.
굵은 글씨가 의도된 것이라면, pretendard_semibold 폰트를 사용하는 TextAppearance.App.BodySm.Semibold 스타일을 사용하고 textStyle 속성을 제거하는 것을 권장합니다.

Suggested change
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" />

<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular"
android:textColor="@color/text_secondary"
android:text="내 현재 위치는 장소 검색 및 경로에서 사용자 위치 확인을 \n위한 정보로 활용됩니다."
android:textSize="12sp" />
/>
Comment on lines +84 to +87
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

PR 설명에서 언급하신 것처럼 폰트 크기가 작아 보일 수 있습니다. TextAppearance.App.CaptionMd.Regular 스타일은 10sp로, 기존의 12sp보다 작습니다.
만약 12sp 크기를 유지하고 싶다면, 12sp로 정의된 TextAppearance.App.CaptionLg.Regular 스타일을 사용하는 것을 고려해 보세요.

Suggested change
android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular"
android:textColor="@color/text_secondary"
android:text="내 현재 위치는 장소 검색 및 경로에서 사용자 위치 확인을 \n위한 정보로 활용됩니다."
android:textSize="12sp" />
/>
android:textAppearance="@style/TextAppearance.App.CaptionLg.Regular"
android:textColor="@color/text_secondary"
android:text="내 현재 위치는 장소 검색 및 경로에서 사용자 위치 확인을 \n위한 정보로 활용됩니다." />

</LinearLayout>
</LinearLayout>

<LinearLayout
android:id="@+id/layout_calendar"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:layout_marginTop="22dp"
android:layout_marginTop="17dp"
android:orientation="horizontal"
android:gravity="center_vertical"
app:layout_constraintTop_toBottomOf="@id/layout_location">
Expand All @@ -106,13 +111,15 @@
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:text="캘린더"
android:textColor="@color/black"
android:textAppearance="@style/TextAppearance.App.BodySm.Medium"
android:textColor="@color/text_primary"
android:textStyle="bold" />
Comment on lines +114 to 116
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

textAppearance를 통해 폰트 스타일을 지정하면서 textStyle을 별도로 설정하면 일관성이 떨어지고 혼란을 줄 수 있습니다. TextAppearance.App.BodySm.Mediumpretendard_medium 폰트를 사용하는데, textStyle="bold"와 충돌합니다.
굵은 글씨가 의도된 것이라면, pretendard_semibold 폰트를 사용하는 TextAppearance.App.BodySm.Semibold 스타일을 사용하고 textStyle 속성을 제거하는 것을 권장합니다.

Suggested change
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" />

<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:text="일정 확인, 추가, 수정 및 연동을 위해 권한이 필요합니다."
android:textSize="12sp" />
android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular"
android:textColor="@color/text_secondary" />
Comment on lines +121 to +122
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

이 설명 텍스트 또한 12sp에서 10sp로 크기가 줄었습니다. 12sp 크기를 유지하고 싶다면, TextAppearance.App.CaptionLg.Regular 스타일을 사용하는 것을 고려해 보세요.

Suggested change
android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular"
android:textColor="@color/text_secondary" />
android:textAppearance="@style/TextAppearance.App.CaptionLg.Regular"
android:textColor="@color/text_secondary" />

</LinearLayout>
</LinearLayout>

Expand All @@ -131,8 +138,8 @@
android:layout_height="wrap_content"
android:layout_marginTop="48dp"
android:text="선택 접근 권한"
android:textColor="@color/black"
android:textSize="14sp"
android:textAppearance="@style/TextAppearance.App.BodySm.Semibold"
android:textColor="@color/text_primary"
android:textStyle="bold"
Comment on lines +141 to 143
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

TextAppearance.App.BodySm.Semibold 스타일은 이미 pretendard_semibold 폰트를 사용하여 굵은 글씨를 표현하므로, android:textStyle="bold" 속성은 불필요합니다. 스타일 가이드의 일관성을 위해 제거하는 것이 좋습니다.

Suggested change
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"

app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toBottomOf="@id/layout_calendar" />
Expand Down Expand Up @@ -160,13 +167,15 @@
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:text="알림"
android:textColor="@color/black"
android:textAppearance="@style/TextAppearance.App.BodySm.Medium"
android:textColor="@color/text_primary"
android:textStyle="bold" />
Comment on lines +170 to 172
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

textAppearance를 통해 폰트 스타일을 지정하면서 textStyle을 별도로 설정하면 일관성이 떨어지고 혼란을 줄 수 있습니다. TextAppearance.App.BodySm.Mediumpretendard_medium 폰트를 사용하는데, textStyle="bold"와 충돌합니다.
굵은 글씨가 의도된 것이라면, pretendard_semibold 폰트를 사용하는 TextAppearance.App.BodySm.Semibold 스타일을 사용하고 textStyle 속성을 제거하는 것을 권장합니다.

Suggested change
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" />

<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:text="일정 및 출발 알림 제공을 위해 권한이 필요합니다."
android:textSize="12sp" />
android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular"
android:textColor="@color/text_secondary" />
Comment on lines +177 to +178
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

이 설명 텍스트 또한 12sp에서 10sp로 크기가 줄었습니다. 12sp 크기를 유지하고 싶다면, TextAppearance.App.CaptionLg.Regular 스타일을 사용하는 것을 고려해 보세요.

Suggested change
android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular"
android:textColor="@color/text_secondary" />
android:textAppearance="@style/TextAppearance.App.CaptionLg.Regular"
android:textColor="@color/text_secondary" />

</LinearLayout>
</LinearLayout>

Expand All @@ -176,6 +185,7 @@
android:layout_height="wrap_content"
android:background="@color/primary_500"
android:text="다음"
android:textAppearance="@style/TextAppearance.App.BodySm.Semibold"
android:textColor="@color/white"
android:textSize="16sp"
android:textStyle="bold"
Comment on lines +188 to 191
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

버튼에 textAppearance를 사용하면서 textSizetextStyle을 별도로 지정하면 일관성이 떨어집니다.

  • TextAppearance.App.BodySm.Semibold는 14sp인데, textSize는 16sp로 오버라이드하고 있습니다.
  • BodySm.Semibold는 이미 pretendard_semibold 폰트를 사용하므로 textStyle="bold"는 불필요합니다.

16sp 굵은 글씨가 의도된 것이라면, TextAppearance.App.BodyMd.SemiBold 스타일을 사용하고 textSizetextStyle 속성을 제거하는 것을 권장합니다.

Suggested change
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"

Expand Down
7 changes: 4 additions & 3 deletions app/src/main/res/layout/item_calendar_account.xml
Original file line number Diff line number Diff line change
Expand Up @@ -24,15 +24,16 @@
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:text="내 휴대전화"
android:textColor="@color/black"
android:textSize="14sp" />
android:textAppearance="@style/TextAppearance.App.BodySm.Regular"
android:textColor="@color/text_primary" />

<TextView
android:id="@+id/tv_account_name"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:text="Pace0908@gmail.com"
android:textColor="@color/gray_400"
android:textAppearance="@style/TextAppearance.App.CaptionMd.Regular"
android:textColor="@color/gray_500"
android:textSize="12sp" />
Comment on lines +35 to 37
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

textAppearance를 통해 폰트 스타일을 지정하면서 textSize를 별도로 설정하면 일관성이 떨어지고 혼란을 줄 수 있습니다. TextAppearance.App.CaptionMd.Regular는 10sp로 정의되어 있지만, textSize는 12sp로 오버라이드하고 있습니다.
12sp가 의도한 크기라면, 12sp로 정의된 TextAppearance.App.CaptionLg.Regular 스타일을 사용하고 textSize 속성을 제거하는 것을 권장합니다. 10sp가 맞다면 textSize 속성을 제거해주세요.

Suggested change
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" />

</LinearLayout>

Expand Down