From 7e8ea6affbb040613d807cccd017879440db1681 Mon Sep 17 00:00:00 2001
From: Hannaeko <bksp@dns-witch.eu.org>
Date: Sun, 22 Jun 2025 18:29:09 +0100
Subject: [PATCH] reduce whitespaces in output html

---
 templates/macros/display_rrset.html           | 38 ++++++------
 templates/macros/form.html                    | 45 +++++++-------
 templates/macros/icons.html                   |  6 +-
 templates/pages/new_record.html               | 28 ++++-----
 .../pages/new_record/configure_record.html    | 14 ++---
 templates/pages/records.html                  | 62 +++++++++----------
 6 files changed, 95 insertions(+), 98 deletions(-)

diff --git a/templates/macros/display_rrset.html b/templates/macros/display_rrset.html
index ec16ad5..db82c6f 100644
--- a/templates/macros/display_rrset.html
+++ b/templates/macros/display_rrset.html
@@ -1,15 +1,15 @@
 {% macro rrset(record, zone, lang) %}
 <li class="rrset">
     <div class="rtype">
-        {% if record.record_type == "service" %}
-            {% if record.service.service_type.service_type == "other" %}
+        {%- if record.record_type == "service" -%}
+            {%- if record.service.service_type.service_type == "other" -%}
                 {{ record.service.service_type.name }}/{{ record.service.service_type.protocol }}
-            {% else %}
+            {%- else -%}
                 {{ record.srvice.service_type.service_type }}
-            {% endif %}
-        {% else %}
+            {%- endif -%}
+        {%- else -%}
             {{ tr(msg="zone-content-record-type-" ~ record.record_type, attr="type-name", lang=lang) }}
-        {% endif %}
+        {%- endif -%}
         <div class="action">
             <a class="button icon" href="#">
                 {{ icons::pencil() }}
@@ -18,8 +18,8 @@
     </div>
 
     <ul>
-        {% if record.record_type == "addresses" %}
-            {% for address in record.addresses.addresses %}
+        {%- if record.record_type == "addresses" -%}
+            {%- for address in record.addresses.addresses -%}
                 <li>
                     <div class="rdata">
                         <div class="rdata-main">
@@ -29,9 +29,9 @@
                         </div>
                     </div>
                 </li>
-            {% endfor %}
-        {% elif record.record_type == "mailservers" %}
-            {% for mailserver in record.mailservers.mailservers %}
+            {%- endfor -%}
+        {%- elif record.record_type == "mailservers" -%}
+            {%- for mailserver in record.mailservers.mailservers -%}
                 <li>
                     <div class="rdata-main">
                         <span class="pill">
@@ -48,9 +48,9 @@
                         </span>
                     </div>
                 </li>
-            {% endfor %}
-        {% elif record.record_type == "nameservers" %}
-            {% for nameserver in record.nameservers.nameservers %}
+            {%- endfor -%}
+        {%- elif record.record_type == "nameservers" -%}
+            {%- for nameserver in record.nameservers.nameservers -%}
                 <li>
                     <div class="rdata-main">
                         <span class="pill">
@@ -58,9 +58,9 @@
                         </span>
                     </div>
                 </li>
-            {% endfor %}
-        {% elif record.record_type == "service" %}
-            {% for service_target in record.service.service_targets %}
+            {%- endfor -%}
+        {%- elif record.record_type == "service" -%}
+            {%- for service_target in record.service.service_targets -%}
                 <li>
                     <div class="rdata-main">
                         <span class="pill">
@@ -84,8 +84,8 @@
                         </span>
                     </div>
                 </li>
-            {% endfor %}
-        {% endif %}
+            {%- endfor -%}
+        {%- endif -%}
     </ul>
 </li>
 {% endmacro rrset %}
diff --git a/templates/macros/form.html b/templates/macros/form.html
index 7a944d0..7feda69 100644
--- a/templates/macros/form.html
+++ b/templates/macros/form.html
@@ -1,52 +1,49 @@
-{% macro input(id, name, value, type="text", description="", errors, errors_keys, error_msg_id, label, label_attributes, attributes) %}
+{% macro input(id, name, value, type="text", description="", errors, errors_keys, error_msg_id, label, label_attributes, attributes) -%}
 
-{% set filtered_errors = [] %}
+{%- set filtered_errors = [] -%}
 
