From 58dfb7df4586e85794cb3a700d9323776273ec23 Mon Sep 17 00:00:00 2001 From: Samuel Mannehed Date: Tue, 4 Oct 2022 14:25:13 +0200 Subject: Don't react to interactions with disabled buttons Disabled buttons should not change appearance on mouse hover, click or on keyboard focus. Doing so destroys the "disabled" impression. --- app/styles/base.css | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/app/styles/base.css b/app/styles/base.css index 0dc31f6..b7840c5 100644 --- a/app/styles/base.css +++ b/app/styles/base.css @@ -522,19 +522,19 @@ select:active { .noVNC_button:focus { outline: none; } -.noVNC_button:active { +.noVNC_button:not(:disabled):active { padding-top: 5px; padding-bottom: 3px; } /* Android browsers don't properly update hover state if touch events * are intercepted, but focus should be safe to display */ -:root:not(.noVNC_touch) .noVNC_button.noVNC_selected:hover, -.noVNC_button.noVNC_selected:focus { +:root:not(.noVNC_touch) .noVNC_button.noVNC_selected:not(:disabled):hover, +.noVNC_button.noVNC_selected:not(:disabled):focus { border-color: rgba(0, 0, 0, 0.4); background: rgba(0, 0, 0, 0.2); } -:root:not(.noVNC_touch) .noVNC_button:hover, -.noVNC_button:focus { +:root:not(.noVNC_touch) .noVNC_button:not(:disabled):hover, +.noVNC_button:not(:disabled):focus { background: rgba(255, 255, 255, 0.2); } .noVNC_button.noVNC_hidden { -- cgit v1.2.1