Add some extra styling to the test connection message

* Colour makes the message easier to understand at a glance.
Jack-Dane · Aug 25, 2024 · 2617226 · 2617226
1 parent 2a5de35
commit 2617226
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 9 deletions.
diff --git a/admin/pages/connections/test_connection_result.css b/admin/pages/connections/test_connection_result.css
@@ -5,3 +5,11 @@
     border: 1px solid #c3c4c7;
     padding: 8px 10px;
 }
+
+.test-result-ok {
+    background: #ddffdd;
+}
+
+.test-result-failure {
+    background: #ffdddd;
+}
diff --git a/admin/pages/connections/test_connection_result.php b/admin/pages/connections/test_connection_result.php
@@ -1,10 +1,12 @@
-<div class="wrap odoo-conn-message-notification-box">
-    <?php $test_result = $test_result ?? null ?>
+<?php
+$test_result = $test_result ?? null;
+if (!is_null($test_result)) {
+	$connection_result_class = $test_result['success'] ? 'test-result-ok' : 'test-result-failure'
+	?>
+    <div class="wrap odoo-conn-message-notification-box <?= $connection_result_class ?>">
+        <h1>Test Result: <?= $test_result['success'] ? 'Success' : 'Failure' ?> </h1>
 
-    <?php if (!is_null($test_result)) { ?>
-        <h1>Test Result: <?= $test_result['success'] ? "Success" : "Failure" ?> </h1>
-
-        <?php if (!$test_result['success']) { ?>
+		<?php if (!$test_result['success']) { ?>
             <!-- Display error details on failure -->
             <div>
                 <b>Error Message: </b>
@@ -15,6 +17,6 @@
                 <b>Error Code: </b>
                 <span><?= esc_html__($test_result['error_code']) ?></span>
             </div>
-        <?php } ?>
-    <?php } ?>
-</div>
+		<?php } ?>
+    </div>
+<?php } ?>