diff --git a/app/src/main/java/com/cappielloantonio/play/adapter/AlbumAdapter.java b/app/src/main/java/com/cappielloantonio/play/adapter/AlbumAdapter.java index 7d9b8a50..6ebd140c 100644 --- a/app/src/main/java/com/cappielloantonio/play/adapter/AlbumAdapter.java +++ b/app/src/main/java/com/cappielloantonio/play/adapter/AlbumAdapter.java @@ -74,14 +74,14 @@ public class AlbumAdapter extends RecyclerView.Adapter @Override public void onClick(View view) { Bundle bundle = new Bundle(); - bundle.putParcelable("album_object", albums.get(getAdapterPosition())); + bundle.putParcelable("album_object", albums.get(getBindingAdapterPosition())); Navigation.findNavController(view).navigate(R.id.action_libraryFragment_to_albumPageFragment, bundle); } @Override public boolean onLongClick(View v) { Bundle bundle = new Bundle(); - bundle.putParcelable("album_object", albums.get(getAdapterPosition())); + bundle.putParcelable("album_object", albums.get(getBindingAdapterPosition())); Navigation.findNavController(v).navigate(R.id.albumBottomSheetDialog, bundle); return true; } diff --git a/app/src/main/java/com/cappielloantonio/play/adapter/AlbumArtistPageAdapter.java b/app/src/main/java/com/cappielloantonio/play/adapter/AlbumArtistPageAdapter.java index d452a0fd..1c331ddb 100644 --- a/app/src/main/java/com/cappielloantonio/play/adapter/AlbumArtistPageAdapter.java +++ b/app/src/main/java/com/cappielloantonio/play/adapter/AlbumArtistPageAdapter.java @@ -71,14 +71,14 @@ public class AlbumArtistPageAdapter extends RecyclerView.Adapter @Override public void onClick(View view) { if (itemClickListener != null) - itemClickListener.onItemClick(view, getAdapterPosition()); + itemClickListener.onItemClick(view, getBindingAdapterPosition()); } } diff --git a/app/src/main/java/com/cappielloantonio/play/adapter/GenreCatalogueAdapter.java b/app/src/main/java/com/cappielloantonio/play/adapter/GenreCatalogueAdapter.java index 50452363..0839e225 100644 --- a/app/src/main/java/com/cappielloantonio/play/adapter/GenreCatalogueAdapter.java +++ b/app/src/main/java/com/cappielloantonio/play/adapter/GenreCatalogueAdapter.java @@ -61,7 +61,7 @@ public class GenreCatalogueAdapter extends RecyclerView.Adapter { @Override public void onClick(View view) { - if (itemClickListener != null) itemClickListener.onItemClick(view, getAdapterPosition()); + if (itemClickListener != null) itemClickListener.onItemClick(view, getBindingAdapterPosition()); } } diff --git a/app/src/main/res/layout/item_player_now_playing_song.xml b/app/src/main/res/layout/item_player_now_playing_song.xml index ce8af70e..35d42ade 100644 --- a/app/src/main/res/layout/item_player_now_playing_song.xml +++ b/app/src/main/res/layout/item_player_now_playing_song.xml @@ -3,9 +3,7 @@ xmlns:app="http://schemas.android.com/apk/res-auto" android:layout_width="match_parent" android:layout_height="match_parent" - android:layout_marginStart="24dp" - android:layout_marginEnd="24dp" - android:layout_marginBottom="8dp" + android:layout_margin="24dp" android:backgroundTint="@color/cardColor" app:cardCornerRadius="4dp"> @@ -13,6 +11,6 @@ android:id="@+id/now_playing_song_cover_image_view" android:layout_width="match_parent" android:layout_height="match_parent" - android:background="?android:attr/selectableItemBackground"/> + android:foreground="?android:attr/selectableItemBackground"/> \ No newline at end of file diff --git a/app/src/main/res/layout/player_body_bottom_sheet.xml b/app/src/main/res/layout/player_body_bottom_sheet.xml index 6ee8ddc9..82eea61d 100644 --- a/app/src/main/res/layout/player_body_bottom_sheet.xml +++ b/app/src/main/res/layout/player_body_bottom_sheet.xml @@ -13,6 +13,7 @@ android:layout_width="wrap_content" android:layout_height="wrap_content" android:background="@android:color/transparent" + android:layout_marginBottom="-4dp" android:paddingStart="24dp" android:paddingTop="20dp" android:text="Now playing" @@ -26,11 +27,10 @@ android:id="@+id/player_song_cover_view_pager" android:layout_width="0dp" android:layout_height="0dp" - android:layout_marginTop="20dp" app:layout_constraintDimensionRatio="H,1:1" app:layout_constraintLeft_toLeftOf="parent" app:layout_constraintRight_toRightOf="parent" - app:layout_constraintTop_toBottomOf="@+id/collapse_bottom_sheet_button" /> + app:layout_constraintTop_toBottomOf="@+id/collapse_bottom_sheet_button"/>