From 5de6314d268a9d425ea439da0739280f993d6cbd Mon Sep 17 00:00:00 2001
From: dkanada <dkanada@users.noreply.github.com>
Date: Wed, 6 May 2020 00:16:58 +0900
Subject: [PATCH] fix separator issues on genre adapter

---
 .../kabouzeid/gramophone/adapter/GenreAdapter.java   | 12 ++++--------
 .../gramophone/adapter/PlaylistAdapter.java          |  2 --
 app/src/main/res/layout/item_list_single_row.xml     |  3 ---
 3 files changed, 4 insertions(+), 13 deletions(-)

diff --git a/app/src/main/java/com/kabouzeid/gramophone/adapter/GenreAdapter.java b/app/src/main/java/com/kabouzeid/gramophone/adapter/GenreAdapter.java
index 9960a6cd..46b0241e 100644
--- a/app/src/main/java/com/kabouzeid/gramophone/adapter/GenreAdapter.java
+++ b/app/src/main/java/com/kabouzeid/gramophone/adapter/GenreAdapter.java
@@ -59,19 +59,15 @@ public class GenreAdapter extends RecyclerView.Adapter<GenreAdapter.ViewHolder>
         final Genre genre = dataSet.get(position);
 
         if (holder.getAdapterPosition() == getItemCount() - 1) {
-            if (holder.separator != null) {
-                holder.separator.setVisibility(View.GONE);
+            if (holder.shortSeparator != null) {
+                holder.shortSeparator.setVisibility(View.GONE);
             }
         } else {
-            if (holder.separator != null) {
-                holder.separator.setVisibility(View.VISIBLE);
+            if (holder.shortSeparator != null) {
+                holder.shortSeparator.setVisibility(View.VISIBLE);
             }
         }
 
-        if (holder.shortSeparator != null) {
-            holder.shortSeparator.setVisibility(View.GONE);
-        }
-
         if (holder.menu != null) {
             holder.menu.setVisibility(View.GONE);
         }
diff --git a/app/src/main/java/com/kabouzeid/gramophone/adapter/PlaylistAdapter.java b/app/src/main/java/com/kabouzeid/gramophone/adapter/PlaylistAdapter.java
index 2c35bc91..681e09ce 100644
--- a/app/src/main/java/com/kabouzeid/gramophone/adapter/PlaylistAdapter.java
+++ b/app/src/main/java/com/kabouzeid/gramophone/adapter/PlaylistAdapter.java
@@ -1,6 +1,5 @@
 package com.kabouzeid.gramophone.adapter;
 
-import android.graphics.PorterDuff;
 import android.graphics.drawable.Drawable;
 import androidx.annotation.LayoutRes;
 import androidx.annotation.NonNull;
@@ -13,7 +12,6 @@ import android.view.ViewGroup;
 import android.widget.PopupMenu;
 
 import com.bumptech.glide.Glide;
-import com.kabouzeid.appthemehelper.util.ATHUtil;
 import com.kabouzeid.gramophone.R;
 import com.kabouzeid.gramophone.adapter.base.AbsMultiSelectAdapter;
 import com.kabouzeid.gramophone.adapter.base.MediaEntryViewHolder;
diff --git a/app/src/main/res/layout/item_list_single_row.xml b/app/src/main/res/layout/item_list_single_row.xml
index 1a6ed7cb..4ac18cc4 100644
--- a/app/src/main/res/layout/item_list_single_row.xml
+++ b/app/src/main/res/layout/item_list_single_row.xml
@@ -29,8 +29,6 @@
             android:layout_width="wrap_content"
             android:layout_height="match_parent"
             android:paddingEnd="0dp"
-            android:paddingLeft="16dp"
-            android:paddingRight="0dp"
             android:paddingStart="16dp">
 
             <ImageView
@@ -89,7 +87,6 @@
         android:layout_width="match_parent"
         android:layout_height="1dp"
         android:layout_gravity="bottom"
-        android:layout_marginLeft="72dp"
         android:layout_marginStart="72dp"
         android:background="?attr/dividerColor" />