From 59bc66f0ef4d34201d0a81e8586b2df5b10399f3 Mon Sep 17 00:00:00 2001
From: Gilles De Mey <gilles.de.mey@gmail.com>
Date: Mon, 20 Mar 2023 07:54:50 +0100
Subject: [PATCH] Alerting: Normalize border-radius usage (#64613)

---
 .../app/features/alerting/unified/components/DynamicTable.tsx   | 2 +-
 .../receivers/grafanaAppReceivers/GrafanaAppBadge.tsx           | 2 +-
 .../unified/components/rule-editor/RuleEditorSection.tsx        | 2 +-
 .../alerting/unified/components/silences/SilencesTable.tsx      | 2 +-
 public/app/plugins/panel/alertlist/AlertList.tsx                | 2 +-
 public/app/plugins/panel/alertlist/UnifiedAlertList.tsx         | 2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/public/app/features/alerting/unified/components/DynamicTable.tsx b/public/app/features/alerting/unified/components/DynamicTable.tsx
index a627f8eb1c8..caf74ef40d2 100644
--- a/public/app/features/alerting/unified/components/DynamicTable.tsx
+++ b/public/app/features/alerting/unified/components/DynamicTable.tsx
@@ -187,7 +187,7 @@ const getStyles = <T extends unknown>(
   return (theme: GrafanaTheme2) => ({
     container: css`
       border: 1px solid ${theme.colors.border.strong};
-      border-radius: 2px;
+      border-radius: ${theme.shape.borderRadius()};
       color: ${theme.colors.text.secondary};
     `,
     row: css`
diff --git a/public/app/features/alerting/unified/components/receivers/grafanaAppReceivers/GrafanaAppBadge.tsx b/public/app/features/alerting/unified/components/receivers/grafanaAppReceivers/GrafanaAppBadge.tsx
index 473654d08c1..39fb34cd0dc 100644
--- a/public/app/features/alerting/unified/components/receivers/grafanaAppReceivers/GrafanaAppBadge.tsx
+++ b/public/app/features/alerting/unified/components/receivers/grafanaAppReceivers/GrafanaAppBadge.tsx
@@ -26,7 +26,7 @@ const getStyles = (theme: GrafanaTheme2) => ({
     height: 22px;
     display: inline-flex;
     padding: 1px 4px;
-    border-radius: 3px;
+    border-radius: ${theme.shape.borderRadius()};
     border: 1px solid rgba(245, 95, 62, 1);
     color: rgba(245, 95, 62, 1);
     font-weight: ${theme.typography.fontWeightRegular};
diff --git a/public/app/features/alerting/unified/components/rule-editor/RuleEditorSection.tsx b/public/app/features/alerting/unified/components/rule-editor/RuleEditorSection.tsx
index 2941cfaff36..8fe6d96113d 100644
--- a/public/app/features/alerting/unified/components/rule-editor/RuleEditorSection.tsx
+++ b/public/app/features/alerting/unified/components/rule-editor/RuleEditorSection.tsx
@@ -57,7 +57,7 @@ const getStyles = (theme: GrafanaTheme2) => ({
     width: ${theme.spacing(4)};
     height: ${theme.spacing(4)};
     line-height: ${theme.spacing(4)};
-    border-radius: ${theme.spacing(4)};
+    border-radius: 100%;
     text-align: center;
     color: ${theme.colors.text.maxContrast};
     background-color: ${theme.colors.background.canvas};
diff --git a/public/app/features/alerting/unified/components/silences/SilencesTable.tsx b/public/app/features/alerting/unified/components/silences/SilencesTable.tsx
index 379577cac70..a86e3aed00d 100644
--- a/public/app/features/alerting/unified/components/silences/SilencesTable.tsx
+++ b/public/app/features/alerting/unified/components/silences/SilencesTable.tsx
@@ -151,7 +151,7 @@ const getStyles = (theme: GrafanaTheme2) => ({
   callout: css`
     background-color: ${theme.colors.background.secondary};
     border-top: 3px solid ${theme.colors.info.border};
-    border-radius: 2px;
+    border-radius: ${theme.shape.borderRadius()};
     height: 62px;
     display: flex;
     flex-direction: row;
diff --git a/public/app/plugins/panel/alertlist/AlertList.tsx b/public/app/plugins/panel/alertlist/AlertList.tsx
index 9c7e7c8f68f..59094ee0fd1 100644
--- a/public/app/plugins/panel/alertlist/AlertList.tsx
+++ b/public/app/plugins/panel/alertlist/AlertList.tsx
@@ -234,7 +234,7 @@ const getStyles = (theme: GrafanaTheme2) => ({
     height: 100%;
     background: ${theme.colors.background.secondary};
     padding: ${theme.spacing(0.5)} ${theme.spacing(1)};
-    border-radius: ${theme.shape.borderRadius(2)};
+    border-radius: ${theme.shape.borderRadius()};
     margin-bottom: ${theme.spacing(0.5)};
   `,
   alertRuleItemIcon: css`
diff --git a/public/app/plugins/panel/alertlist/UnifiedAlertList.tsx b/public/app/plugins/panel/alertlist/UnifiedAlertList.tsx
index a3e034cee50..b33c853f824 100644
--- a/public/app/plugins/panel/alertlist/UnifiedAlertList.tsx
+++ b/public/app/plugins/panel/alertlist/UnifiedAlertList.tsx
@@ -261,7 +261,7 @@ export const getStyles = (theme: GrafanaTheme2) => ({
     height: 100%;
     background: ${theme.colors.background.secondary};
     padding: ${theme.spacing(0.5)} ${theme.spacing(1)};
-    border-radius: ${theme.shape.borderRadius(2)};
+    border-radius: ${theme.shape.borderRadius()};
     margin-bottom: ${theme.spacing(0.5)};
 
     gap: ${theme.spacing(2)};