From a076eeeb9690d319d2fafd6ff5b60aafa2e3fca3 Mon Sep 17 00:00:00 2001
From: K Simmons <keith@zed.dev>
Date: Wed, 20 Jul 2022 13:43:57 -0700
Subject: [PATCH] tweak diagnostic popover colors

---
 styles/src/styleTree/hoverPopover.ts |  6 +++---
 styles/src/themes/common/base16.ts   | 18 +++++++++---------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/styles/src/styleTree/hoverPopover.ts b/styles/src/styleTree/hoverPopover.ts
index aeac9c01669992efc223c1cb675fe33292d2bfb1..0aac4e66c9ae1ce3e39def70f42a770c32f40597 100644
--- a/styles/src/styleTree/hoverPopover.ts
+++ b/styles/src/styleTree/hoverPopover.ts
@@ -24,7 +24,7 @@ export default function HoverPopover(theme: Theme) {
       ...baseContainer,
       background: backgroundColor(theme, "on500Info"),
       border: {
-        color: theme.ramps.blue(0.2).hex(),
+        color: theme.ramps.blue(0).hex(),
         width: 1,
       },
     },
@@ -32,7 +32,7 @@ export default function HoverPopover(theme: Theme) {
       ...baseContainer,
       background: backgroundColor(theme, "on500Warning"),
       border: {
-        color: theme.ramps.yellow(0.2).hex(),
+        color: theme.ramps.yellow(0).hex(),
         width: 1,
       },
     },
@@ -40,7 +40,7 @@ export default function HoverPopover(theme: Theme) {
       ...baseContainer,
       background: backgroundColor(theme, "on500Error"),
       border: {
-        color: theme.ramps.red(0.2).hex(),
+        color: theme.ramps.red(0).hex(),
         width: 1,
       }
     },
diff --git a/styles/src/themes/common/base16.ts b/styles/src/themes/common/base16.ts
index 78856d6191cb90ea0b77195cee059a7b8deb9755..321184d40d77976058e952c90f840c8088aa6f2f 100644
--- a/styles/src/themes/common/base16.ts
+++ b/styles/src/themes/common/base16.ts
@@ -89,9 +89,9 @@ export function createTheme(
       active: withOpacity(sample(ramps.red, 0.5), 0.25),
     },
     on500Error: {
-      base: sample(ramps.red, 0.1),
-      hovered: sample(ramps.red, 0.15),
-      active: sample(ramps.red, 0.2),
+      base: sample(ramps.red, 0.05),
+      hovered: sample(ramps.red, 0.1),
+      active: sample(ramps.red, 0.15),
     },
     warning: {
       base: withOpacity(sample(ramps.yellow, 0.5), 0.15),
@@ -99,9 +99,9 @@ export function createTheme(
       active: withOpacity(sample(ramps.yellow, 0.5), 0.25),
     },
     on500Warning: {
-      base: sample(ramps.yellow, 0.1),
-      hovered: sample(ramps.yellow, 0.15),
-      active: sample(ramps.yellow, 0.2),
+      base: sample(ramps.yellow, 0.05),
+      hovered: sample(ramps.yellow, 0.1),
+      active: sample(ramps.yellow, 0.15),
     },
     info: {
       base: withOpacity(sample(ramps.blue, 0.5), 0.15),
@@ -109,9 +109,9 @@ export function createTheme(
       active: withOpacity(sample(ramps.blue, 0.5), 0.25),
     },
     on500Info: {
-      base: sample(ramps.blue, 0.1),
-      hovered: sample(ramps.blue, 0.15),
-      active: sample(ramps.blue, 0.2),
+      base: sample(ramps.blue, 0.05),
+      hovered: sample(ramps.blue, 0.1),
+      active: sample(ramps.blue, 0.15),
     },
   };