From d5613ee71891ffd98f541b8b0bb7e6a740894925 Mon Sep 17 00:00:00 2001 From: m5r Date: Sat, 29 Aug 2026 01:29:11 +0200 Subject: [PATCH] admin: pointer cursor on every button the shared button variants set no cursor, so only components with local one-off styles showed pointer. all nine variants and the six buttons styled outside them now carry pointer, with not-allowed when disabled. --- admin/src/features/activity/LiveActivity.tsx | 1 + admin/src/features/clients/NetworkAssignments.tsx | 1 + admin/src/features/configuration/SettingsForm.tsx | 1 + admin/src/features/overview/OverviewFrame.tsx | 1 + admin/src/features/pause/PauseControl.tsx | 1 + admin/src/lib/InlineError.tsx | 1 + admin/src/ui/ConfirmDialog.tsx | 1 + admin/src/ui/styles.ts | 9 +++++++++ 8 files changed, 16 insertions(+) diff --git a/admin/src/features/activity/LiveActivity.tsx b/admin/src/features/activity/LiveActivity.tsx index 36ebddf..5bf6102 100644 --- a/admin/src/features/activity/LiveActivity.tsx +++ b/admin/src/features/activity/LiveActivity.tsx @@ -88,6 +88,7 @@ const styles = stylex.create({ lineHeight: "1.25rem", }, dismiss: { + cursor: { default: "pointer", ":disabled": "not-allowed" }, borderStyle: "none", backgroundColor: "transparent", padding: 0, diff --git a/admin/src/features/clients/NetworkAssignments.tsx b/admin/src/features/clients/NetworkAssignments.tsx index 7e090f3..e5c1645 100644 --- a/admin/src/features/clients/NetworkAssignments.tsx +++ b/admin/src/features/clients/NetworkAssignments.tsx @@ -65,6 +65,7 @@ const styles = stylex.create({ width: "5rem", }, removeButton: { + cursor: { default: "pointer", ":disabled": "not-allowed" }, borderRadius: "0.25rem", borderWidth: 1, borderStyle: "solid", diff --git a/admin/src/features/configuration/SettingsForm.tsx b/admin/src/features/configuration/SettingsForm.tsx index d3da647..f8726c4 100644 --- a/admin/src/features/configuration/SettingsForm.tsx +++ b/admin/src/features/configuration/SettingsForm.tsx @@ -100,6 +100,7 @@ const styles = stylex.create({ gap: "0.75rem", }, save: { + cursor: { default: "pointer", ":disabled": "not-allowed" }, borderStyle: "none", borderRadius: "0.25rem", paddingInline: "1rem", diff --git a/admin/src/features/overview/OverviewFrame.tsx b/admin/src/features/overview/OverviewFrame.tsx index 2edf33e..ea8d227 100644 --- a/admin/src/features/overview/OverviewFrame.tsx +++ b/admin/src/features/overview/OverviewFrame.tsx @@ -38,6 +38,7 @@ const styles = stylex.create({ gap: "0.25rem", }, period: { + cursor: { default: "pointer", ":disabled": "not-allowed" }, borderStyle: "none", borderRadius: "0.25rem", paddingInline: "0.625rem", diff --git a/admin/src/features/pause/PauseControl.tsx b/admin/src/features/pause/PauseControl.tsx index a37758c..3fcd700 100644 --- a/admin/src/features/pause/PauseControl.tsx +++ b/admin/src/features/pause/PauseControl.tsx @@ -82,6 +82,7 @@ const styles = stylex.create({ boxShadow: "0 1px 3px 0 rgb(0 0 0 / 0.1), 0 1px 2px -1px rgb(0 0 0 / 0.1)", }, menuItem: { + cursor: { default: "pointer", ":disabled": "not-allowed" }, borderStyle: "none", backgroundColor: { default: "transparent", ":hover": colors.surfaceHover }, color: "inherit", diff --git a/admin/src/lib/InlineError.tsx b/admin/src/lib/InlineError.tsx index 11989ff..f94fd75 100644 --- a/admin/src/lib/InlineError.tsx +++ b/admin/src/lib/InlineError.tsx @@ -12,6 +12,7 @@ const styles = stylex.create({ color: colors.danger, }, retry: { + cursor: { default: "pointer", ":disabled": "not-allowed" }, borderStyle: "none", backgroundColor: "transparent", padding: 0, diff --git a/admin/src/ui/ConfirmDialog.tsx b/admin/src/ui/ConfirmDialog.tsx index 8c9f504..dec2374 100644 --- a/admin/src/ui/ConfirmDialog.tsx +++ b/admin/src/ui/ConfirmDialog.tsx @@ -67,6 +67,7 @@ const styles = stylex.create({ marginTop: "1.5rem", }, dangerButton: { + cursor: { default: "pointer", ":disabled": "not-allowed" }, borderRadius: "0.25rem", borderStyle: "none", backgroundColor: colors.danger, diff --git a/admin/src/ui/styles.ts b/admin/src/ui/styles.ts index 32c10aa..b92e82a 100644 --- a/admin/src/ui/styles.ts +++ b/admin/src/ui/styles.ts @@ -60,6 +60,7 @@ export const styles = stylex.create({ }, button: { + cursor: { default: "pointer", [DISABLED]: "not-allowed" }, borderRadius: "0.25rem", borderWidth: 1, borderStyle: "solid", @@ -70,6 +71,7 @@ export const styles = stylex.create({ lineHeight: "1.25rem", }, smallButton: { + cursor: { default: "pointer", [DISABLED]: "not-allowed" }, borderRadius: "0.25rem", borderWidth: 1, borderStyle: "solid", @@ -80,6 +82,7 @@ export const styles = stylex.create({ lineHeight: "1.25rem", }, largeButton: { + cursor: { default: "pointer", [DISABLED]: "not-allowed" }, borderRadius: "0.25rem", borderWidth: 1, borderStyle: "solid", @@ -90,6 +93,7 @@ export const styles = stylex.create({ }, primaryButton: { + cursor: { default: "pointer", [DISABLED]: "not-allowed" }, borderRadius: "0.25rem", borderStyle: "none", backgroundColor: colors.primary, @@ -102,6 +106,7 @@ export const styles = stylex.create({ opacity: { default: 1, [DISABLED]: 0.5 }, }, largePrimaryButton: { + cursor: { default: "pointer", [DISABLED]: "not-allowed" }, borderRadius: "0.25rem", borderStyle: "none", backgroundColor: colors.primary, @@ -113,6 +118,7 @@ export const styles = stylex.create({ }, rowButton: { + cursor: { default: "pointer", [DISABLED]: "not-allowed" }, borderRadius: "0.25rem", borderStyle: "none", backgroundColor: "transparent", @@ -123,6 +129,7 @@ export const styles = stylex.create({ color: colors.primaryOnSurface, }, linkButton: { + cursor: { default: "pointer", [DISABLED]: "not-allowed" }, borderStyle: "none", backgroundColor: "transparent", padding: 0, @@ -132,6 +139,7 @@ export const styles = stylex.create({ color: colors.primaryOnSurface, }, dangerLinkButton: { + cursor: { default: "pointer", [DISABLED]: "not-allowed" }, borderStyle: "none", backgroundColor: "transparent", padding: 0, @@ -142,6 +150,7 @@ export const styles = stylex.create({ opacity: { default: 1, [DISABLED]: 0.5 }, }, retryButton: { + cursor: { default: "pointer", [DISABLED]: "not-allowed" }, marginTop: "0.75rem", borderRadius: "0.25rem", borderWidth: 1,