From 354d5448439dfe1f1a5de70bdcf54f22f34dd202 Mon Sep 17 00:00:00 2001 From: Samuel Mannehed Date: Mon, 25 Nov 2019 17:03:55 +0100 Subject: [PATCH] Simplify logic for status hierarchy Removes unnecessary variable --- kasmweb/app/ui.js | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/kasmweb/app/ui.js b/kasmweb/app/ui.js index b421d7d..897cd00 100644 --- a/kasmweb/app/ui.js +++ b/kasmweb/app/ui.js @@ -523,19 +523,14 @@ const UI = { // Don't overwrite more severe visible statuses and never // errors. Only shows the first error. - let visible_status_type = 'none'; if (statusElem.classList.contains("noVNC_open")) { if (statusElem.classList.contains("noVNC_status_error")) { - visible_status_type = 'error'; - } else if (statusElem.classList.contains("noVNC_status_warn")) { - visible_status_type = 'warn'; - } else { - visible_status_type = 'normal'; + return; + } + if (statusElem.classList.contains("noVNC_status_warn") && + status_type === 'normal') { + return; } - } - if (visible_status_type === 'error' || - (visible_status_type === 'warn' && status_type === 'normal')) { - return; } clearTimeout(UI.statusTimeout);