From b9bdac049892bab300ca095ed93de8489d1dae66 Mon Sep 17 00:00:00 2001 From: orest-s <orest.hlodan@deque.com> Date: Fri, 5 Jan 2024 16:05:46 +0200 Subject: [PATCH] fix: keep rest background for disabled:active button state --- packages/styles/button.css | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/packages/styles/button.css b/packages/styles/button.css index 8f4310468..8a8321b85 100644 --- a/packages/styles/button.css +++ b/packages/styles/button.css @@ -107,7 +107,7 @@ button.Link { background-color: var(--button-background-color-primary-disabled); } -.Button--primary:active { +.Button--primary:not([disabled]):not([aria-disabled='true']):active { background: var(--button-background-color-primary-active); } @@ -123,7 +123,7 @@ button.Link { background-color: var(--button-background-color-secondary-disabled); } -.Button--secondary:active { +.Button--secondary:not([disabled]):not([aria-disabled='true']):active { background-color: var(--button-background-color-secondary-active); } @@ -140,7 +140,7 @@ button.Link { background-color: var(--button-background-color-error-disabled); } -.Button--error:active { +.Button--error:not([disabled]):not([aria-disabled='true']):active { background-color: var(--button-background-color-error-active); } @@ -236,12 +236,15 @@ button.Link { color: var(--dark-workspace-color); } -.cauldron--theme-dark .Button--primary:active, -.cauldron--theme-dark .Button--secondary:active { +.cauldron--theme-dark + .Button--primary:not([disabled]):not([aria-disabled='true']):active, +.cauldron--theme-dark + .Button--secondary:not([disabled]):not([aria-disabled='true']):active { color: var(--accent-medium); } -.cauldron--theme-dark .Button--error:active { +.cauldron--theme-dark + .Button--error:not([disabled]):not([aria-disabled='true']):active { background-color: #fea7a6; color: var(--accent-medium); }