Fix flickering problem when clicking on some disabled buttons

DaoDaoNoCode · mprahl · commit 901c6ebbb7c6 · 2026-02-05T20:15:40.000-05:00
diff --git a/mlflow/server/js/src/common/styles/patternfly/pf-shell-overrides.scss b/mlflow/server/js/src/common/styles/patternfly/pf-shell-overrides.scss
@@ -93,9 +93,9 @@
       }
     }
 
-    &:hover,
-    &:hover:active,
-    &:hover:focus {
+    &:hover:not(:disabled),
+    &:hover:active:not(:disabled),
+    &:hover:focus:not(:disabled) {
       color: var(--pf-t--global--text--color--brand--hover) !important;
       background: transparent !important;
       &::after {
@@ -190,7 +190,7 @@
     color: var(--pf-t--global--text--color--status--on-danger--default) !important;
     border-color: transparent !important;
 
-    &:hover {
+    &:hover:not(:disabled) {
       color: var(--pf-t--global--text--color--status--on-danger--hover) !important;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -93,9 +93,9 @@`
`93`	`93`	`}`
`94`	`94`	`}`
`95`	`95`
`96`		`- &:hover,`
`97`		`- &:hover:active,`
`98`		`- &:hover:focus {`
	`96`	`+ &:hover:not(:disabled),`
	`97`	`+ &:hover:active:not(:disabled),`
	`98`	`+ &:hover:focus:not(:disabled) {`
`99`	`99`	`color: var(--pf-t--global--text--color--brand--hover) !important;`
`100`	`100`	`background: transparent !important;`
`101`	`101`	`&::after {`
`@@ -190,7 +190,7 @@`
`190`	`190`	`color: var(--pf-t--global--text--color--status--on-danger--default) !important;`
`191`	`191`	`border-color: transparent !important;`
`192`	`192`
`193`		`- &:hover {`
	`193`	`+ &:hover:not(:disabled) {`
`194`	`194`	`color: var(--pf-t--global--text--color--status--on-danger--hover) !important;`
`195`	`195`	`}`
`196`	`196`