Use item decoration for recycler spacing

#212
alexandr7035 · Jan 4, 2022 · c97484a · c97484a
1 parent 2bcd966
commit c97484a
Show file tree

Hide file tree

Showing 5 changed files with 41 additions and 6 deletions.
diff --git a/app/src/main/java/by/alexandr7035/gitstat/view/repositories/ActiveRepositoriesFragment.kt b/app/src/main/java/by/alexandr7035/gitstat/view/repositories/ActiveRepositoriesFragment.kt
@@ -4,9 +4,11 @@ import android.os.Bundle
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
+import androidx.core.content.ContextCompat
 import androidx.fragment.app.Fragment
 import androidx.navigation.fragment.findNavController
 import androidx.navigation.navGraphViewModels
+import androidx.recyclerview.widget.DividerItemDecoration
 import androidx.recyclerview.widget.LinearLayoutManager
 import by.alexandr7035.gitstat.R
 import by.alexandr7035.gitstat.RepositoriesListGraphDirections
@@ -34,15 +36,28 @@ class ActiveRepositoriesFragment : Fragment(), RepoClickListener {
 
         // Setup adapter
         adapter = RepositoriesAdapter(this)
-        binding!!.recycler.adapter = adapter
-        binding!!.recycler.layoutManager = LinearLayoutManager(context)
+        binding?.recycler?.adapter = adapter
+        val layoutManager = LinearLayoutManager(context)
+
+        val decoration = DividerItemDecoration(
+            binding?.recycler?.context,
+            layoutManager.orientation
+        )
+
+        ContextCompat.getDrawable(requireContext(), R.drawable.decoration_repository_item)?.let {
+            decoration.setDrawable(it)
+        }
+
+        binding?.recycler?.addItemDecoration(decoration)
+        binding?.recycler?.layoutManager = layoutManager
+
 
         viewModel.getTabRefreshedLiveData().observeNullSafe(viewLifecycleOwner, {
             // If current fragment
             // FIXME find better solution
             if (it == 0) {
                 //binding!!.root.smoothScrollToPosition(0)
-                binding!!.recycler.layoutManager!!.scrollToPosition(0)
+                binding?.recycler?.layoutManager?.scrollToPosition(0)
             }
         })
     }

diff --git a/app/src/main/java/by/alexandr7035/gitstat/view/repositories/ArchivedRepositoriesFragment.kt b/app/src/main/java/by/alexandr7035/gitstat/view/repositories/ArchivedRepositoriesFragment.kt
@@ -4,9 +4,11 @@ import android.os.Bundle
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
+import androidx.core.content.ContextCompat
 import androidx.fragment.app.Fragment
 import androidx.navigation.fragment.findNavController
 import androidx.navigation.navGraphViewModels
+import androidx.recyclerview.widget.DividerItemDecoration
 import androidx.recyclerview.widget.LinearLayoutManager
 import by.alexandr7035.gitstat.R
 import by.alexandr7035.gitstat.RepositoriesListGraphDirections
@@ -35,8 +37,21 @@ class ArchivedRepositoriesFragment : Fragment(), RepoClickListener {
 
         // Setup adapter
         adapter = RepositoriesAdapter(this)
-        binding!!.recycler.adapter = adapter
-        binding!!.recycler.layoutManager = LinearLayoutManager(context)
+        binding?.recycler?.adapter = adapter
+        val layoutManager = LinearLayoutManager(context)
+
+        val decoration = DividerItemDecoration(
+            binding?.recycler?.context,
+            layoutManager.orientation
+        )
+
+        ContextCompat.getDrawable(requireContext(), R.drawable.decoration_repository_item)?.let {
+            decoration.setDrawable(it)
+        }
+
+        binding?.recycler?.addItemDecoration(decoration)
+        binding?.recycler?.layoutManager = layoutManager
+
 
         viewModel.getTabRefreshedLiveData().observeNullSafe(viewLifecycleOwner, {
             // If current fragment

diff --git a/app/src/main/res/drawable/decoration_repository_item.xml b/app/src/main/res/drawable/decoration_repository_item.xml
@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="utf-8"?>
+<shape xmlns:android="http://schemas.android.com/apk/res/android">
+    <size android:height="@dimen/decoration_repository_item_spacing"/>
+    <solid android:color="@android:color/transparent"/>
+</shape>
diff --git a/app/src/main/res/layout/view_repository.xml b/app/src/main/res/layout/view_repository.xml
@@ -7,7 +7,6 @@
     android:paddingStart="16dp"
     android:paddingEnd="16dp"
     android:elevation="5dp"
-    android:layout_marginBottom="16dp"
     android:background="@drawable/background_repository_item"
     xmlns:tools="http://schemas.android.com/tools"
     xmlns:app="http://schemas.android.com/apk/res-auto">

diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
@@ -4,4 +4,5 @@
     <dimen name="profile_card_field_margin">16dp</dimen>
     <dimen name="profile_card_label_margin">8dp</dimen>
     <dimen name="profile_fields_vertical_spacing">4dp</dimen>
+    <dimen name="decoration_repository_item_spacing">16dp</dimen>
 </resources>