Final brushing

Author: 0x5bfa

src/Files.App/Filesystem/Security/AccessControlEntry.cs

@@ -30,6 +30,13 @@ public AccessControlType AccessControlType
 			}
 		}
 
+		public string AccessControlTypeHumanized
+			=> AccessControlType switch
+			{
+				AccessControlType.Allow => "Allow",
+				_ => "Deny" // AccessControlType.Deny
+			};
+
 		public string AccessControlTypeGlyph
 			=> AccessControlType switch
 			{

src/Files.App/Strings/en-US/Resources.resw

@@ -1608,7 +1608,7 @@
   <data name="Allow" xml:space="preserve">
     <value>Allow</value>
   </data>
-  <data name="SecurityDenyLabel.Text" xml:space="preserve">
+  <data name="Deny" xml:space="preserve">
     <value>Deny</value>
   </data>
   <data name="SecurityFullControlLabel.Text" xml:space="preserve">
@@ -1680,10 +1680,10 @@
   <data name="Access" xml:space="preserve">
     <value>Access</value>
   </data>
-  <data name="SecurityAdvancedAppliesToLabel.Text" xml:space="preserve">
+  <data name="SecurityAdvancedAppliesTo" xml:space="preserve">
     <value>Applies to</value>
   </data>
-  <data name="SecurityAdvancedPrincipal" xml:space="preserve">
+  <data name="Principal" xml:space="preserve">
     <value>Principal</value>
   </data>
   <data name="SecurityAdvancedFlagsFilesLabel" xml:space="preserve">
@@ -1695,10 +1695,10 @@
   <data name="SecurityAdvancedFlagsSubfoldersLabel" xml:space="preserve">
     <value>subfolders</value>
   </data>
-  <data name="SecurityAdvancedInheritedLabel.Text" xml:space="preserve">
+  <data name="Inherited" xml:space="preserve">
     <value>Inherited</value>
   </data>
-  <data name="SecurityAdvancedPermissionsLabel.Text" xml:space="preserve">
+  <data name="Permissions" xml:space="preserve">
     <value>Permissions</value>
   </data>
   <data name="SecurityAdvancedCannotReadProperties.Text" xml:space="preserve">
@@ -2670,4 +2670,7 @@
   <data name="ExitCompactOverlayDescription" xml:space="preserve">
     <value>Exit compact overlay</value>
   </data>
+  <data name="Change" xml:space="preserve">
+    <value>Change</value>
+  </data>
 </root>

src/Files.App/ViewModels/Properties/SecurityViewModel.cs

@@ -61,8 +61,14 @@ public AccessControlEntry SelectedAccessControlEntry
 			get => _selectedAccessControlEntry;
 			set
 			{
+				if (_selectedAccessControlEntry is not null)
+					_selectedAccessControlEntry.IsSelected = false;
+
 				if (SetProperty(ref _selectedAccessControlEntry, value))
+				{
+					value.IsSelected = true;
 					RemoveAccessControlEntryCommand.NotifyCanExecuteChanged();
+				}
 			}
 		}
 
@@ -91,16 +97,32 @@ public string DisableInheritanceOption
 		private bool _preserveInheritance;
 
 		private GridLength _columnType = new(64d);
-		public GridLength ColumnType { get => _columnType; set => SetProperty(ref _columnType, value); }
+		public GridLength ColumnType
+		{
+			get => _columnType;
+			set => SetProperty(ref _columnType, value);
+		}
 
 		private GridLength _columnPrincipal = new(200d);
-		public GridLength ColumnPrincipal { get => _columnPrincipal; set => SetProperty(ref _columnPrincipal, value); }
+		public GridLength ColumnPrincipal
+		{
+			get => _columnPrincipal;
+			set => SetProperty(ref _columnPrincipal, value);
+		}
 
 		private GridLength _columnAccess = new(160d);
-		public GridLength ColumnAccess { get => _columnAccess; set => SetProperty(ref _columnAccess, value); }
+		public GridLength ColumnAccess
+		{
+			get => _columnAccess;
+			set => SetProperty(ref _columnAccess, value);
+		}
 
 		private GridLength _columnInherited = new(70d);
-		public GridLength ColumnInherited { get => _columnInherited; set => SetProperty(ref _columnInherited, value); }
+		public GridLength ColumnInherited
+		{
+			get => _columnInherited;
+			set => SetProperty(ref _columnInherited, value);
+		}
 
 		public RelayCommand ChangeOwnerCommand { get; set; }
 		public RelayCommand AddAccessControlEntryCommand { get; set; }