From ec652f6bde8f272763d8402f4e8ee25021432ee8 Mon Sep 17 00:00:00 2001 From: Joseph Doherty Date: Mon, 10 Aug 2026 16:32:47 -0400 Subject: [PATCH] ui(adminui): group driver/acl shared button pairs, adopt notice-alert + tree-twist MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sweep task 5 (Shared/Drivers + Acls): - btn-group the Save/Cancel pair in DriverFormShell (all 13 driver forms), CollectionEditor foot + per-row Edit/Delete, DriverConfigModal / DeviceModal / DriverTagPicker / AclScopePickerModal feet, DriverStatusPanel Reconnect+Restart and its inline restart confirm, and the Refresh+Close browse toolbars on the FOCAS/AbCip/TwinCAT pickers. - Replace the no-op inline border-color:var(--alert) tint with .notice-alert in DriverFormShell, DriverConfigModal, DeviceModal, MqttDriverForm, MTConnectDriverForm, MqttDeviceForm. - Replace inline style="width:18px" expander sizing with .tree-twist in DriverBrowseTree and AclScopeTree. Markup only — every @onclick, binding, disabled and arm->confirm flow is unchanged. --- .../Shared/Acls/AclScopePickerModal.razor | 8 ++- .../Components/Shared/Acls/AclScopeTree.razor | 4 +- .../Shared/Drivers/CollectionEditor.razor | 12 ++-- .../Drivers/DeviceForms/MqttDeviceForm.razor | 2 +- .../Shared/Drivers/DeviceModal.razor | 22 +++--- .../Shared/Drivers/DriverBrowseTree.razor | 6 +- .../Shared/Drivers/DriverConfigModal.razor | 22 +++--- .../Shared/Drivers/DriverFormShell.razor | 14 ++-- .../Shared/Drivers/DriverStatusPanel.razor | 68 ++++++++++--------- .../Shared/Drivers/DriverTagPicker.razor | 12 ++-- .../Drivers/Forms/MTConnectDriverForm.razor | 2 +- .../Shared/Drivers/Forms/MqttDriverForm.razor | 2 +- .../Pickers/AbCipAddressPickerBody.razor | 18 ++--- .../Pickers/FOCASAddressPickerBody.razor | 18 ++--- .../Pickers/TwinCATAddressPickerBody.razor | 18 ++--- 15 files changed, 125 insertions(+), 103 deletions(-) diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Acls/AclScopePickerModal.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Acls/AclScopePickerModal.razor index 2f1c898c..2f735729 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Acls/AclScopePickerModal.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Acls/AclScopePickerModal.razor @@ -92,9 +92,11 @@ }
- - +
+ + +
diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Acls/AclScopeTree.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Acls/AclScopeTree.razor index f48e4c8b..da36d834 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Acls/AclScopeTree.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Acls/AclScopeTree.razor @@ -36,14 +36,14 @@
@if (hasExpander) { - } else { - + } @if (node.Selectable) diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/CollectionEditor.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/CollectionEditor.razor index 9905b11b..3b62a1fd 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/CollectionEditor.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/CollectionEditor.razor @@ -26,8 +26,10 @@ @RowTemplate(Items[idx]) - - +
+ + +
} @@ -55,8 +57,10 @@ }
diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DeviceForms/MqttDeviceForm.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DeviceForms/MqttDeviceForm.razor index c44627be..60aa74f6 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DeviceForms/MqttDeviceForm.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DeviceForms/MqttDeviceForm.razor @@ -20,7 +20,7 @@ @if (_model.LegacyConnectionKeys.Count > 0) { -
+
Legacy connection keys on this device

diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DeviceModal.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DeviceModal.razor index 52db7687..5363f91e 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DeviceModal.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DeviceModal.razor @@ -21,7 +21,7 @@

diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverBrowseTree.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverBrowseTree.razor index 3b5124ad..e0da9035 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverBrowseTree.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverBrowseTree.razor @@ -132,14 +132,14 @@ diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverFormShell.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverFormShell.razor index aae54a47..988bfa69 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverFormShell.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverFormShell.razor @@ -6,15 +6,17 @@ @if (!string.IsNullOrWhiteSpace(Error)) { -
@Error
+
@Error
}
- - Cancel +
+ + Cancel +
@if (OnDelete.HasValue) { +
+ - + +
@if (_opResultMessage is not null) { @@ -176,9 +178,9 @@ Restart driver @DriverInstanceId?
This briefly interrupts active subscriptions and clears in-memory state.

-
- - +
+ +
} diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverTagPicker.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverTagPicker.razor index b5922e43..a5a26f07 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverTagPicker.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/DriverTagPicker.razor @@ -20,11 +20,13 @@ { @CurrentAddress } - - +
+ + +
diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Forms/MTConnectDriverForm.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Forms/MTConnectDriverForm.razor index 5da6e6c6..9b2d95d6 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Forms/MTConnectDriverForm.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Forms/MTConnectDriverForm.razor @@ -141,7 +141,7 @@ @if (_form.Validate() is { } error) { -
@error
+
@error
} diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Forms/MqttDriverForm.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Forms/MqttDriverForm.razor index bef3d0d3..63302485 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Forms/MqttDriverForm.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Forms/MqttDriverForm.razor @@ -289,7 +289,7 @@ @if (_validationError is not null) { -
@_validationError
+
@_validationError
} diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/AbCipAddressPickerBody.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/AbCipAddressPickerBody.razor index 6ac0f423..9c70b18c 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/AbCipAddressPickerBody.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/AbCipAddressPickerBody.razor @@ -46,14 +46,16 @@ else { Browser open - - +
+ + +
} @if (_openError is not null) { @TruncatedError() } diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/FOCASAddressPickerBody.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/FOCASAddressPickerBody.razor index 5c0a17be..66d56405 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/FOCASAddressPickerBody.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/FOCASAddressPickerBody.razor @@ -74,14 +74,16 @@ else { Browser open - - +
+ + +
} @if (_openError is not null) { @TruncatedError() } diff --git a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/TwinCATAddressPickerBody.razor b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/TwinCATAddressPickerBody.razor index 5c4152c3..22875bdc 100644 --- a/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/TwinCATAddressPickerBody.razor +++ b/src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Drivers/Pickers/TwinCATAddressPickerBody.razor @@ -35,14 +35,16 @@ else { Browser open - - +
+ + +
} @if (_openError is not null) { @TruncatedError() }