From 8f614d82ae2068259b86fdf8106ff9c575ef57be Mon Sep 17 00:00:00 2001 From: ericdecanini Date: Tue, 6 Sep 2022 10:26:43 +0200 Subject: [PATCH] Fixes padding of selectable background in recents item --- .../list/home/recent/RecentRoomCarouselController.kt | 11 +++-------- vector/src/main/res/layout/item_recent_room.xml | 3 ++- 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/vector/src/main/java/im/vector/app/features/home/room/list/home/recent/RecentRoomCarouselController.kt b/vector/src/main/java/im/vector/app/features/home/room/list/home/recent/RecentRoomCarouselController.kt index 0c5d9a3533..df5ce28da5 100644 --- a/vector/src/main/java/im/vector/app/features/home/room/list/home/recent/RecentRoomCarouselController.kt +++ b/vector/src/main/java/im/vector/app/features/home/room/list/home/recent/RecentRoomCarouselController.kt @@ -41,7 +41,7 @@ class RecentRoomCarouselController @Inject constructor( private val hPadding = TypedValue.applyDimension( TypedValue.COMPLEX_UNIT_DIP, - 16f, + 4f, resources.displayMetrics ).toInt() @@ -51,12 +51,6 @@ class RecentRoomCarouselController @Inject constructor( resources.displayMetrics ).toInt() - private val itemSpacing = TypedValue.applyDimension( - TypedValue.COMPLEX_UNIT_DIP, - 24f, - resources.displayMetrics - ).toInt() - fun submitList(recentList: List) { this.data = recentList requestModelBuild() @@ -72,7 +66,8 @@ class RecentRoomCarouselController @Inject constructor( host.topPadding, host.hPadding, 0, - host.itemSpacing) + 0, + ) ) onBind { _, view, _ -> val colorSurface = MaterialColors.getColor(view, R.attr.vctr_toolbar_background) diff --git a/vector/src/main/res/layout/item_recent_room.xml b/vector/src/main/res/layout/item_recent_room.xml index 2cd8ff49c5..b2d311d328 100644 --- a/vector/src/main/res/layout/item_recent_room.xml +++ b/vector/src/main/res/layout/item_recent_room.xml @@ -3,12 +3,13 @@ xmlns:app="http://schemas.android.com/apk/res-auto" xmlns:tools="http://schemas.android.com/tools" android:id="@+id/recentRoot" - android:layout_width="60dp" + android:layout_width="84dp" android:layout_height="wrap_content" android:background="?vctr_toolbar_background" android:clickable="true" android:focusable="true" android:foreground="?attr/selectableItemBackground" + android:paddingHorizontal="12dp" tools:viewBindingIgnore="true">