From c22c9c917bb465d9e44324e947af0dac5c5a44ee Mon Sep 17 00:00:00 2001
From: fenn-cs <fenn25.fn@gmail.com>
Date: Mon, 4 Sep 2023 23:24:39 +0100
Subject: [PATCH] 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 <fenn25.fn@gmail.com>
---
 .../src/views/SharingDetailsTab.vue              | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/apps/files_sharing/src/views/SharingDetailsTab.vue b/apps/files_sharing/src/views/SharingDetailsTab.vue
index 175812a0ff2..6368bb20b9f 100644
--- 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() {
 
-- 
2.39.5