Merge pull request #7015 from vector-im/bugfix/eric/recents-bg-color
New Layout - Fixes recents background color not matching app bar
This commit is contained in:
commit
ab989e9ff0
@ -23,6 +23,8 @@ import com.airbnb.epoxy.CarouselModelBuilder
|
||||
import com.airbnb.epoxy.EpoxyController
|
||||
import com.airbnb.epoxy.EpoxyModel
|
||||
import com.airbnb.epoxy.carousel
|
||||
import com.google.android.material.color.MaterialColors
|
||||
import im.vector.app.R
|
||||
import im.vector.app.features.home.AvatarRenderer
|
||||
import im.vector.app.features.home.room.list.RoomListListener
|
||||
import org.matrix.android.sdk.api.session.room.model.RoomSummary
|
||||
@ -43,6 +45,12 @@ class RecentRoomCarouselController @Inject constructor(
|
||||
resources.displayMetrics
|
||||
).toInt()
|
||||
|
||||
private val topPadding = TypedValue.applyDimension(
|
||||
TypedValue.COMPLEX_UNIT_DIP,
|
||||
12f,
|
||||
resources.displayMetrics
|
||||
).toInt()
|
||||
|
||||
private val itemSpacing = TypedValue.applyDimension(
|
||||
TypedValue.COMPLEX_UNIT_DIP,
|
||||
24f,
|
||||
@ -61,11 +69,15 @@ class RecentRoomCarouselController @Inject constructor(
|
||||
id("recents_carousel")
|
||||
padding(Carousel.Padding(
|
||||
host.hPadding,
|
||||
0,
|
||||
host.topPadding,
|
||||
host.hPadding,
|
||||
0,
|
||||
host.itemSpacing)
|
||||
)
|
||||
onBind { _, view, _ ->
|
||||
val colorSurface = MaterialColors.getColor(view, R.attr.vctr_toolbar_background)
|
||||
view.setBackgroundColor(colorSurface)
|
||||
}
|
||||
withModelsFrom(data) { roomSummary ->
|
||||
val onClick = host.listener?.let { it::onRoomClicked }
|
||||
val onLongClick = host.listener?.let { it::onRoomLongClicked }
|
||||
|
@ -50,7 +50,7 @@
|
||||
android:layout_height="1dp"
|
||||
android:layout_marginStart="22dp"
|
||||
android:layout_marginEnd="16dp"
|
||||
android:background="?vctr_list_separator_system"
|
||||
android:background="?vctr_list_separator"
|
||||
app:layout_constraintBottom_toBottomOf="parent"
|
||||
app:layout_constraintEnd_toEndOf="parent"
|
||||
app:layout_constraintStart_toStartOf="parent" />
|
||||
|
@ -5,7 +5,7 @@
|
||||
android:id="@+id/recentRoot"
|
||||
android:layout_width="60dp"
|
||||
android:layout_height="wrap_content"
|
||||
android:background="?android:colorBackground"
|
||||
android:background="?vctr_toolbar_background"
|
||||
android:clickable="true"
|
||||
android:focusable="true"
|
||||
android:foreground="?attr/selectableItemBackground"
|
||||
|
Loading…
Reference in New Issue
Block a user