From 681f908218c52b9e12d587ad896e4abc48629096 Mon Sep 17 00:00:00 2001
From: Sukhwinder Dhillon <sukhwinder33445@gmail.com>
Date: Thu, 14 Oct 2021 14:25:58 +0200
Subject: [PATCH] Add optional modules to a container and add color acording to
 availability

---
 application/views/scripts/config/module.phtml |  2 +-
 public/css/icinga/main.less                   | 20 ++++++++++++++++---
 2 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/application/views/scripts/config/module.phtml b/application/views/scripts/config/module.phtml
index 3608e953c..a65f95916 100644
--- a/application/views/scripts/config/module.phtml
+++ b/application/views/scripts/config/module.phtml
@@ -107,7 +107,7 @@
                         $i = 0;
                         $has = $moduleManager->hasAny($modules);
                       ?>
-                    <tbody class="optional-module-group">
+                    <tbody class="optional-module-group <?= ! $has ? 'missing-all-optional' : null ?>">
                       <?php foreach ($modules as $moduleName => $versionString): ?>
                           <?php if ($i > 0): ?>
                               <tr><th class="or-separator"><?= $this->translate('or') ?></th></tr>
diff --git a/public/css/icinga/main.less b/public/css/icinga/main.less
index b8fa6e9f3..cd5fd4233 100644
--- a/public/css/icinga/main.less
+++ b/public/css/icinga/main.less
@@ -441,9 +441,23 @@ a:hover > .icon-cancel {
       font-weight: bold;
     }
 
-    .or-separator {
-      position: relative;
-      transform: translate(70%, 0);
+    .optional-module-group {
+      display: block;
+      margin-top: 1em;
+      margin-left: -1.1em;
+      padding-left: 1em;
+      border-left: 2px solid;
+      .var(border-left-color, color-up);
+
+      &.missing-all-optional {
+        .var(border-left-color, color-critical);
+      }
+
+      .or-separator {
+        position: absolute;
+        transform: translate(0, -50%);
+        right: 0;
+      }
     }
   }
 }