change: rename and reorder RoomPermissionsSection to match design
This commit is contained in:
@@ -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,
|
||||
|
||||
Reference in New Issue
Block a user