-{% for error_key in errors_keys %}
-    {% set error = errors | get(key=error_key, default="") %}
-    {% if error %}
-        {% set_global filtered_errors = filtered_errors | concat(with=error) %}
-    {% endif %}
-{% endfor %}
+{%- for error_key in errors_keys -%}
+    {%- set error = errors | get(key=error_key, default="") -%}
+    {%- if error -%}
+        {%- set_global filtered_errors = filtered_errors | concat(with=error) -%}
+    {%- endif -%}
+{%- endfor -%}
 
 <label
     for="{{ id }}"
 
-    {% for i in range(start=0, end=label_attributes|length, step_by=2) %}
+    {%- for i in range(start=0, end=label_attributes|length, step_by=2) %}
     {{ label_attributes[i] }}="{{ label_attributes | nth(n=i+1) }}"
-    {% endfor %}
+    {%- endfor %}
 >
     {{ label }}
 </label>
 <div>
     <input
-        {% if type == "number" %}
-        type="text"
-        inputmode="numeric"
-        {% else %}
-        type="{{ type }}"
-        {% endif %}
+        {%- if type == "number" %} type="text" inputmode="numeric"
+        {%- else %} type="{{ type }}"
+        {%- endif %}
         name="{{ name }}"
         id="{{ id }}"
-        {% for i in range(start=0, end=attributes|length, step_by=2) %}
+        {%- for i in range(start=0, end=attributes|length, step_by=2) %}
         {{ attributes[i] }}="{{ attributes | nth(n=i+1) }}"
-        {% endfor %}
-        aria-describedby="{% for error in filtered_errors %} {{ id }}-error-{{ loop.index0 }}{% endfor %}{% if description %} {{ id }}-description{%endif %}"
+        {%- endfor %}
+        aria-describedby="{% for error in filtered_errors %} {{ id }}-error-{{ loop.index0 }}{% endfor %}{% if description %} {{ id }}-description{% endif %}"
         {% if filtered_errors %}aria-invalid="true"{% endif %}
         value="{{ value }}"
     >
-    {% for error in filtered_errors %}
+    {%- for error in filtered_errors -%}
         <p class="error" id="{{ id }}-error-{{ loop.index0 }}" data-new-item-skip>
             {{ tr(
                 msg=error_msg_id,
                 attr="error-" ~ error.code | replace(from=":", to="-"),
                 lang=lang) }}
         </p>
-    {% endfor %}
-    {% if description %}
+    {%- endfor -%}
+    {%- if description -%}
         <p class="help" id="{{ id }}-description">
             {{ description }}
         </p>
-    {% endif %}
+    {%- endif -%}
 </div>
-{% endmacro %}
+{%- endmacro %}
diff --git a/templates/macros/icons.html b/templates/macros/icons.html
index 3b99dbd..6e888dd 100644
--- a/templates/macros/icons.html
+++ b/templates/macros/icons.html
@@ -1,11 +1,11 @@
-{% macro plus_circle() %}
+{% macro plus_circle() -%}
 <svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="currentColor" class="bi bi-plus-circle" viewBox="0 0 16 16" aria-hidden="true">
     <path d="M8 15A7 7 0 1 1 8 1a7 7 0 0 1 0 14m0 1A8 8 0 1 0 8 0a8 8 0 0 0 0 16"/>
     <path d="M8 4a.5.5 0 0 1 .5.5v3h3a.5.5 0 0 1 0 1h-3v3a.5.5 0 0 1-1 0v-3h-3a.5.5 0 0 1 0-1h3v-3A.5.5 0 0 1 8 4"/>
 </svg>
-{% endmacro %}
+{%- endmacro %}
 
-{% macro pencil() %}
+{% macro pencil() -%}
 <svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="currentColor" class="bi bi-pencil" viewBox="0 0 16 16">
     <path d="M12.146.146a.5.5 0 0 1 .708 0l3 3a.5.5 0 0 1 0 .708l-10 10a.5.5 0 0 1-.168.11l-5 2a.5.5 0 0 1-.65-.65l2-5a.5.5 0 0 1 .11-.168zM11.207 2.5 13.5 4.793 14.793 3.5 12.5 1.207zm1.586 3L10.5 3.207 4 9.707V10h.5a.5.5 0 0 1 .5.5v.5h.5a.5.5 0 0 1 .5.5v.5h.293zm-9.761 5.175-.106.106-1.528 3.821 3.821-1.528.106-.106A.5.5 0 0 1 5 12.5V12h-.5a.5.5 0 0 1-.5-.5V11h-.5a.5.5 0 0 1-.468-.325"/>
 </svg>
