Merge pull request #87 from wordpress-mobile/issue/13396-media-picker-edge-to-edge-support

Media picker edge to edge support

Author: atorresveiga

mediapicker/src/main/java/org/wordpress/android/mediapicker/ui/MediaPickerActivity.kt

@@ -6,11 +6,13 @@ import android.content.Intent
 import android.os.Bundle
 import android.view.MenuItem
 import androidx.appcompat.app.AppCompatActivity
+import androidx.core.view.updatePadding
 import dagger.hilt.android.AndroidEntryPoint
 import org.wordpress.android.mediapicker.R
 import org.wordpress.android.mediapicker.api.Log
 import org.wordpress.android.mediapicker.api.MediaPickerSetup
 import org.wordpress.android.mediapicker.databinding.MediaPickerLibActivityBinding
+import org.wordpress.android.mediapicker.util.doOnApplyWindowInsets
 import javax.inject.Inject
 
 @AndroidEntryPoint
@@ -27,6 +29,17 @@ class MediaPickerActivity : AppCompatActivity() {
             setDisplayHomeAsUpEnabled(true)
             setDisplayShowTitleEnabled(true)
         }
+        applyEdgeToEdgeSettings(binding)
+    }
+
+    fun applyEdgeToEdgeSettings(binding: MediaPickerLibActivityBinding) {
+        binding.root.doOnApplyWindowInsets { insets ->
+            binding.toolbarMain.setPadding(0, insets.top, 0, 0)
+            binding.root.updatePadding(
+                left = insets.left,
+                right = insets.right
+            )
+        }
     }
 
     override fun onOptionsItemSelected(item: MenuItem): Boolean {

mediapicker/src/main/java/org/wordpress/android/mediapicker/ui/MediaPickerFragment.kt

@@ -20,7 +20,9 @@ import androidx.appcompat.app.AppCompatActivity
 import androidx.appcompat.widget.SearchView
 import androidx.core.text.HtmlCompat
 import androidx.core.view.MenuProvider
+import androidx.core.view.WindowInsetsCompat
 import androidx.core.view.isVisible
+import androidx.core.view.updatePadding
 import androidx.fragment.app.Fragment
 import androidx.fragment.app.viewModels
 import androidx.lifecycle.lifecycleScope
@@ -68,6 +70,7 @@ import org.wordpress.android.mediapicker.util.MediaPickerLinkMovementMethod
 import org.wordpress.android.mediapicker.util.MediaPickerPermissionUtils
 import org.wordpress.android.mediapicker.util.MediaUtils
 import org.wordpress.android.mediapicker.util.UiHelpers
+import org.wordpress.android.mediapicker.util.doOnApplyWindowInsets
 import org.wordpress.android.mediapicker.viewmodel.MediaPickerViewModel
 import org.wordpress.android.mediapicker.viewmodel.observeEvent
 import javax.inject.Inject
@@ -225,6 +228,7 @@ internal class MediaPickerFragment : Fragment(), MenuProvider {
                     }
                     setupFab(uiState.fabUiModel)
                     pullToRefresh.isRefreshing = uiState.isRefreshing
+                    applyEdgeToEdgeSettings(this)
                 }
             }
 
@@ -277,6 +281,19 @@ internal class MediaPickerFragment : Fragment(), MenuProvider {
         }
     }
 
+    private fun applyEdgeToEdgeSettings(binding: MediaPickerLibFragmentBinding){
+        binding.root.doOnApplyWindowInsets(
+            insetsMask = WindowInsetsCompat.Type.navigationBars(),
+            consumeInsets = true
+        ) { insets ->
+            binding.recycler.clipToPadding = false
+            binding.recycler.updatePadding(bottom = insets.bottom)
+            binding.actionableEmptyView.updatePadding(bottom = insets.bottom)
+            binding.softAskView.updatePadding(bottom = insets.bottom)
+            binding.loadingView.updatePadding(bottom = insets.bottom)
+        }
+    }
+
     private fun onActionSelected(action: MediaPickerAction) {
         when (action) {
             is OpenSystemPicker -> {

mediapicker/src/main/java/org/wordpress/android/mediapicker/util/ViewUtils.kt

@@ -8,6 +8,9 @@ import android.view.View
 import android.view.animation.AccelerateDecelerateInterpolator
 import android.view.animation.AnimationUtils
 import android.view.animation.LinearInterpolator
+import androidx.core.graphics.Insets
+import androidx.core.view.ViewCompat
+import androidx.core.view.WindowInsetsCompat
 
 fun View.redirectContextClickToLongPressListener() {
     this.setOnContextClickListener { it.performLongClick() }
@@ -62,3 +65,14 @@ fun View?.fadeOut(duration: Long) {
         }.start()
     }
 }
+
+inline fun View.doOnApplyWindowInsets(
+    insetsMask: Int = WindowInsetsCompat.Type.systemBars() or WindowInsetsCompat.Type.displayCutout(),
+    consumeInsets: Boolean = false,
+    crossinline action: (Insets) -> Unit
+) {
+    ViewCompat.setOnApplyWindowInsetsListener(this) { _, insets ->
+        action(insets.getInsets(insetsMask))
+        if (consumeInsets) WindowInsetsCompat.CONSUMED else insets
+    }
+}