From e0c1f654788ce8c24051548225d2f44fc16151ce Mon Sep 17 00:00:00 2001
From: lvydra <torgal@hotmail.cz>
Date: Fri, 4 Apr 2025 12:27:26 +0200
Subject: [PATCH] [HAL-2007] Fix utilization bar

---
 .../jboss/hal/ballroom/chart/Utilization.java | 88 +++++++++++--------
 1 file changed, 49 insertions(+), 39 deletions(-)

diff --git a/ballroom/src/main/java/org/jboss/hal/ballroom/chart/Utilization.java b/ballroom/src/main/java/org/jboss/hal/ballroom/chart/Utilization.java
index 309bdd7ce7..095c78b054 100644
--- a/ballroom/src/main/java/org/jboss/hal/ballroom/chart/Utilization.java
+++ b/ballroom/src/main/java/org/jboss/hal/ballroom/chart/Utilization.java
@@ -28,7 +28,19 @@
 
 import static org.jboss.gwt.elemento.core.Elements.div;
 import static org.jboss.gwt.elemento.core.Elements.span;
-import static org.jboss.hal.resources.CSS.*;
+import static org.jboss.hal.resources.CSS.progress;
+import static org.jboss.hal.resources.CSS.progressBar;
+import static org.jboss.hal.resources.CSS.progressBarDanger;
+import static org.jboss.hal.resources.CSS.progressBarRemaining;
+import static org.jboss.hal.resources.CSS.progressBarSuccess;
+import static org.jboss.hal.resources.CSS.progressBarWarning;
+import static org.jboss.hal.resources.CSS.progressContainer;
+import static org.jboss.hal.resources.CSS.progressDescription;
+import static org.jboss.hal.resources.CSS.progressDescriptionLeft;
+import static org.jboss.hal.resources.CSS.progressLabelRight;
+import static org.jboss.hal.resources.CSS.progressLabelTopRight;
+import static org.jboss.hal.resources.CSS.srOnly;
+import static org.jboss.hal.resources.CSS.width;
 import static org.jboss.hal.resources.UIConstants.PROGRESSBAR;
 import static org.jboss.hal.resources.UIConstants.ROLE;
 import static org.jboss.hal.resources.UIConstants.TOGGLE;
@@ -102,46 +114,44 @@ public void update(long current) {
     }
 
     public void update(long current, long total) {
-        if (current <= total) {
-            this.total = total;
-            double currentPercent = Math.round(((double) current) / ((double) total) * 100.0);
-            long remaining = total - current;
-            double remainingPercent = 100.0 - currentPercent;
-
-            valueBar.setAttribute(aria(VALUE_NOW), String.valueOf(current));
-            valueBar.setAttribute(aria(VALUE_MAX), String.valueOf(total));
-            valueBar.style.width = width(currentPercent + "%");
-            Tooltip.element(valueBar).setTitle(MESSAGES.used(currentPercent));
-            //noinspection HardCodedStringLiteral
-            valueElement.innerHTML = new SafeHtmlBuilder()
-                    .appendHtmlConstant("<strong>")
-                    .appendEscaped(MESSAGES.currentOfTotal(current, total))
-                    .appendHtmlConstant("</strong>")
-                    .appendEscaped(" " + unit)
-                    .toSafeHtml().asString();
-
-            remainingBar.setAttribute(aria(VALUE_NOW), String.valueOf(remaining));
-            remainingBar.setAttribute(aria(VALUE_MAX), String.valueOf(total));
-            remainingBar.style.width = width(remainingPercent + "%");
-            Tooltip.element(remainingBar).setTitle(MESSAGES.available(remainingPercent));
-            remainingElement.textContent = MESSAGES.available(remainingPercent);
-
-            if (thresholds) {
-                valueBar.classList.remove(progressBarDanger);
-                valueBar.classList.remove(progressBarWarning);
-                valueBar.classList.remove(progressBarSuccess);
-                if (currentPercent > 90) {
-                    valueBar.classList.add(progressBarDanger);
-                } else if (currentPercent > 75) {
-                    valueBar.classList.add(progressBarWarning);
-                } else {
-                    valueBar.classList.add(progressBarSuccess);
-                }
-            }
-
-        } else {
+        if (current > total) {
             logger.error("Invalid values for utilization bar chart: current > total ({} > {})", current, total);
         }
+        this.total = total;
+        double currentPercent = Math.round(((double) current) / ((double) total) * 100.0);
+        long remaining = total - current;
+        double remainingPercent = 100.0 - currentPercent;
+
+        valueBar.setAttribute(aria(VALUE_NOW), String.valueOf(current));
+        valueBar.setAttribute(aria(VALUE_MAX), String.valueOf(total));
+        valueBar.style.width = width(currentPercent + "%");
+        Tooltip.element(valueBar).setTitle(MESSAGES.used(currentPercent));
+        //noinspection HardCodedStringLiteral
+        valueElement.innerHTML = new SafeHtmlBuilder()
+                .appendHtmlConstant("<strong>")
+                .appendEscaped(MESSAGES.currentOfTotal(current, total))
+                .appendHtmlConstant("</strong>")
+                .appendEscaped(" " + unit)
+                .toSafeHtml().asString();
+
+        remainingBar.setAttribute(aria(VALUE_NOW), String.valueOf(remaining));
+        remainingBar.setAttribute(aria(VALUE_MAX), String.valueOf(total));
+        remainingBar.style.width = width(remainingPercent + "%");
+        Tooltip.element(remainingBar).setTitle(MESSAGES.available(remainingPercent));
+        remainingElement.textContent = MESSAGES.available(remainingPercent);
+
+        if (thresholds) {
+            valueBar.classList.remove(progressBarDanger);
+            valueBar.classList.remove(progressBarWarning);
+            valueBar.classList.remove(progressBarSuccess);
+            if (currentPercent > 90) {
+                valueBar.classList.add(progressBarDanger);
+            } else if (currentPercent > 75) {
+                valueBar.classList.add(progressBarWarning);
+            } else {
+                valueBar.classList.add(progressBarSuccess);
+            }
+        }
     }
 
     private String aria(String name) {