diff --git a/templates/pages/new_record.html b/templates/pages/new_record.html
index 014f0f4..d9873d8 100644
--- a/templates/pages/new_record.html
+++ b/templates/pages/new_record.html
@@ -1,23 +1,23 @@
-{% extends "bases/app.html" %}
+{%- extends "bases/app.html" -%}
 
-{% import "macros/icons.html" as icons %}
-{% import "macros/form.html" as form %}
+{%- import "macros/icons.html" as icons -%}
+{%- import "macros/form.html" as form -%}
 
-{% block title %}{{ tr(msg="new-record-title", lang=lang) }} – {{ current_zone }} ‑ {% endblock title %}
+{%- block title -%}{{ tr(msg="new-record-title", lang=lang) }} – {{ current_zone }} ‑ {%- endblock title -%}
 
-{% block main %}
+{%- block main -%}
 <h1>{{ tr(msg="record-creation-process-heading", zone=current_zone, lang=lang) }}</h1>
 
 {{ errors | json_encode(pretty=true) }}
 
-{% set domain_error = errors | get(key="/name", default="") %}
+{%- set domain_error = errors | get(key="/name", default="") -%}
 
-{% if not new_record_name or (new_record_name and domain_error) %}
-    {% include "pages/new_record/choose_name.html" %}
-{% elif not config and not rtype %}
-    {% include "pages/new_record/choose_record.html" %}
-{% else %}
-    {% include "pages/new_record/configure_record.html" %}
-{% endif %}
+{%- if not new_record_name or (new_record_name and domain_error) -%}
+    {%- include "pages/new_record/choose_name.html" -%}
+{%- elif not config and not rtype -%}
+    {%- include "pages/new_record/choose_record.html" -%}
+{%- else -%}
+    {%- include "pages/new_record/configure_record.html" -%}
+{%- endif -%}
 
-{% endblock %}
+{%- endblock -%}
diff --git a/templates/pages/new_record/configure_record.html b/templates/pages/new_record/configure_record.html
index 356d54f..6666e1b 100644
--- a/templates/pages/new_record/configure_record.html
+++ b/templates/pages/new_record/configure_record.html
@@ -1,4 +1,4 @@
-{% if config == "web" %}
+{%- if config == "web" -%}
 
 <h2>{{ tr(msg="record-config-web-heading", name=new_record_name, lang=lang) }}</h2>
 
@@ -24,7 +24,7 @@
         }}
         </div>
     </div>
-    {% for address in input_data.addresses.data.addresses | default(value=[""]) %}
+    {%- for address in input_data.addresses.data.addresses | default(value=[""]) -%}
         <div class="form-input" data-new-item-template="address">
             {{
                 form::input(
@@ -56,14 +56,14 @@
             {{ icons::plus_circle() }}
             {{ tr(msg="button-add-address", lang=lang) }}
         </button>
-    {% endfor %}
+    {%- endfor -%}
 
     <div class="form-action">
         <button type="submit">{{ tr(msg="button-save-configuration", lang=lang) }}</button>
     </div>
 </form>
 
-{% elif config == "mail" %}
+{%- elif config == "mail" -%}
 
 <h2>{{ tr(msg="record-config-mail-heading", name=new_record_name, lang=lang) }}</h2>
 
@@ -89,7 +89,7 @@
         }}
     </div>
 
-    {% for mailserver in input_data.mailservers.data.mailservers | default(value=[""]) %}
+    {%- for mailserver in input_data.mailservers.data.mailservers | default(value=[""]) -%}
         <fieldset data-new-item-template="mailserver">
             <legend data-new-item-template-content="{{ tr(msg="record-input-mailservers", attr="legend", index="{i}", lang=lang) }}">
                 {{ tr(msg="record-input-mailservers", attr="legend", index=loop.index, lang=lang) }}
@@ -146,7 +146,7 @@
                 }}
             </div>
         </fieldset>
-    {% endfor %}
+    {%- endfor -%}
 
     <button class="form-new-item" type="button" data-new-item="mailserver">
         {{ icons::plus_circle() }}
@@ -260,4 +260,4 @@
         <button type="submit">{{ tr(msg="button-save-configuration", lang=lang) }}</button>
     </div>
 </form>
-{% endif %}
+{%- endif -%}
diff --git a/templates/pages/records.html b/templates/pages/records.html
index 8df4c94..fdce5f1 100644
--- a/templates/pages/records.html
+++ b/templates/pages/records.html
@@ -1,11 +1,11 @@
-{% import "macros/display_rrset.html" as rrset %}
-{% import "macros/icons.html" as icons %}
+{%- import "macros/display_rrset.html" as rrset -%}
+{%- import "macros/icons.html" as icons -%}
 
