Tweak design

Author: seproDev

src/_layouts/base.njk

@@ -23,7 +23,7 @@
                             <a class="nav-link{% if '/modules/' === page.url %} active{% endif %}" href="{{ "/modules/" | url }}">Modules</a>
                         </li>
                         <li class="nav-item">
-                            <a class="nav-link" href="https://github.com/seproDev/depctrl-browser">Github</a>
+                            <a class="nav-link" href="https://github.com/seproDev/depctrl-browser">GitHub</a>
                         </li>
                     </ul>
                 </div>

src/index.njk

@@ -12,13 +12,13 @@ layout: base.njk
             macros and modules. This site allows browsing of those macros and modules.
         </p>
         <div class="d-grid gap-2 d-sm-flex justify-content-sm-center">
-            <a href="{{ "/feeds/" | url }}" role="button" class="btn btn-primary btn-lg px-4 gap-3">
+            <a href="{{ "/feeds/" | url }}" role="button" class="btn btn-outline-dark btn-lg px-4 gap-3">
                 Feeds <span class="badge text-bg-secondary">{{ feeds | length }}</span>
             </a>
-            <a href="{{ "/macros/" | url }}" role="button" class="btn btn-primary btn-lg px-4 gap-3">
+            <a href="{{ "/macros/" | url }}" role="button" class="btn btn-outline-dark btn-lg px-4 gap-3">
                 Macros <span class="badge text-bg-secondary">{{ macros | length }}</span>
             </a>
-            <a href="{{ "/modules/" | url }}" role="button" class="btn btn-primary btn-lg px-4 gap-3">
+            <a href="{{ "/modules/" | url }}" role="button" class="btn btn-outline-dark btn-lg px-4 gap-3">
                 Modules <span class="badge text-bg-secondary">{{ modules | length }}</span>
             </a>
         </div>

src/pages/feeds-list.njk

@@ -13,7 +13,7 @@ permalink: "feeds/"
       <th scope="col">Macros</th>
       <th scope="col">Modules</th>
       <th scope="col">File Hash</th>
-      <th scope="col">Notes</th>
+      <th scope="col">Issues</th>
     </tr>
   </thead>
   <tbody>
@@ -25,16 +25,16 @@ permalink: "feeds/"
       <td></td>
       <td></td>
       <td></td>
-      <td>JSON could not be parsed</td>
+      <td>⚠️ JSON could not be parsed</td>
     </tr>
     {% else %}
-    <tr {% if feed._defective or not feed._validHash %} class="table-warning"{% endif %}>
+    <tr>
       <th><a href="{{ ["/feeds/" , feed._identifier, "/"] | join | url }}">{{ feed.name }}</a> <a role="button" class="badge btn btn-secondary" href="{{ ["/feeds/" , feed._identifier, "/"] | join | url }}">{{ feed._identifier}}</a></th>
       <td>{{ feed.maintainer }}</td>
       <td>{{ feed.macros | length }}</td>
       <td>{{ feed.modules | length }}</td>
       <td>{% if feed._validHash %}✅{% else %}❌{% endif %}</td>
-      <td>{{ feed._defective}}</td>
+      <td>{% if feed._defective %}⚠️ {{ feed._defective}}{% endif %}</td>
     </tr>
     {% endif %}
     {% endfor %}

src/pages/feeds-pages.njk

@@ -39,7 +39,7 @@ Feed may not be JSON compliant due to {{ feed._defective }}
   </thead>
   <tbody>
     {% for macroNamespace, macro in feed.macros %}
-      <tr {% if not macro._validHash %} class="table-warning"{% endif %}>
+      <tr>
         <th><a href="{{ ["/macros/" , macroNamespace, "/"] | join | url }}">{{ macro.name }}</a></th>
         <td>{{ macroNamespace }}</td>
         <td>{{ macro.author }}</td>
@@ -68,7 +68,7 @@ Feed may not be JSON compliant due to {{ feed._defective }}
   </thead>
   <tbody>
     {% for modulesNamespace, module in feed.modules %}
-      <tr {% if not module._validHash %} class="table-warning"{% endif %}>
+      <tr>
         <th><a href="{{ ["/modules/" , modulesNamespace, "/"] | join | url }}">{{ module.name }}</a></th>
         <td>{{ modulesNamespace }}</td>
         <td>{{ module.author }}</td>

src/pages/macros-list.njk

@@ -19,7 +19,7 @@ permalink: "macros/"
     {% for macroList in macros | automationListSort %}
       {% if (macroList | length) === 1 %}
         {% set macro = macroList | first %}
-        <tr {% if not macro._validHash %} class="table-warning"{% endif %}>
+        <tr {% if not macro._validHash %}class="table-danger"{% endif %}>
           <th><a href="{{ ["/macros/" , macro._namespace, "/"] | join | url }}">{{ macro.name }}</a></th>
           <td>{{ macro._namespace }}</td>
           <td>{{ macro.author }}</td>
@@ -28,7 +28,7 @@ permalink: "macros/"
         </tr>
       {% else %}
         {% set firstMacro = macroList | first %}
-        <tr class="table-warning">
+        <tr {% if (macroList | singleValue("_validHash") === false ) and not firstMacro._validHash %}class="table-danger"{% endif %}>
           <th><a href="{{ ["/macros/" , firstMacro._namespace, "/"] | join | url }}">{{ macroList | singleValue("name") }}</a></th>
           <td>{{ firstMacro._namespace }}</td>
           <td>{{ macroList | singleValue("author") }}</td>

src/pages/modules-list.njk

@@ -19,7 +19,7 @@ permalink: "modules/"
     {% for moduleList in modules | automationListSort %}
       {% if (moduleList | length) === 1 %}
         {% set module = moduleList | first %}
-        <tr {% if not module._validHash %} class="table-warning"{% endif %}>
+        <tr {% if not module._validHash %} class="table-danger"{% endif %}>
           <th><a href="{{ ["/modules/" , module._namespace, "/"] | join | url }}">{{ module.name }}</a></th>
           <td>{{ module._namespace }}</td>
           <td>{{ module.author }}</td>
@@ -28,7 +28,7 @@ permalink: "modules/"
         </tr>
       {% else %}
         {% set firstModule = moduleList | first %}
-        <tr class="table-warning">
+        <tr {% if (moduleList | singleValue("_validHash") === false ) and not firstModule._validHash %}class="table-danger"{% endif %}>
           <th><a href="{{ ["/modules/" , firstModule._namespace, "/"] | join | url }}">{{ moduleList | singleValue("name") }}</a></th>
           <td>{{ firstModule._namespace }}</td>
           <td>{{ moduleList | singleValue("author") }}</td>