Skip to content
This repository was archived by the owner on Nov 21, 2024. It is now read-only.

Commit d8d413e

Browse files
committed
Move line heights to theme text styles.
Change-Id: I6259471405b064ada7fcfcae3b6ac6f485113928
1 parent 2677e7b commit d8d413e

File tree

6 files changed

+9
-13
lines changed

6 files changed

+9
-13
lines changed

app/src/main/res/layout/course_item.xml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,7 @@
8181
android:lines="2"
8282
android:ellipsize="end"
8383
android:text="@{course.name}"
84-
android:textAppearance="?attr/textAppearanceSubtitle1"
85-
app:lineHeight="22sp"/>
84+
android:textAppearance="?attr/textAppearanceSubtitle1"/>
8685

8786
<ImageView
8887
android:id="@+id/course_icon"

app/src/main/res/layout/featured_item.xml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,7 @@
9898
android:paddingHorizontal="@dimen/grid_2"
9999
android:gravity="center"
100100
android:text="@{course.name}"
101-
android:textAppearance="?attr/textAppearanceSubtitle1"
102-
app:lineHeight="22sp"/>
101+
android:textAppearance="?attr/textAppearanceSubtitle1"/>
103102

104103
<androidx.constraintlayout.widget.Guideline
105104
android:id="@+id/guideline"

app/src/main/res/layout/fragment_learn.xml

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -118,8 +118,7 @@
118118
android:paddingHorizontal="@dimen/grid_2"
119119
android:gravity="center"
120120
android:text="@{course.name}"
121-
android:textAppearance="?attr/textAppearanceHeadline4"
122-
app:lineHeight="40sp"/>
121+
android:textAppearance="?attr/textAppearanceHeadline4"/>
123122

124123
<TextView
125124
android:id="@+id/description"
@@ -132,7 +131,6 @@
132131
android:padding="@dimen/grid_2"
133132
android:gravity="center"
134133
android:textAppearance="?attr/textAppearanceBody1"
135-
app:lineHeight="28sp"
136134
android:text="@string/course_desc"/>
137135

138136
<View
@@ -154,7 +152,6 @@
154152
android:padding="@dimen/grid_2"
155153
android:gravity="center"
156154
android:textAppearance="?attr/textAppearanceHeadline6"
157-
app:lineHeight="28sp"
158155
android:text="@string/what_you_ll_need"/>
159156

160157
<TextView
@@ -167,7 +164,6 @@
167164
android:padding="@dimen/grid_2"
168165
android:gravity="center"
169166
android:textAppearance="?attr/textAppearanceBody1"
170-
app:lineHeight="28sp"
171167
android:text="@string/needs"/>
172168

173169
<View
@@ -193,7 +189,6 @@
193189
android:gravity="center"
194190
android:textAppearance="?attr/textAppearanceHeadline6"
195191
android:textColor="?attr/colorOnPrimarySurface"
196-
app:lineHeight="28sp"
197192
android:text="@string/you_ll_also_like"
198193
android:theme="@style/ThemeOverlay.Owl.Blue"/>
199194

app/src/main/res/layout/related_course_item.xml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,7 @@
6161
android:maxLines="2"
6262
android:ellipsize="end"
6363
android:text="@{course.name}"
64-
android:textAppearance="?attr/textAppearanceBody2"
65-
app:lineHeight="16sp"/>
64+
android:textAppearance="?attr/textAppearanceBody2"/>
6665

6766
<ImageView
6867
android:id="@+id/course_icon"

app/src/main/res/layout/step_item.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@
4747
android:layout_marginTop="@dimen/grid_2"
4848
android:gravity="center"
4949
android:textAppearance="?attr/textAppearanceBody1"
50-
app:lineHeight="28sp"
5150
android:text="@{step.body}"/>
5251

5352
</LinearLayout>

app/src/main/res/values/type.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
<item name="fontFamily">@font/rubik_bold</item>
3434
<item name="android:textStyle">bold</item>
3535
<item name="android:textColor">?android:attr/textColorPrimary</item>
36+
<item name="lineHeight">40sp</item>
3637
</style>
3738

3839
<style name="TextAppearance.Owl.Headline5" parent="TextAppearance.MaterialComponents.Headline5">
@@ -42,10 +43,12 @@
4243

4344
<style name="TextAppearance.Owl.Headline6" parent="TextAppearance.MaterialComponents.Headline6">
4445
<item name="fontFamily">@font/rubik_medium</item>
46+
<item name="lineHeight">28sp</item>
4547
</style>
4648

4749
<style name="TextAppearance.Owl.Subtitle1" parent="TextAppearance.MaterialComponents.Subtitle1">
4850
<item name="fontFamily">@font/rubik_medium</item>
51+
<item name="lineHeight">22sp</item>
4952
</style>
5053

5154
<style name="TextAppearance.Owl.Subtitle2" parent="TextAppearance.MaterialComponents.Subtitle2">
@@ -54,10 +57,12 @@
5457

5558
<style name="TextAppearance.Owl.Body1" parent="TextAppearance.MaterialComponents.Body1">
5659
<item name="fontFamily">@font/rubik</item>
60+
<item name="lineHeight">28sp</item>
5761
</style>
5862

5963
<style name="TextAppearance.Owl.Body2" parent="TextAppearance.MaterialComponents.Body2">
6064
<item name="fontFamily">@font/rubik</item>
65+
<item name="lineHeight">16sp</item>
6166
</style>
6267

6368
<style name="TextAppearance.Owl.Button" parent="TextAppearance.MaterialComponents.Button">

0 commit comments

Comments
 (0)