-{% extends "bases/app.html" %}
+{%- extends "bases/app.html" -%}
 
-{% block title %}{{ tr(msg="zone-content-title", lang=lang) }} – {{ current_zone }} – {% endblock title %}
+{%- block title -%}{{ tr(msg="zone-content-title", lang=lang) }} – {{ current_zone }} – {%- endblock title -%}
 
-{% block main %}
+{%- block main -%}
     <h1>{{ tr(msg="zone-header", lang=lang, zone_name="<strong>" ~ current_zone ~ "</strong>") | safe }}</h1>
     <svg width="0" height="0" aria-hidden="true" style="position: absolute;">
         <defs>
@@ -17,7 +17,7 @@
 
     <section>
         <h2>{{ tr(msg="zone-content-records-header", lang=lang) }}</h2>
-        {% for node in records.records %}
+        {%- for node in records.records -%}
             <article class="domain">
                 <header>
                     <h3 class="folder-tab">{{ node.name }}</h3>
@@ -27,75 +27,75 @@
                         {{ tr(msg="zone-content-new-record-button", lang=lang) }}
                     </a>
                 </header>
-                {% set sections = node.records | group_by(attribute="record_section") %}
+                {%- set sections = node.records | group_by(attribute="record_section") -%}
                 <div class="records">
-                    {% if sections.web %}
-                    {% set records = sections.web | group_by(attribute="record_type") %}
+                    {%- if sections.web -%}
+                    {%- set records = sections.web | group_by(attribute="record_type") -%}
                     <h4>{{ tr(msg="zone-content-section-web-header", lang=lang) }}</h4>
                     <ul>
-                        {% if records.addresses %}
+                        {%- if records.addresses -%}
                             {{ rrset::rrset(
                                 record=records.addresses.0,
                                 zone=current_zone,
                                 lang=lang) }}
-                        {% endif %}
+                        {%- endif -%}
                     </ul>
-                    {% endif %}
+                    {%- endif -%}
 
-                    {% if sections.mail %}
-                    {% set records = sections.mail | group_by(attribute="record_type") %}
+                    {%- if sections.mail -%}
+                    {%- set records = sections.mail | group_by(attribute="record_type") -%}
                     <h4>{{ tr(msg="zone-content-section-mail-header", lang=lang) }}</h4>
                     <ul>
-                        {% if records.mailservers %}
+                        {%- if records.mailservers -%}
                             {{ rrset::rrset(
                                 record=records.mailservers.0,
                                 zone=current_zone,
                                 lang=lang) }}
-                        {% endif %}
-                        {% if records.spf %}
+                        {%- endif -%}
+                        {%- if records.spf -%}
                             {{ rrset::rrset(
                                 record=records.spf.0,
                                 zone=current_zone,
                                 lang=lang) }}
-                        {% endif %}
+                        {%- endif -%}
                     </ul>
-                    {% endif %}
+                    {%- endif -%}
 
-                    {% if sections.services %}
+                    {%- if sections.services -%}
                     <h4>{{ tr(msg="zone-content-section-services-header", lang=lang) }}</h4>
                     <ul>
-                        {% for service in sections.services %}
+                        {%- for service in sections.services -%}
                             {{ rrset::rrset(
                                 record=service,
                                 zone=current_zone,
                                 lang=lang) }}
-                        {% endfor %}
+                        {%- endfor -%}
                     </ul>
-                    {% endif %}
+                    {%- endif -%}
 
-                    {% if sections.miscellaneous %}
+                    {%- if sections.miscellaneous -%}
                     <h4>{{ tr(msg="zone-content-section-general-header", lang=lang) }}</h4>
                     <ul>
-                        {% for record in sections.miscellaneous %}
+                        {%- for record in sections.miscellaneous -%}
                             {{ rrset::rrset(
                                 record=record,
                                 zone=current_zone,
                                 lang=lang) }}
-                        {% endfor %}
+                        {%- endfor -%}
                     </ul>
-                    {% endif %}
+                    {%- endif -%}
 
                 </div>
             </article>
-        {% endfor %}
+        {%- endfor -%}
     </section>
     <section>
         <h2>{{ tr(msg="zone-content-aliases-header", lang=lang) }}</h2>
 
         <ul>
-        {% for alias in records.aliases %}
+        {%- for alias in records.aliases -%}
             <li>{{ alias.from }} → {{ alias.target }}</li>
-        {% endfor %}
+        {%- endfor -%}
         </ul>
     </section>
-{% endblock main %}
+{%- endblock main -%}