Do not toggle custom permissions

Currently, swithcing between permissions in the sharing permissions details view would toggle the custom permissions view. We think, it's better for the custom permission option to simple expand the custom permissions view and never close it gain even if the user is switching between bundled permissions. The user can always collapse the view by using the "Advanced settings" toggle. Signed-off-by: fenn-cs <[email protected]>
nextcloud · Sep 6, 2023 · 6e479df · 6e479df
1 parent 387cf9e
commit 6e479df
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/apps/files_sharing/src/views/SharingDetailsTab.vue b/apps/files_sharing/src/views/SharingDetailsTab.vue
@@ -61,7 +61,7 @@
 					name="sharing_permission_radio"
 					type="radio"
 					button-variant-grouped="vertical"
-					@update:checked="toggleCustomPermissions">
+					@update:checked="expandCustomPermissions">
 					{{ t('files_sharing', 'Custom permissions') }}
 					<small>{{ t('files_sharing', customPermissionsList) }}</small>
 					<template #icon>
@@ -666,16 +666,14 @@ export default {
 				this.$set(this.share, 'hasDownloadPermission', isDownloadChecked)
 			}
 		},
-
-		toggleCustomPermissions(selectedPermission) {
-			if (this.sharingPermission === 'custom') {
+		expandCustomPermissions() {
+			if (!this.advancedSectionAccordionExpanded)	{
 				this.advancedSectionAccordionExpanded = true
-				this.setCustomPermissions = true
-			} else {
-				this.advancedSectionAccordionExpanded = false
-				this.revertSharingPermission = selectedPermission
-				this.setCustomPermissions = false
 			}
+			this.toggleCustomPermissions()
+		},
+		toggleCustomPermissions() {
+			this.setCustomPermissions = this.sharingPermission === 'custom'
 		},
 		initializeAttributes() {