Allow consuming of insets when using setOnApplyInsetsListener (#89)

* Typo * Improve setOnApplyInsetsListener allowing to consume insets by itself * Use Insetter consume option when using setOnApplyInsetsListener as well * Fix code style * Fix test warnings
chrisbanes · Jan 21, 2021 · 9b1eed5 · 9b1eed5
1 parent 05310d4
commit 9b1eed5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/library/src/main/java/dev/chrisbanes/insetter/Insetter.kt b/library/src/main/java/dev/chrisbanes/insetter/Insetter.kt
@@ -368,7 +368,8 @@ class Insetter private constructor(builder: Builder) {
                 // If we have an onApplyInsetsListener, invoke it
                 onApplyInsetsListener.onApplyInsets(v, insets, initialState)
                 // We don't know what sides have been applied, so we assume all
-                return@setOnApplyWindowInsetsListener WindowInsetsCompat.CONSUMED
+                return@setOnApplyWindowInsetsListener if (consume != CONSUME_NONE) insets
+                else WindowInsetsCompat.CONSUMED
             }
 
             // Otherwise we applied through applyInsetsToView()

diff --git a/library/src/main/java/dev/chrisbanes/insetter/Types.kt b/library/src/main/java/dev/chrisbanes/insetter/Types.kt
@@ -19,7 +19,7 @@ package dev.chrisbanes.insetter
 import androidx.core.view.WindowInsetsCompat
 
 /**
- * Convenienve function for building a combination of [WindowInsetsCompat.Type] values.
+ * Convenience function for building a combination of [WindowInsetsCompat.Type] values.
  */
 fun windowInsetTypesOf(
     ime: Boolean = false,