Skip to content

Commit 6340cf6

Browse files
Fix Android Lint issues. Moved ripple effect to foreground of views and updated navigation component.
1 parent 17a493b commit 6340cf6

File tree

5 files changed

+6
-6
lines changed

5 files changed

+6
-6
lines changed

app/src/main/java/guide/graphql/toc/ui/sections/SectionsFragment.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class SectionsFragment : Fragment() {
1919
private val viewModel: SectionsViewModel by viewModels()
2020

2121
private lateinit var binding: SectionsFragmentBinding
22-
val args: SectionsFragmentArgs by navArgs()
22+
private val args: SectionsFragmentArgs by navArgs()
2323

2424
override fun onCreateView(
2525
inflater: LayoutInflater,

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
xmlns:app="http://schemas.android.com/apk/res-auto"
44
android:layout_width="match_parent"
55
android:layout_height="wrap_content"
6-
android:background="?selectableItemBackground"
6+
android:foreground="?selectableItemBackground"
77
android:paddingStart="20dp"
88
android:paddingEnd="20dp"
99
android:paddingTop="16dp"

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
xmlns:app="http://schemas.android.com/apk/res-auto"
44
android:layout_width="match_parent"
55
android:layout_height="wrap_content"
6-
android:background="?selectableItemBackground"
6+
android:foreground="?selectableItemBackground"
77
android:paddingStart="30dp"
88
android:paddingTop="20dp"
99
android:paddingEnd="30dp"

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,12 @@
1212
</style>
1313

1414
<style name="AppTheme.Heading">
15-
<item name="android:textSize">18dp</item>
15+
<item name="android:textSize">18sp</item>
1616
<item name="android:textStyle">bold</item>
1717
</style>
1818

1919
<style name="AppTheme.Subheading">
20-
<item name="android:textSize">18dp</item>
20+
<item name="android:textSize">18sp</item>
2121
</style>
2222

2323
</resources>

build.gradle.kts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ buildscript {
66

77
dependencies {
88
classpath("com.android.tools.build:gradle:4.1.0-beta03")
9-
classpath("androidx.navigation:navigation-safe-args-gradle-plugin:2.2.2")
9+
classpath("androidx.navigation:navigation-safe-args-gradle-plugin:2.3.0")
1010
classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.72")
1111
}
1212
}

0 commit comments

Comments
 (0)