Sulkta-OSS/element-x-ada
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

RoomListFilters : rename empty

Browse source
This commit is contained in:
ganfra 2024-02-21 14:48:01 +01:00
parent 5868a46e82
commit 95ca770122
3 changed files with 8 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

2
features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/filters/RoomListFiltersState.kt
View file

@ -24,5 +24,5 @@ data class RoomListFiltersState(
val isFeatureEnabled: Boolean,
val eventSink: (RoomListFiltersEvents) -> Unit,
) {
val showClearFilterButton = selectedFilters.isNotEmpty()
val hasAnyFilterSelected = selectedFilters.isNotEmpty()
}

4
features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/filters/RoomListFiltersView.kt
View file

@ -64,12 +64,12 @@ fun RoomListFiltersView(
state.eventSink(RoomListFiltersEvents.ToggleFilter(filter))
}
val startPadding = if (state.showClearFilterButton) 4.dp else 16.dp
val startPadding = if (state.hasAnyFilterSelected) 4.dp else 16.dp
Row(
modifier = modifier.padding(start = startPadding, end = 16.dp),
verticalAlignment = Alignment.CenterVertically,
) {
AnimatedVisibility(visible = state.showClearFilterButton) {
AnimatedVisibility(visible = state.hasAnyFilterSelected) {
RoomListClearFiltersButton(onClick = ::onClearFiltersClicked)
}
val lazyListState = rememberLazyListState()

10
features/roomlist/impl/src/test/kotlin/io/element/android/features/roomlist/impl/filters/RoomListFiltersPresenterTests.kt
View file

@ -38,7 +38,7 @@ class RoomListFiltersPresenterTests {
}.test {
awaitItem().let { state ->
assertThat(state.selectedFilters).isEmpty()
assertThat(state.showClearFilterButton).isFalse()
assertThat(state.hasAnyFilterSelected).isFalse()
assertThat(state.unselectedFilters).containsExactly(
RoomListFilter.Rooms,
RoomListFilter.People,
@ -62,7 +62,7 @@ class RoomListFiltersPresenterTests {
awaitLastSequentialItem().let { state ->
assertThat(state.selectedFilters).containsExactly(RoomListFilter.Rooms)
assertThat(state.showClearFilterButton).isTrue()
assertThat(state.hasAnyFilterSelected).isTrue()
assertThat(state.unselectedFilters).containsExactly(
RoomListFilter.Unread,
RoomListFilter.Favourites,
@ -78,7 +78,7 @@ class RoomListFiltersPresenterTests {
awaitLastSequentialItem().let { state ->
assertThat(state.selectedFilters).isEmpty()
assertThat(state.showClearFilterButton).isFalse()
assertThat(state.hasAnyFilterSelected).isFalse()
assertThat(state.unselectedFilters).containsExactly(
RoomListFilter.Rooms,
RoomListFilter.People,
@ -103,12 +103,12 @@ class RoomListFiltersPresenterTests {
awaitItem().eventSink.invoke(RoomListFiltersEvents.ToggleFilter(RoomListFilter.Rooms))
awaitLastSequentialItem().let { state ->
assertThat(state.selectedFilters).isNotEmpty()
assertThat(state.showClearFilterButton).isTrue()
assertThat(state.hasAnyFilterSelected).isTrue()
state.eventSink.invoke(RoomListFiltersEvents.ClearSelectedFilters)
}
awaitLastSequentialItem().let { state ->
assertThat(state.selectedFilters).isEmpty()
assertThat(state.showClearFilterButton).isFalse()
assertThat(state.hasAnyFilterSelected).isFalse()
}
}
}