Use control borders for subnav + disabled button fix (#2643)

* use control borders * Create red-flowers-own.md
primer · Jun 20, 2024 · ab6076c · ab6076c
1 parent cc3854c
commit ab6076c
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/.changeset/red-flowers-own.md b/.changeset/red-flowers-own.md
@@ -0,0 +1,5 @@
+---
+"@primer/css": patch
+---
+
+Use `control` borders for subnav + disabled button fix
diff --git a/src/buttons/button.scss b/src/buttons/button.scss
@@ -93,8 +93,8 @@
   &.disabled,
   &[aria-disabled='true'] {
     color: var(--fgColor-disabled, var(--color-primer-fg-disabled));
-    background-color: var(--button-default-bgColor-rest, var(--color-btn-bg));
-    border-color: var(--button-default-borderColor-rest, var(--color-btn-border));
+    background-color: var(--button-default-bgColor-disabled, var(--color-btn-bg));
+    border-color: var(--button-default-borderColor-disabled, var(--color-btn-border));
 
     .octicon {
       color: var(--fgColor-disabled, var(--color-primer-fg-disabled));

diff --git a/src/navigation/subnav.scss b/src/navigation/subnav.scss
@@ -28,7 +28,7 @@
   // stylelint-disable-next-line primer/typography
   line-height: 20px;
   color: var(--fgColor-default, var(--color-fg-default));
-  border: $border-width $border-style var(--borderColor-default, var(--color-border-default));
+  border: $border-width $border-style var(--control-borderColor-rest, var(--color-border-default));
 
   + .subnav-item {
     // stylelint-disable-next-line primer/spacing