From 0c8972f344f80265f66ef71e7f28f9cba4de91a4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9mai=20G=C3=A1bor?= <remaigabor@gmail.com>
Date: Mon, 17 Feb 2025 16:25:49 +0100
Subject: [PATCH] Feat mgmt fixes (#213)

* Fix role and group UI permissions

* remove magin on mgmt-UI
---
 src/components/managementservice/groups/Groups.tsx | 3 +--
 src/components/managementservice/role/Role.tsx     | 7 +++++--
 src/views/management/Management.tsx                | 2 +-
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/components/managementservice/groups/Groups.tsx b/src/components/managementservice/groups/Groups.tsx
index 2bfd3425..32331741 100644
--- a/src/components/managementservice/groups/Groups.tsx
+++ b/src/components/managementservice/groups/Groups.tsx
@@ -100,7 +100,7 @@ const GroupTable = () => {
 		setId(0);
 		setName('');
 		setDescription('');
-		setDescriptionDisabled(false);
+		setDescriptionDisabled(true);
 		// try to get current tenantId
 		// TODO
 		setTenantId(0);
@@ -209,7 +209,6 @@ const GroupTable = () => {
 						id="description"
 						label="description"
 						type="text"
-						required
 						fullWidth
 						disabled={descriptionDisabled}
 						onChange={handleDescriptionChange}
diff --git a/src/components/managementservice/role/Role.tsx b/src/components/managementservice/role/Role.tsx
index 31cea6b2..21188767 100644
--- a/src/components/managementservice/role/Role.tsx
+++ b/src/components/managementservice/role/Role.tsx
@@ -121,11 +121,12 @@ const RoleTable = () => {
 		setDescription('');
 		setTenantId(0);
 		setChecked(new Array(permissions.length).fill(false));
-
+		setCheckedDisabled(true);
 		setOpen(true);
 	};
 
 	const handleClickOpenNoreset = () => {
+		setCheckedDisabled(false);
 		setOpen(true);
 	};
 
@@ -226,6 +227,7 @@ const RoleTable = () => {
 	const [ permissions, setPermissions ] = React.useState(Array<Permissions>);
 
 	const [ checked, setChecked ] = React.useState(new Array(0).fill(true));
+	const [ checkedDisabled, setCheckedDisabled ] = React.useState(false);
 
 	const handleChange1 = (event: React.ChangeEvent<HTMLInputElement>) => {
 		setChecked(new Array(permissions.length).fill(event.target.checked));
@@ -243,6 +245,7 @@ const RoleTable = () => {
 		<Box sx={{ display: 'flex', flexDirection: 'column', ml: 3 }}>
 			{Object.entries(permissions).map(([ key, value ]) =>
 				<FormControlLabel
+					disabled={checkedDisabled}
 					key={`${key}uniqe`}
 					control={<Checkbox checked={checked[parseInt(key)]}
 						onChange={(event) => handleChangeMod(event, parseInt(key))
@@ -285,7 +288,6 @@ const RoleTable = () => {
 						id="description"
 						label="description"
 						type="text"
-						required
 						fullWidth
 						onChange={handleDescriptionChange}
 						value={description}
@@ -306,6 +308,7 @@ const RoleTable = () => {
 							label="All permissions"
 							control={
 								<Checkbox
+									disabled={checkedDisabled}
 									checked={checked.every((num) => num === true)}
 									indeterminate={checked.some((num) => num === true) && checked.some((num) => num === false)}
 									onChange={handleChange1}
diff --git a/src/views/management/Management.tsx b/src/views/management/Management.tsx
index f4f0f07b..1297a27d 100644
--- a/src/views/management/Management.tsx
+++ b/src/views/management/Management.tsx
@@ -221,7 +221,7 @@ export default function ManagementUI(/* props: Props */) {
 	);
 
 	return (
-		<Box sx={{ display: 'flex', flex: 1, marginRight: { sm: '300px' } }}>
+		<Box sx={{ display: 'flex', flex: 1, marginRight: { sm: '0px' } }}>
 			<CssBaseline />
 			<AppBar
 				position="fixed"