change: rename and reorder RoomPermissionsSection to match design
This commit is contained in:
parent
b04c1ea893
commit
c06e397d8a
5 changed files with 6 additions and 5 deletions
|
|
@ -45,7 +45,7 @@ class ChangeRoomPermissionsPresenter(
|
|||
RoomPermissionType.SEND_EVENTS,
|
||||
RoomPermissionType.REDACT_EVENTS,
|
||||
)
|
||||
RoomPermissionsSection.MembershipModeration -> persistentListOf(
|
||||
RoomPermissionsSection.ManageMembers -> persistentListOf(
|
||||
RoomPermissionType.INVITE,
|
||||
RoomPermissionType.KICK,
|
||||
RoomPermissionType.BAN,
|
||||
|
|
@ -59,7 +59,7 @@ class ChangeRoomPermissionsPresenter(
|
|||
private fun RoomPermissionsSection.shouldShow(isSpace: Boolean): Boolean {
|
||||
return when (this) {
|
||||
RoomPermissionsSection.EditDetails -> true
|
||||
RoomPermissionsSection.MembershipModeration -> true
|
||||
RoomPermissionsSection.ManageMembers -> true
|
||||
RoomPermissionsSection.MessagesAndContent -> !isSpace
|
||||
RoomPermissionsSection.ManageSpace -> isSpace
|
||||
}
|
||||
|
|
|
|||
|
|
@ -50,9 +50,9 @@ data class ChangeRoomPermissionsState(
|
|||
}
|
||||
|
||||
enum class RoomPermissionsSection {
|
||||
ManageMembers,
|
||||
EditDetails,
|
||||
MessagesAndContent,
|
||||
MembershipModeration,
|
||||
ManageSpace
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -26,6 +26,7 @@ class ChangeRoomPermissionsStateProvider : PreviewParameterProvider<ChangeRoomPe
|
|||
saveAction = AsyncAction.Failure(IllegalStateException("Failed to save changes"))
|
||||
),
|
||||
aChangeRoomPermissionsState(hasChanges = true, saveAction = AsyncAction.ConfirmingCancellation),
|
||||
aChangeRoomPermissionsState(itemsBySection = ChangeRoomPermissionsPresenter.buildItems(isSpace = true)),
|
||||
)
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -112,7 +112,7 @@ fun ChangeRoomPermissionsView(
|
|||
private fun titleForSection(section: RoomPermissionsSection): String = when (section) {
|
||||
RoomPermissionsSection.EditDetails -> stringResource(R.string.screen_room_change_permissions_room_details)
|
||||
RoomPermissionsSection.MessagesAndContent -> stringResource(R.string.screen_room_change_permissions_messages_and_content)
|
||||
RoomPermissionsSection.MembershipModeration -> stringResource(R.string.screen_room_change_permissions_member_moderation)
|
||||
RoomPermissionsSection.ManageMembers -> stringResource(R.string.screen_room_change_permissions_member_moderation)
|
||||
RoomPermissionsSection.ManageSpace -> stringResource(R.string.screen_room_change_permissions_manage_space)
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -62,7 +62,7 @@ class ChangeRoomPermissionsPresenterTest {
|
|||
RoomPermissionType.SEND_EVENTS,
|
||||
RoomPermissionType.REDACT_EVENTS,
|
||||
)
|
||||
assertThat(itemsBySection[RoomPermissionsSection.MembershipModeration]).containsExactly(
|
||||
assertThat(itemsBySection[RoomPermissionsSection.ManageMembers]).containsExactly(
|
||||
RoomPermissionType.INVITE,
|
||||
RoomPermissionType.KICK,
|
||||
RoomPermissionType.BAN,
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue