Improve upgradable status display

komodorio · Mar 7, 2023 · 209f5b5 · 209f5b5
1 parent a0680a4
commit 209f5b5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/pkg/dashboard/static/list-view.js b/pkg/dashboard/static/list-view.js
@@ -88,14 +88,18 @@ function buildChartCard(elm) {
             return
         }
 
-        if (isNewerVersion(elm.chartVersion, data[0].version)) {
-            const icon = $("<span class='ms-2 text-success' data-bs-toggle='tooltip' data-bs-placement='bottom' data-bs-title='Upgrade available: " + data[0].version + " from " + data[0].repository + "'></span>")
+        if (isNewerVersion(elm.chartVersion, data[0].version) || data[0].isSuggestedRepo) {
+            const icon = $("<br/><span class='ms-2 fw-bold' data-bs-toggle='tooltip' data-bs-placement='bottom'></span>")
             if (data[0].isSuggestedRepo) {
-                icon.addClass("bi-arrow-up-circle")
+                icon.addClass("bi-plus-circle-fill text-primary")
+                icon.text(" ADD REPO")
+                icon.attr("data-bs-title", "Add '" + data[0].repository+"' to list of known repositories")
             } else {
-                icon.addClass("bi-arrow-up-circle-fill")
+                icon.addClass("bi-arrow-up-circle-fill text-primary")
+                icon.text(" UPGRADE")
+                icon.attr("data-bs-title", "Upgrade available: " + data[0].version + " from " + data[0].repository)
             }
-            card.find(".rel-name span").append(icon)
+            card.find(".rel-chart div").append(icon)
 
             const tooltipTriggerList = card.find('[data-bs-toggle="tooltip"]')
             const tooltipList = [...tooltipTriggerList].map(tooltipTriggerEl => new bootstrap.Tooltip(tooltipTriggerEl))

diff --git a/pkg/dashboard/static/scripts.js b/pkg/dashboard/static/scripts.js
@@ -117,8 +117,8 @@ $("#topNav ul a").click(function () {
     initView()
 })
 
-const myAlert = document.getElementById('errorAlert')
-myAlert.addEventListener('close.bs.alert', event => {
+const errAlert = document.getElementById('errorAlert')
+errAlert.addEventListener('close.bs.alert', event => {
     event.preventDefault()
     $("#errorAlert").hide()
 })