Skip to content

fix(ui): resolve UX audit issues across workflows #324

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
MrYuion merged 5 commits into from
Oct 21, 2026
Merged

fix(ui): resolve UX audit issues across workflows #324

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
245462c
fix: repair zone bulk-add, import links, edge keys and driver modules
MrYuion Oct 21, 2026
51e4bad
fix(ui): stop edit modals and forms dropping unsaved work
MrYuion Oct 21, 2026
1193c00
fix(ui): name delete targets and report failed list loads
MrYuion Oct 21, 2026
19ea2e1
fix(ui): fit tablet layouts, readable dark placeholders, closable menus
MrYuion Oct 21, 2026
1c5a392
fix(ui): name icon buttons, guard hotkeys and fix copy
MrYuion Oct 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 26 additions & 10 deletions public/assets/locale/en-AU.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,21 @@
"NONE": "None",
"CANCEL": "Cancel",
"CLOSE": "Close",
"MENU": "Menu",
"CLEAR_VALUE": "Clear",
"REMOVE": "Remove",
"VIEW": "View",
"MENU_EXPAND": "Expand menu",
"MENU_COLLAPSE": "Collapse menu",
"MORE_ACTIONS": "More actions",
"FILTER": "Filter",
"INCREASE": "Increase",
"DECREASE": "Decrease",
"DISCARD": "Discard",
"DISCARD_CHANGES": "Discard changes?",
"DISCARD_CHANGES_MSG": "You have unsaved changes. Close and lose them?",
"CONFIRM": "Confirm",
"DELETE": "Delete",
"ITEM_REMOVE": "Remove {{ item }}",
"FIELD_REQUIRED": "This field is required",
"DATE_EMPTY": "No date selected",
Expand All @@ -67,6 +81,7 @@
"COPY_LINK": "Copy Link",
"RETRY": "Retry",
"EDITOR_LOAD_ERROR": "Could not load the editor. Check your connection and retry.",
"LOAD_ERROR": "Failed to load data. Error: {{ error }}",
"IMAGES": "Images",
"NEW_ITEM": "New item",
"ITEM_ADD": "Add {{ item }}",
Expand Down Expand Up @@ -172,7 +187,7 @@
"GIT_PULL_LATEST": "Pull latest commit",
"COMMIT_HASH_COPIED": "Commit hash copied to clipboard",
"DESCRIPTION_EMPTY": "No description",
"INVALID_FIELDS": "Some fields are invalid. [{{ field_list }}]",
"INVALID_FIELDS": "Check these fields: {{ field_list }}",
"BULK_ADD": "Bulk add {{ type }}",
"BULK_DOWNLOAD": "Download Template",
"BULK_DROP_MSG": "Select or drop TSV file to begin",
Expand Down Expand Up @@ -244,7 +259,7 @@
"ADD": "Add system",
"EDIT": "Edit system",
"DELETE": "Delete system",
"DELETE_MSG": "<p>Are you sure you want delete this system(<strong>{{ name }}</strong>)?</p><p>Deleting this will <strong>immediately</strong> delete modules that are not in another system</p>",
"DELETE_MSG": "<p>Delete the system <strong>{{ name }}</strong>?</p><p>Modules that are not in another system are deleted <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting system...",
"DELETE_SUCCESS": "Successfully deleted system.",
"DELETE_ERROR": "Failed to delete system. Error: {{ error }}",
Expand Down Expand Up @@ -332,7 +347,7 @@
"SEARCH": "Search for modules...",
"ADD": "Add module",
"DELETE": "Delete module",
"DELETE_MSG": "<p>Are you sure you want delete this module?</p><p>Deleting this will module <strong>immediately</strong> remove it from any system associated with it</p>",
"DELETE_MSG": "<p>Delete the module <strong>{{ name }}</strong>?</p><p>It is removed <strong>immediately</strong> from every system that uses it.</p>",
"DELETE_LOADING": "Deleting module...",
"DELETE_SUCCESS": "Successfully deleted module.",
"DELETE_ERROR": "Failed to delete module. Error: {{ error }}",
Expand Down Expand Up @@ -389,7 +404,7 @@
"ADD": "Add zone",
"EDIT": "Edit zone",
"DELETE": "Delete zone",
"DELETE_MSG": "<p>Are you sure you want delete this zone?</p><p>Zones beneath it, its triggers, metadata and settings are removed <strong>immediately</strong>. Systems in this zone are kept unless you also remove associated resources.</p>",
"DELETE_MSG": "<p>Delete the zone <strong>{{ name }}</strong>?</p><p>Zones beneath it, its triggers, metadata and settings are removed <strong>immediately</strong>. Systems in this zone are kept unless you also remove associated resources.</p>",
"DELETE_LOADING": "Deleting zone...",
"DELETE_SUCCESS": "Successfully deleted zone.",
"DELETE_ERROR": "Failed to delete zone. Error: {{ error }}",
Expand Down Expand Up @@ -463,7 +478,7 @@
"ADD": "Add driver",
"EDIT": "Edit driver",
"DELETE": "Delete driver",
"DELETE_MSG": "<p>Are you sure you want delete this driver?</p>",
"DELETE_MSG": "<p>Delete the driver <strong>{{ name }}</strong>?</p>",
"DELETE_LOADING": "Deleting driver...",
"DELETE_SUCCESS": "Successfully deleted driver.",
"DELETE_ERROR": "Failed to delete driver. Error: {{ error }}",
Expand All @@ -476,6 +491,7 @@
"UPDATE": "Update Drivers",
"UPDATE_COUNT": "{{ count }} updates available",
"UPDATE_SELECTED": "Update selected drivers",
"UPDATING": "Updating drivers...",
"NO_UPDATES": "No drivers require updating.",
"TAB_ABOUT": "About",
"TAB_DOCS": "Readme",
Expand Down Expand Up @@ -518,7 +534,7 @@
"DELETE": "Delete repository",
"REMOVE": "Remove repository",
"NEW": "New repository",
"DELETE_MSG": "<p>Deleting this repository will <strong>immediately</strong> remove assoicated drivers and modules</p>",
"DELETE_MSG": "<p>Delete the repository <strong>{{ name }}</strong>?</p><p>Its drivers and their modules are removed <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting repository...",
"DELETE_SUCCESS": "Successfully deleted repository.",
"DELETE_ERROR": "Failed to delete repository. Error: {{ error }}",
Expand Down Expand Up @@ -567,7 +583,7 @@
"DELETE": "Delete trigger",
"DELETE_INSTANCE": "Delete instance",
"REMOVE": "Remove trigger",
"DELETE_MSG": "<p>Are you sure you want delete this trigger?</p><p>Deleting this trigger will <strong>immediately</strong> remove it from all associated systems and zones</p>",
"DELETE_MSG": "<p>Delete the trigger <strong>{{ name }}</strong>?</p><p>It is removed <strong>immediately</strong> from all systems and zones that use it.</p>",
"DELETE_LOADING": "Deleting trigger...",
"DELETE_SUCCESS": "Successfully deleted trigger.",
"DELETE_ERROR": "Failed to delete trigger. Error: {{ error }}",
Expand Down Expand Up @@ -684,7 +700,7 @@
"BULK": "Bulk add users",
"REMOVE": "Remove user",
"DELETE": "Delete user",
"DELETE_MSG": "<p>Are you sure you want delete this user?</p><p>The user will be removed from the system within 24 hours</p>",
"DELETE_MSG": "<p>Delete the user <strong>{{ name }}</strong>?</p><p>The user is removed within 24 hours.</p>",
"DELETE_LOADING": "Deleting user...",
"DELETE_SUCCESS": "Successfully deleted user.",
"DELETE_ERROR": "Failed to delete user. Error: {{ error }}",
Expand Down Expand Up @@ -748,7 +764,7 @@
"NEW": "New group",
"REMOVE": "Remove group",
"DELETE": "Delete group",
"DELETE_MSG": "<p>Are you sure you want delete this group?</p><p>The group will be deleted <strong>immediately.</strong></p>",
"DELETE_MSG": "<p>Delete the group <strong>{{ name }}</strong>?</p><p>The group is deleted <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting group...",
"DELETE_SUCCESS": "Successfully deleted group.",
"DELETE_ERROR": "Failed to delete group. Error: {{ error }}",
Expand Down Expand Up @@ -842,7 +858,7 @@
"BULK": "Bulk add domains",
"REMOVE": "Remove domain",
"DELETE": "Delete domain",
"DELETE_MSG": "<p>Are you sure you want delete this domain?</p><p>Its users, auth sources and groups are removed <strong>immediately</strong>.</p>",
"DELETE_MSG": "<p>Delete the domain <strong>{{ name }}</strong>?</p><p>Its users, auth sources and groups are removed <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting domain...",
"DELETE_SUCCESS": "Successfully deleted domain.",
"DELETE_ERROR": "Failed to delete domain. Error: {{ error }}",
Expand Down
20 changes: 10 additions & 10 deletions public/assets/locale/en-GB.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@
"METADATA_REMOVE": "Remove Metadata block",
"METADATA_EMPTY": "No Metadata blocks found",
"METADATA_SAVE": "Successfully saved metadata",
"TOTAL_ITEMS": "{{ count }} items(s)",
"TOTAL_ITEMS": "{{ count }} item(s)",
"ITEM_ERROR": "Failed to save item. Error: {{ error }}",
"ITEM_SAVE": "Successfully saved item",
"UPDATE_AVAILABLE": "Update Available",
Expand Down Expand Up @@ -152,7 +152,7 @@
"GIT_COMMIT": "Commit hash",
"GIT_PULL_LATEST": "Pull latest commit",
"DESCRIPTION_EMPTY": "No description",
"INVALID_FIELDS": "Some fields are invalid. [{{ field_list }}]",
"INVALID_FIELDS": "Check these fields: {{ field_list }}",
"BULK_ADD": "Bulk add {{ type }}",
"BULK_DOWNLOAD": "Download Template",
"BULK_DROP_MSG": "Select or drop TSV file to begin",
Expand Down Expand Up @@ -213,7 +213,7 @@
"ADD": "Add system",
"EDIT": "Edit system",
"DELETE": "Delete system",
"DELETE_MSG": "<p>Are you sure you want delete this system(<strong>{{ name }}</strong>)?</p><p>Deleting this will <strong>immediately</strong> delete modules that are not in another system</p>",
"DELETE_MSG": "<p>Delete the system <strong>{{ name }}</strong>?</p><p>Modules that are not in another system are deleted <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting system...",
"DELETE_SUCCESS": "Successfully deleted system.",
"DELETE_ERROR": "Failed to delete system. Error: {{ error }}",
Expand Down Expand Up @@ -292,7 +292,7 @@
"SEARCH": "Search for modules...",
"ADD": "Add module",
"DELETE": "Delete module",
"DELETE_MSG": "<p>Are you sure you want delete this module?</p><p>Deleting this will module <strong>immediately</strong> remove it from any system associated with it</p>",
"DELETE_MSG": "<p>Delete the module <strong>{{ name }}</strong>?</p><p>It is removed <strong>immediately</strong> from every system that uses it.</p>",
"DELETE_LOADING": "Deleting module...",
"DELETE_SUCCESS": "Successfully deleted module.",
"DELETE_ERROR": "Failed to delete module. Error: {{ error }}",
Expand Down Expand Up @@ -348,7 +348,7 @@
"ADD": "Add zone",
"EDIT": "Edit zone",
"DELETE": "Delete zone",
"DELETE_MSG": "<p>Are you sure you want delete this zone?</p><p>Zones beneath it, its triggers, metadata and settings are removed <strong>immediately</strong>. Systems in this zone are kept unless you also remove associated resources.</p>",
"DELETE_MSG": "<p>Delete the zone <strong>{{ name }}</strong>?</p><p>Zones beneath it, its triggers, metadata and settings are removed <strong>immediately</strong>. Systems in this zone are kept unless you also remove associated resources.</p>",
"DELETE_LOADING": "Deleting zone...",
"DELETE_SUCCESS": "Successfully deleted zone.",
"DELETE_ERROR": "Failed to delete zone. Error: {{ error }}",
Expand Down Expand Up @@ -403,7 +403,7 @@
"ADD": "Add driver",
"EDIT": "Edit driver",
"DELETE": "Delete driver",
"DELETE_MSG": "<p>Are you sure you want delete this driver?</p>",
"DELETE_MSG": "<p>Delete the driver <strong>{{ name }}</strong>?</p>",
"DELETE_LOADING": "Deleting driver...",
"DELETE_SUCCESS": "Successfully deleted driver.",
"DELETE_ERROR": "Failed to delete driver. Error: {{ error }}",
Expand Down Expand Up @@ -457,7 +457,7 @@
"DELETE": "Delete repository",
"REMOVE": "Remove repository",
"NEW": "New repository",
"DELETE_MSG": "<p>Deleting this repository will <strong>immediately</strong> remove assoicated drivers and modules</p>",
"DELETE_MSG": "<p>Delete the repository <strong>{{ name }}</strong>?</p><p>Its drivers and their modules are removed <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting repository...",
"DELETE_SUCCESS": "Successfully deleted repository.",
"DELETE_ERROR": "Failed to delete repository. Error: {{ error }}",
Expand Down Expand Up @@ -500,7 +500,7 @@
"DELETE": "Delete trigger",
"DELETE_INSTANCE": "Delete instance",
"REMOVE": "Remove trigger",
"DELETE_MSG": "<p>Are you sure you want delete this trigger?</p><p>Deleting this trigger will <strong>immediately</strong> remove it from all associated systems and zones</p>",
"DELETE_MSG": "<p>Delete the trigger <strong>{{ name }}</strong>?</p><p>It is removed <strong>immediately</strong> from all systems and zones that use it.</p>",
"DELETE_LOADING": "Deleting trigger...",
"DELETE_SUCCESS": "Successfully deleted trigger.",
"DELETE_ERROR": "Failed to delete trigger. Error: {{ error }}",
Expand Down Expand Up @@ -607,7 +607,7 @@
"NEW": "New user",
"BULK": "Bulk add users",
"REMOVE": "Remove user",
"DELETE_MSG": "<p>Are you sure you want delete this user?</p><p>The user will be removed from the system within 24 hours</p>",
"DELETE_MSG": "<p>Delete the user <strong>{{ name }}</strong>?</p><p>The user is removed within 24 hours.</p>",
"DELETE_LOADING": "Deleting user...",
"DELETE_SUCCESS": "Successfully deleted user.",
"DELETE_ERROR": "Failed to delete user. Error: {{ error }}",
Expand Down Expand Up @@ -651,7 +651,7 @@
"BULK": "Bulk add domains",
"REMOVE": "Remove domain",
"DELETE": "Delete domain",
"DELETE_MSG": "<p>Are you sure you want delete this domain?</p><p>Its users, auth sources and groups are removed <strong>immediately</strong>.</p>",
"DELETE_MSG": "<p>Delete the domain <strong>{{ name }}</strong>?</p><p>Its users, auth sources and groups are removed <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting domain...",
"DELETE_SUCCESS": "Successfully deleted domain.",
"DELETE_ERROR": "Failed to delete domain. Error: {{ error }}",
Expand Down
20 changes: 10 additions & 10 deletions public/assets/locale/en-US.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@
"METADATA_REMOVE": "Remove Metadata block",
"METADATA_EMPTY": "No Metadata blocks found",
"METADATA_SAVE": "Successfully saved metadata",
"TOTAL_ITEMS": "{{ count }} items(s)",
"TOTAL_ITEMS": "{{ count }} item(s)",
"ITEM_ERROR": "Failed to save item. Error: {{ error }}",
"ITEM_SAVE": "Successfully saved item",
"UPDATE_AVAILABLE": "Update Available",
Expand Down Expand Up @@ -149,7 +149,7 @@
"GIT_COMMIT": "Commit hash",
"GIT_PULL_LATEST": "Pull latest commit",
"DESCRIPTION_EMPTY": "No description",
"INVALID_FIELDS": "Some fields are invalid. [{{ field_list }}]",
"INVALID_FIELDS": "Check these fields: {{ field_list }}",
"BULK_ADD": "Bulk add {{ type }}",
"BULK_DOWNLOAD": "Download Template",
"BULK_DROP_MSG": "Select or drop TSV file to begin",
Expand Down Expand Up @@ -210,7 +210,7 @@
"ADD": "Add system",
"EDIT": "Edit system",
"DELETE": "Delete system",
"DELETE_MSG": "<p>Are you sure you want delete this system(<strong>{{ name }}</strong>)?</p><p>Deleting this will <strong>immediately</strong> delete modules that are not in another system</p>",
"DELETE_MSG": "<p>Delete the system <strong>{{ name }}</strong>?</p><p>Modules that are not in another system are deleted <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting system...",
"DELETE_SUCCESS": "Successfully deleted system.",
"DELETE_ERROR": "Failed to delete system. Error: {{ error }}",
Expand Down Expand Up @@ -289,7 +289,7 @@
"SEARCH": "Search for modules...",
"ADD": "Add module",
"DELETE": "Delete module",
"DELETE_MSG": "<p>Are you sure you want delete this module?</p><p>Deleting this will module <strong>immediately</strong> remove it from any system associated with it</p>",
"DELETE_MSG": "<p>Delete the module <strong>{{ name }}</strong>?</p><p>It is removed <strong>immediately</strong> from every system that uses it.</p>",
"DELETE_LOADING": "Deleting module...",
"DELETE_SUCCESS": "Successfully deleted module.",
"DELETE_ERROR": "Failed to delete module. Error: {{ error }}",
Expand Down Expand Up @@ -345,7 +345,7 @@
"ADD": "Add zone",
"EDIT": "Edit zone",
"DELETE": "Delete zone",
"DELETE_MSG": "<p>Are you sure you want delete this zone?</p><p>Zones beneath it, its triggers, metadata and settings are removed <strong>immediately</strong>. Systems in this zone are kept unless you also remove associated resources.</p>",
"DELETE_MSG": "<p>Delete the zone <strong>{{ name }}</strong>?</p><p>Zones beneath it, its triggers, metadata and settings are removed <strong>immediately</strong>. Systems in this zone are kept unless you also remove associated resources.</p>",
"DELETE_LOADING": "Deleting zone...",
"DELETE_SUCCESS": "Successfully deleted zone.",
"DELETE_ERROR": "Failed to delete zone. Error: {{ error }}",
Expand Down Expand Up @@ -400,7 +400,7 @@
"ADD": "Add driver",
"EDIT": "Edit driver",
"DELETE": "Delete driver",
"DELETE_MSG": "<p>Are you sure you want delete this driver?</p>",
"DELETE_MSG": "<p>Delete the driver <strong>{{ name }}</strong>?</p>",
"DELETE_LOADING": "Deleting driver...",
"DELETE_SUCCESS": "Successfully deleted driver.",
"DELETE_ERROR": "Failed to delete driver. Error: {{ error }}",
Expand Down Expand Up @@ -454,7 +454,7 @@
"DELETE": "Delete repository",
"REMOVE": "Remove repository",
"NEW": "New repository",
"DELETE_MSG": "<p>Deleting this repository will <strong>immediately</strong> remove assoicated drivers and modules</p>",
"DELETE_MSG": "<p>Delete the repository <strong>{{ name }}</strong>?</p><p>Its drivers and their modules are removed <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting repository...",
"DELETE_SUCCESS": "Successfully deleted repository.",
"DELETE_ERROR": "Failed to delete repository. Error: {{ error }}",
Expand Down Expand Up @@ -497,7 +497,7 @@
"DELETE": "Delete trigger",
"DELETE_INSTANCE": "Delete instance",
"REMOVE": "Remove trigger",
"DELETE_MSG": "<p>Are you sure you want delete this trigger?</p><p>Deleting this trigger will <strong>immediately</strong> remove it from all associated systems and zones</p>",
"DELETE_MSG": "<p>Delete the trigger <strong>{{ name }}</strong>?</p><p>It is removed <strong>immediately</strong> from all systems and zones that use it.</p>",
"DELETE_LOADING": "Deleting trigger...",
"DELETE_SUCCESS": "Successfully deleted trigger.",
"DELETE_ERROR": "Failed to delete trigger. Error: {{ error }}",
Expand Down Expand Up @@ -604,7 +604,7 @@
"NEW": "New user",
"BULK": "Bulk add users",
"REMOVE": "Remove user",
"DELETE_MSG": "<p>Are you sure you want delete this user?</p><p>The user will be removed from the system within 24 hours</p>",
"DELETE_MSG": "<p>Delete the user <strong>{{ name }}</strong>?</p><p>The user is removed within 24 hours.</p>",
"DELETE_LOADING": "Deleting user...",
"DELETE_SUCCESS": "Successfully deleted user.",
"DELETE_ERROR": "Failed to delete user. Error: {{ error }}",
Expand Down Expand Up @@ -648,7 +648,7 @@
"BULK": "Bulk add domains",
"REMOVE": "Remove domain",
"DELETE": "Delete domain",
"DELETE_MSG": "<p>Are you sure you want delete this domain?</p><p>Its users, auth sources and groups are removed <strong>immediately</strong>.</p>",
"DELETE_MSG": "<p>Delete the domain <strong>{{ name }}</strong>?</p><p>Its users, auth sources and groups are removed <strong>immediately</strong>.</p>",
"DELETE_LOADING": "Deleting domain...",
"DELETE_SUCCESS": "Successfully deleted domain.",
"DELETE_ERROR": "Failed to delete domain. Error: {{ error }}",
Expand Down
3 changes: 3 additions & 0 deletions src/app/admin/admin.component.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import { DebugOutputComponent } from '../ui/debug-output.component';
import { IconComponent } from '../ui/icon.component';
import { ItemTab } from '../ui/item-tablist.component';
import { SidebarMenuComponent } from '../ui/sidebar-menu.component';
import { TranslatePipe } from '../ui/translate.pipe';

@Component({
selector: 'app-engine',
Expand Down Expand Up @@ -51,6 +52,7 @@ import { SidebarMenuComponent } from '../ui/sidebar-menu.component';
matRipple
class="absolute top-2 left-4 z-40 mr-2 sm:hidden"
(click)="open_menu = true"
[attr.aria-label]="'COMMON.MENU' | translate"
>
<icon>menu</icon>
</button>
Expand Down Expand Up @@ -81,6 +83,7 @@ import { SidebarMenuComponent } from '../ui/sidebar-menu.component';
`,
],
imports: [
TranslatePipe,
DebugOutputComponent,
IconComponent,
MatRippleModule,
Expand Down
Loading
Loading