Reuse contentInset values when updating for keyboard to ensure we don…

…'t clobber other values.
square · Feb 10, 2021 · 7e21b08 · 7e21b08
1 parent 4ad8474
commit 7e21b08
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,8 @@
 
 ### Fixed
 
+- [When updating `contentInset`, retain the values pulled from the `CollectionView`](TODO). This is to avoid clobbering the content inset potentially set by other things like navigation controllers. 
+
 ### Added
 
 ### Removed

diff --git a/ListableUI/Sources/ListView/ListView.swift b/ListableUI/Sources/ListView/ListView.swift
@@ -325,12 +325,9 @@ public final class ListView : UIView, KeyboardObserverDelegate
             $0.bottom = max($0.bottom, keyboardBottomInset)
         }
 
-        let contentInsets = UIEdgeInsets(
-            top: 0,
-            left: 0,
-            bottom: keyboardBottomInset,
-            right: 0
-        )
+        let contentInsets = modified(self.collectionView.contentInset) {
+            $0.bottom = keyboardBottomInset
+        }
 
         return (contentInsets, scrollIndicatorInsets)
     }