jekyll-theme-centos 2.52.0.beta.104 → 2.52.0.beta.106

data/_includes/base/configuration_variables.html.liquid

@@ -1,27 +1,25 @@
 {% comment %}
-Generates the top-level property table and initiates the recursive generation
-of separate, nested DataTables for complex properties.
+configuration_variables: renders the page component's schema properties as a table, then one nested table per object or array-of-objects property.
+Data: include.schema > site.data.<component>_schema > site.data.base.<component>_schema
+Schema: src/YAML/configuration_variables_schema.yml
 {% endcomment %}
 
 {% assign component             = page.component.name %}
-{% assign property_name         = "with_" | append: component %}
 {% assign property_name_schema  = component | append: "_schema" %}
 {% assign property_schema       = include.schema | default: site.data[property_name_schema] | default: site.data.base[property_name_schema] %}
 
-{% assign col_name        = include.col_name        | default: "Name" %}
-{% assign col_type        = include.col_type        | default: "Type" %}
-{% assign col_default     = include.col_default     | default: "Default" %}
-{% assign col_description = include.col_description | default: "Description" %}
+{% assign col_name        = include.col_name        | default: site.data.base.configuration_variables_schema.properties.col_name.default %}
+{% assign col_type        = include.col_type        | default: site.data.base.configuration_variables_schema.properties.col_type.default %}
+{% assign col_default     = include.col_default     | default: site.data.base.configuration_variables_schema.properties.col_default.default %}
+{% assign col_description = include.col_description | default: site.data.base.configuration_variables_schema.properties.col_description.default %}
 
 {% assign columns = include.columns | default: property_schema.columns %}
 
 {% if property_schema.properties %}
 
-  {% assign table_classes = "table dataTable" %}
-
   <p>The <code>{{ page.component.name }}</code> {{ page.component.type }} understands the following configuration variables:</p>
 
-  <table class="{{ table_classes }}">
+  <table class="table dataTable">
     <thead>
       <tr>
         <th scope="col">{{ col_name }}</th>
@@ -86,5 +84,5 @@ of separate, nested DataTables for complex properties.
   {% endfor -%}
 
 {% else %}
-  <p>The <code>{{ page.component.name }}</code> {{ page.component.type }} {{ pt }} doesn't have configuration variables.</p>
+  <p>The <code>{{ page.component.name }}</code> {{ page.component.type }} doesn't have configuration variables.</p>
 {% endif %}

data/_includes/base/content.html.liquid

@@ -1,3 +1,8 @@
+{% comment %}
+content: renders the main element with the announcement, bits, event, breadcrumb, title, artwork, and toc rows, then the page content.
+Data: include.data > page.with_content_data > site.data.content > site.data.base.content
+Schema: src/YAML/content_schema.yml
+{% endcomment %}
 {%- assign with_content = include.with_content
 | default: page.with_content
 -%}
@@ -21,13 +26,11 @@
 
 {%- assign row_class = include.row_class
 | default: with_content_data.row_class
-| default: content_schema.properties.with_content_data.properties.row_class.default
-| default: 'row mb-3' -%}
+| default: content_schema.properties.with_content_data.properties.row_class.default -%}
 
 {%- assign col_class = include.col_class
 | default: with_content_data.col_class
-| default: content_schema.properties.with_content_data.properties.col_class.default
-| default: 'col' -%}
+| default: content_schema.properties.with_content_data.properties.col_class.default -%}
 
 {%- if with_content -%}
 <main class="{{ class }}">

data/_includes/base/copyright.html.liquid

@@ -1,34 +1,7 @@
 {% comment %}
-================================================================================
-Template: Copyright Footer Component
-Purpose: Renders copyright notice with author name and optional legal links.
-Shows dynamic year, author, and links to Terms of Service, Privacy Policy, etc.
-================================================================================
-
-Design Logic:
-- Dynamic year: automatically uses current year from site.time
-- Legal links looping: flexible number of links in footer
-- Reusable link component: consistent styling
-- Capture-based rendering: builds complex link string before output
-- Border separator styling: visual dividers between links
-
-Required Inputs:
-- author: Copyright holder name (e.g., "The CentOS Project")
-
-Optional Inputs:
-- legals: Array of legal link objects
-  - name: Link text (e.g., "Terms of Service")
-  - link: URL for legal document
-  - color: Link color (defaults to light)
-  - icon: Icon class (optional)
-  - class: Custom CSS classes (optional)
-
-Data Source Hierarchy (priority):
-1. include.data
-2. page.with_copyright_data
-3. site.data.copyright
-4. site.data.base.copyright
-================================================================================
+copyright: renders the copyright line with the current year, the author, and the legal links.
+Data: include.data > page.with_copyright_data > site.data.copyright > site.data.base.copyright
+Schema: src/YAML/copyright_schema.yml
 {% endcomment %}
 
 {%- assign with_copyright = include.with_copyright
@@ -44,9 +17,6 @@ Data Source Hierarchy (priority):
 {%- assign with_copyright_schema = site.data.base.copyright_schema.properties.with_copyright_data.properties %}
 
 {%- if with_copyright %}
-
-{%- comment %} === Presentation === {% endcomment %}
-{%- comment %} --- Defaults --- {% endcomment %}
 {%- assign copyright_author = with_copyright_data.author | default: with_copyright_schema.author.default %}
 {%- assign copyright_legals = with_copyright_data.legals | default: with_copyright_schema.legals.default %}
 
@@ -54,20 +24,19 @@ Data Source Hierarchy (priority):
 
 {%- capture copyright_links -%}
 {%- for item in copyright_legals -%}
-{%- assign copyright_link_color = item.color | default: copyright_legal_schema.color.default %}
-{%- assign copyright_link_class = item.class | default: copyright_legal_schema.class.default | replace: '{color}', copyright_link_color %}
+{%- assign copyright_link_class = item.class | default: copyright_legal_schema.class.default %}
+{%- comment %} The span draws the separator outside the link, so the focus ring covers only the link text. Its border color matches the footer's default text color. {% endcomment -%}
+<span class="border-start border-centos-purple-100 ms-2 ps-2">
 {%- include base/link.html.liquid
     class         = copyright_link_class
     name          = item.name
     link          = item.link
-    color         = item.color
     role          = item.role
     icon          = item.icon
     icon_position = item.icon_position
 -%}
+</span>
 {%- endfor -%}
-{%- endcapture -%}
-
-{%- comment %} --- HTML --- {% endcomment %}
+{%- endcapture %}
 <p class="my-1">Copyright &copy; {{ site.time | date: "%Y" }} {{ copyright_author }} {{ copyright_links }}</p>
 {%- endif -%}

data/_includes/base/copyvalue.html.liquid

@@ -1,59 +1,7 @@
 {% comment %}
-================================================================================
-Template: Copy Value Component
-Purpose: Copy to clipboard with visual feedback (checkmark + tooltip).
-================================================================================
-
-Required Usage (in template):
-  <button type="button" class="copy-btn focus-ring" data-copy-value="{{ command | escape }}"
-          data-bs-toggle="tooltip" data-bs-title="Copy command">
-    <span class="fa-regular fa-copy"></span>
-  </button>
-
-ACCESSIBILITY — every `.copy-btn` must be a `<button type="button">`. This
-gives it keyboard focus and Enter/Space activation automatically.
-`_customization.scss` resets the button's default border, background, and
-padding back to a plain icon trigger. `.input-group-text.copy-btn`
-(checksum and SRI-hash rows) is the exception — it keeps Bootstrap's own
-addon box styling instead. Every `.copy-btn` also carries `focus-ring` so
-its keyboard-focus ring matches the rest of the theme's interactive
-elements — see `.focus-ring:focus:not(:focus-visible)` in
-`_customization.scss`.
-
-SECURITY — `data-copy-value` MUST always go through Liquid's `| escape` (or
-equivalent HTML-attribute encoding) at the point where it is written, with no
-exception for values that "look safe". The browser's own HTML parser decodes
-entities (&quot;, &amp;, &#39;, …) in an attribute value as it parses the
-tag, so by the time the JS reads `data-copy-value` it is already plain text —
-there is no separate decoding step to rely on for sanitization. An unescaped,
-attacker-influenced value (a query parameter, CMS field, etc.) placed
-directly in data-copy-value breaks out of the HTML attribute at render time,
-before this script ever runs.
-
-Behavior:
-- Visual feedback: checkmark icon + success tooltip
-- Auto-reset: returns to original state after delay
-- Clipboard API: native browser copy functionality
-
-Required Inputs (from HTML element):
-- class="copy-btn": Marks the element as a copy trigger
-- data-copy-value: Text to copy to clipboard
-
-Optional Inputs (from data objects):
-- reset_delay: Milliseconds before reset (default varies by schema)
-- checkmark_path: SVG path for success icon
-- checkmark_viewbox: SVG viewBox for success icon
-- success_message: Tooltip message on success
-
-Data Source Hierarchy (in priority order):
-1. include.data             - Explicitly passed configuration
-2. page.with_copyvalue_data - Page front matter
-3. site.data.copyvalue      - Site configuration
-4. site.data.base.copyvalue - Theme defaults
-
-Dependencies:
-Bootstrap 5, FontAwesome 6+, Clipboard API (modern browsers)
-================================================================================
+copyvalue: loads the script that copies each .copy-btn's data-copy-value to the clipboard.
+Data: include.data > page.with_copyvalue_data > site.data.copyvalue > site.data.base.copyvalue
+Schema: src/YAML/copyvalue_schema.yml
 {% endcomment %}
 
 {%- assign with_copyvalue = include.with_copyvalue
@@ -69,30 +17,15 @@ Bootstrap 5, FontAwesome 6+, Clipboard API (modern browsers)
 {%- assign with_copyvalue_schema = site.data.base.copyvalue_schema.properties.with_copyvalue_data.properties %}
 
 {%- if with_copyvalue %}
-
-{%- comment %} === Presentation === {% endcomment %}
-
-{%- comment %} --- Defaults --- {% endcomment %}
 {%- assign copyvalue_reset_delay       = with_copyvalue_data.reset_delay       | default: with_copyvalue_schema.reset_delay.default %}
 {%- assign copyvalue_checkmark_path    = with_copyvalue_data.checkmark_path    | default: with_copyvalue_schema.checkmark_path.default %}
 {%- assign copyvalue_checkmark_viewbox = with_copyvalue_data.checkmark_viewbox | default: with_copyvalue_schema.checkmark_viewbox.default %}
 {%- assign copyvalue_success_message   = with_copyvalue_data.success_message   | default: with_copyvalue_schema.success_message.default %}
-
-{%- comment %} --- HTML --- {% endcomment %}
-<!-- Load Copy Value functionality with configuration via data attributes -->
 <script id="centos-copyvalue"
-        {%- if with_copyvalue_data.reset_delay %}
         data-reset-delay="{{ copyvalue_reset_delay }}"
-        {%- endif %}
-        {%- if with_copyvalue_data.checkmark_path %}
         data-checkmark-path="{{ copyvalue_checkmark_path }}"
-        {%- endif %}
-        {%- if with_copyvalue_data.checkmark_viewbox %}
         data-checkmark-viewbox="{{ copyvalue_checkmark_viewbox }}"
-        {%- endif %}
-        {%- if with_copyvalue_data.success_message %}
         data-success-message="{{ copyvalue_success_message }}"
-        {%- endif %}
         src="{{ site.baseurl }}/assets/js/base/copyvalue.min.js"
         defer></script>
 

data/_includes/base/datatable.html.liquid

@@ -1,32 +1,7 @@
 {% comment %}
-================================================================================
-Template: DataTables Library Initialization Component
-Purpose: Loads DataTables jQuery plugin from CDN with SRI integrity checks.
-Initializes DataTables on all .dataTable elements with responsive design,
-pagination, searching, and sorting.
-================================================================================
-
-Design Logic:
-- CDN-based library loading with SRI integrity checks
-- Responsive table design for mobile devices
-- Smart pagination: only if table has >10 rows
-- Automatic element detection: targets all .dataTable classes
-- jQuery dependency: requires jQuery loaded first
-
-Optional Inputs:
-- type: "head" for CSS include, "script" for JavaScript
-- version: DataTables version (e.g., "1.13.7")
-- css_integrity, script_integrity: SRI integrity hashes
-- page_length: Default rows per page (default: 10)
-- length_menu: Pagination options
-- data: DataTables configuration object
-
-Data Source Hierarchy (priority):
-1. include.data
-2. page.with_datatable_data
-3. site.data.datatable
-4. site.data.base.datatable
-================================================================================
+datatable: loads DataTables from the CDN and enables it on every table.dataTable. head.html.liquid calls it with type="head" (CSS) and type="script" (JS).
+Data: include.data > page.with_datatable_data > site.data.datatable > site.data.base.datatable
+Schema: src/YAML/datatable_schema.yml
 {% endcomment %}
 
 {%- assign with_datatable = include.with_datatable
@@ -43,8 +18,6 @@ Data Source Hierarchy (priority):
 
 {%- if with_datatable %}
 
-{%- comment %} === Presentation === {% endcomment %}
-{%- comment %} --- Defaults --- {% endcomment %}
 {%- assign datatable_version          = with_datatable_data.version          | default: with_datatable_schema.version.default %}
 {%- assign datatable_css_integrity    = with_datatable_data.css_integrity    | default: with_datatable_schema.css_integrity.default %}
 {%- assign datatable_script_integrity = with_datatable_data.script_integrity | default: with_datatable_schema.script_integrity.default %}
@@ -56,24 +29,15 @@ Data Source Hierarchy (priority):
 {%- endcapture %}
 
 {%- capture html_datatable_script %}
-  <!-- DataTables Library -->
   <script src="https://cdn.datatables.net/{{ datatable_version }}/datatables.min.js" integrity="{{ datatable_script_integrity }}" crossorigin="anonymous" referrerpolicy="no-referrer" defer></script>
-
-  <!-- Initialize DataTables with configuration via data attributes -->
   <script id="centos-datatable"
-          {%- if with_datatable_data.page_length %}
           data-page-length="{{ datatable_page_length }}"
-          {%- endif %}
-          {%- if with_datatable_data.length_menu %}
           data-length-menu='{{ datatable_length_menu | split: "," | jsonify }}'
-          {%- endif %}
           src="{{ site.baseurl }}/assets/js/base/datatable.min.js"
           defer></script>
 {%- endcapture %}
 
-{%- comment %} --- HTML --- {% endcomment %}
-{%- assign type = include.type %}
-{%- case type %}
+{%- case include.type %}
 {%- when "head" %}
   {{ html_datatable_head }}
 {%- when "script" %}

data/_includes/base/download.html.liquid

@@ -1,36 +1,7 @@
 {% comment %}
-================================================================================
-Template: Download Component
-Purpose: Renders one or more download entries. Full data-driven section
-component (like bits/sponsors), gated by with_download and resolved through
-the standard 4-level fallback chain. Each entry in with_download_data is an
-independent download "card" (e.g. one per distribution); include.title
-filters to a single entry. Per-release sections (architecture/documentation/
-eol/commands/screenshots) render automatically when their data is present —
-no separate with_X flag per section, matching how card.html.liquid renders
-its own sections only "if content". Shared between both render paths via
-download_sections.html.liquid.
-
-Alternative images (flavors) are not a section of this schema — a flavor
-is just another download entry, linked via `documentation` or page
-content rather than nested inside this one.
-
-Data Source Hierarchy (in priority order):
-1. include.data            - Explicitly passed array
-2. page.with_download_data - Page front matter
-3. site.data.download      - Site data
-4. site.data.base.download - Theme defaults
-
-Optional Inputs (from include):
-- title:            Render only the entry whose title matches exactly.
-- id:                HTML id prefix for the nav-pills tab selector (default: DownloadsCard-1).
-- with_title:        Set to false to skip the entry's own title heading, e.g. when
-                     the calling page already presents it as a real markdown
-                     heading immediately above the include (for TOC/anchor
-                     navigation). Default: true.
-- with_description:  Set to false to skip the entry's own description text for the
-                     same reason. Default: true.
-================================================================================
+download: renders each download entry, or only the one matching include.title, with its releases as tabs, or inline when there is one release.
+Data: include.data > page.with_download_data > site.data.download > site.data.base.download
+Schema: src/YAML/download_schema.yml
 {% endcomment %}
 
 {%- assign with_download = include.with_download
@@ -52,29 +23,17 @@ Optional Inputs (from include):
     {%- continue %}
   {%- endif %}
 
-  {%- comment %} image_base_path defaults to a path derived from the current
-  page's own URL (page.url already ends in "/" for pretty permalinks) rather
-  than a fixed schema default — every page gets its own collision-free
-  screenshot directory automatically, with no per-page front matter needed,
-  unless the entry explicitly overrides it. See download_schema.yml's
-  image_base_path description for the full rationale. {% endcomment %}
+  {%- comment %} Without an override, each page gets its own screenshot directory from page.url; see image_base_path in download_schema.yml. {% endcomment %}
   {%- if entry.image_base_path and entry.image_base_path != "" %}
     {%- assign entry_image_base_path = entry.image_base_path %}
   {%- else %}
     {%- assign entry_image_base_path = "/assets/img" | append: page.url %}
   {%- endif %}
 
-  {%- comment %} entry.title feeds an HTML id (card_id) below. A hand-picked
-  remove: chain (mirroring link.html.liquid's anchor assignment) missed
-  characters like em dashes, which the id spec forbids just the same —
-  slugify handles any non-alphanumeric character, not just a fixed list. {% endcomment %}
+  {%- comment %} slugify, not a remove: chain, because entry.title feeds an HTML id and can hold any character, such as an em dash. {% endcomment %}
   {%- assign entry_title_id_safe = entry.title | slugify %}
 
-  {%- if include.id and include.id != "" %}
-    {%- assign card_id = include.id | append: "-" | append: entry_title_id_safe | downcase | replace: " ", "-" %}
-  {%- else %}
-    {%- assign card_id = "DownloadsCard-1-" | append: entry_title_id_safe | downcase | replace: " ", "-" %}
-  {%- endif %}
+  {%- assign card_id = include.id | default: "DownloadsCard-1" | append: "-" | append: entry_title_id_safe | downcase | replace: " ", "-" %}
 
   <div class="mb-1">
     {%- assign single_release = nil %}
@@ -92,21 +51,13 @@ Optional Inputs (from include):
     {{ entry.description | strip_newlines | markdownify }}
     {%- endif %}
 
-    {%- comment %} A `releases` field always identifies which version(s) an
-    entry covers; when there's more than one, a tabbed nav-pills selector
-    switches between them. When there's exactly one — the common case for a
-    project like CentOS Stream, which only ever has one active version —
-    the tab selector is skipped and that release's sections render
-    directly, since a single, permanently-active, non-interactive tab would
-    just be chrome with nothing to switch between. {% endcomment %}
-    {%- if entry.releases.size == 1 %}
-      {%- for release in entry.releases %}
-      {%- assign release_data = release[1] %}
+    {%- comment %} A single release renders its sections directly; a lone tab would have nothing to switch between. {% endcomment %}
+    {%- if single_release %}
+      {%- assign release_data = single_release[1] %}
       {%- assign release_id = card_id | append: "-pills1" %}
       <div id="{{ release_id }}" class="bg-white rounded py-3 border">
         {%- include base/download_sections.html.liquid release_data=release_data release_id=release_id image_base_path=entry_image_base_path -%}
       </div>
-      {%- endfor %}
     {%- elsif entry.releases.size > 1 %}
     <ul class="nav nav-pills mb-3 justify-content-start" id="{{ card_id }}-pills-tab" role="tablist">
       {%- for release in entry.releases %}
@@ -117,19 +68,14 @@ Optional Inputs (from include):
           data-bs-target="#{{ card_id }}-pills{{ forloop.index }}"
           type="button"
           role="tab"
-          aria-controls="{{ card_id }}-pills-tab{{ forloop.index }}"
-          {%- if forloop.first %} aria-selected="true"{% endif %}
+          aria-controls="{{ card_id }}-pills{{ forloop.index }}"
+          {%- if forloop.first %} aria-selected="true"{% else %} aria-selected="false"{% endif %}
           >{%- if release[1].icon and release[1].icon != "" %}<i class="{{ release[1].icon }} me-1"></i>{% endif %}{{ release[0] }}</button>
       </li>
       {%- endfor %}
     </ul>
 
-    {%- comment %} border/rounded/bg-white live on this outer .tab-content div,
-    not the inner .tab-pane — a .tab-pane's own corners are always square, so if
-    the rounded, filled look lived there instead, its corners would poke past
-    this div's border at each corner. overflow-hidden clips each pane to match.
-    Mirrors the single-release path above, which only ever needs one
-    rounded/bordered/filled box. {% endcomment %}
+    {%- comment %} Border and rounding live on .tab-content, not .tab-pane, whose square corners would poke past the border; overflow-hidden clips each pane. {% endcomment %}
     <div class="tab-content border rounded overflow-hidden bg-white" id="{{ card_id }}-pills-tabContent">
       {%- for release in entry.releases %}
       {%- assign release_data = release[1] %}

data/_includes/base/download_architecture.html.liquid

@@ -1,37 +1,9 @@
 {% comment %}
-================================================================================
-Utility: Download Architecture Grid
-Purpose: One box per architecture, each with that architecture's own main
-download/checksum/signature and mirror links — the single per-architecture
-data source for a release. Called from download_sections.html.liquid.
-
-Required Inputs:
-- architecture: Map of architecture label (e.g. `x86_64`) to
-  {download, checksum, signature, checksum_value, signature_value,
-  mirrors: {iso, rpm, cloud, containers, vagrant, torrent}} — see
-  download_schema.yml for the full field-by-field description.
-- release_id: The parent release's unique id, combined with each
-  architecture's own label to build a page-unique id_prefix for the
-  checksum/signature rows — see download_filevalue.html.liquid's id_prefix
-  note for why that's needed.
-
-Within each box: icon + architecture title, then (if present) a Download
-button, then (if present) one input-group row each for checksum and
-signature, then (if present) the outline-primary mirror buttons — mirrors
-is always the last section, so it never carries a trailing rule.
-
-Design rationale for specific decisions lives as inline comments next to the
-code they explain: the architecture map's shape versus mirrors/flavors
-(above the grid loop below), the Download button's layout (above its
-markup), the checksum/signature row design (above the download_filevalue
-includes), and the responsive column/divider/orphan-row grid math (above
-the column-count and orphan-detection blocks immediately below).
-================================================================================
+download_architecture: renders one box per architecture with its download button, checksum and signature rows, and mirror buttons.
+Schema: src/YAML/download_schema.yml
 {% endcomment %}
 
-{%- comment %} Column count tracks item count (capped at 4) so the grid always
-fills the row width, instead of a fixed row-cols-md-3 leaving a gap when there
-are only 1 or 2 architectures. {% endcomment %}
+{%- comment %} Column count tracks the architecture count so the grid always fills the row width. {% endcomment %}
 {%- assign arch_count = include.architecture.size -%}
 {%- case arch_count -%}
   {%- when 1 -%}{%- assign arch_cols = 1 -%}
@@ -40,13 +12,7 @@ are only 1 or 2 architectures. {% endcomment %}
   {%- else -%}{%- assign arch_cols = 4 -%}
 {%- endcase -%}
 
-{%- comment %} Orphan-row detection: for each breakpoint's column count,
-figure out whether arch_count leaves a trailing partial row, and if so, the
-0-based index where that partial row starts (orphan_*_start) and the width
-each of its boxes should stretch to (orphan_*_width = 100% / remainder). A
-value of -1 means "no orphan row at this breakpoint" — every box divides
-evenly, or this breakpoint isn't reached at all (arch_cols too low).
-{% endcomment %}
+{%- comment %} orphan_*_start is the index where a trailing partial row begins and orphan_*_width the width its boxes stretch to; -1 means no partial row. {% endcomment %}
 {%- assign row_cols_classes = "row-cols-1" -%}
 {%- assign arch_divider_class = "" -%}
 {%- assign orphan_sm_start = -1 -%}
@@ -56,12 +22,7 @@ evenly, or this breakpoint isn't reached at all (arch_cols too low).
 {%- assign orphan_lg_start = -1 -%}
 {%- assign orphan_lg_width = "" -%}
 
-{%- comment %} Steps up one column per breakpoint (1 → sm:2 → md:3 → lg:4)
-instead of jumping straight from 1 to arch_cols at a single breakpoint, so
-stretching the window grows the grid progressively rather than in one hard
-jump. Capped at arch_cols throughout — a 2-architecture release stops
-adding columns at sm and never reaches 3 or 4, so it doesn't leave empty
-gaps at wider viewports. {% endcomment %}
+{%- comment %} Columns step up one per breakpoint, capped at arch_cols, so the grid grows progressively and never leaves empty gaps. {% endcomment %}
 {%- if arch_cols >= 2 -%}
   {%- assign row_cols_classes = row_cols_classes | append: " row-cols-sm-2" -%}
   {%- assign arch_divider_class = "download-arch-divider-sm" -%}
@@ -99,30 +60,15 @@ gaps at wider viewports. {% endcomment %}
   {%- endif -%}
 {%- endif -%}
 
-{%- comment %} Each architecture owns its own download/checksum/signature/
-mirrors row directly in the architecture map, replacing an older separate
-mirrors array (which carried its own `architecture` field per row and could
-drift out of sync with itself across rows). Alternative images (flavors,
-e.g. a desktop spin) are deliberately NOT part of this grid — a flavor is
-just another download entry, not a smaller variant of this one; model it as
-its own with_download_data entry instead (releases keyed by flavor name
-reuse the tab selector) and link the two via `documentation` or page
-content — see examples/components/download-alternative-images.md.
-{% endcomment %}
+{%- assign download_mirror_keys   = "iso,rpm,cloud,containers,vagrant,torrent" | split: "," %}
+{%- assign download_mirror_labels = "ISO Mirrors,RPM Mirrors,Cloud Images,Container Images,Vagrant Boxes,Torrent" | split: "," %}
+
+{%- comment %} Flavors are separate download entries, not part of this grid; see examples/components/download-alternative-images.md. {% endcomment %}
 <div class="row {{ row_cols_classes }} g-3 align-items-start">
   {%- for arch in include.architecture %}
   {%- assign arch_name = arch[0] %}
   {%- assign arch_data = arch[1] %}
-  {%- comment %} Each box beyond the first in its row gets the divider
-  class for the breakpoint the grid has reached (arch_divider_class, set
-  above per breakpoint) — a border-left shown only once the grid reaches
-  its full arch_cols width, not at every intermediate step. That's
-  deliberate: forloop.index0 modulo arch_cols only identifies "first in
-  its row" correctly once the grid is actually arch_cols-wide, so dividers
-  are hidden during the narrower intermediate steps rather than risk
-  misplaced lines while the grid is still at 2 or 3 columns. A box that
-  starts a wrapped second row (5+ architectures) correctly gets no divider
-  either, same as the very first box overall. {% endcomment %}
+  {%- comment %} Dividers show only at the full arch_cols width, the only width where forloop.index0 modulo arch_cols finds each row's first box. {% endcomment %}
   {%- assign arch_row_position = forloop.index0 | modulo: arch_cols %}
   {%- assign arch_col_class = "col" %}
   {%- unless arch_row_position == 0 %}
@@ -144,16 +90,13 @@ content — see examples/components/download-alternative-images.md.
   {%- endif -%}
 
   {%- assign has_mirrors = false %}
-  {%- if arch_data.mirrors.iso and arch_data.mirrors.iso != "" or arch_data.mirrors.rpm and arch_data.mirrors.rpm != "" or arch_data.mirrors.cloud and arch_data.mirrors.cloud != "" or arch_data.mirrors.containers and arch_data.mirrors.containers != "" or arch_data.mirrors.vagrant and arch_data.mirrors.vagrant != "" or arch_data.mirrors.torrent and arch_data.mirrors.torrent != "" %}
-    {%- assign has_mirrors = true %}
-  {%- endif %}
-  {%- comment %} arch_name is a raw YAML map key, so strip
-  attribute/markup-breaking characters before it seeds arch_id_prefix, which
-  in turn becomes id/data-copy-value scoping in download_filevalue.html.liquid.
-  Note: download.html.liquid's own entry_title_id_safe now uses `slugify`
-  instead of this remove: chain, since slugify catches characters (e.g. em
-  dashes) the chain below doesn't. {% endcomment %}
-  {%- assign arch_name_id_safe = arch_name | remove: '"' | remove: "'" | remove: '<' | remove: '>' | remove: '&' | remove: '#' | remove: '?' | remove: '/' %}
+  {%- for mirror_key in download_mirror_keys %}
+    {%- if arch_data.mirrors[mirror_key] and arch_data.mirrors[mirror_key] != "" %}
+      {%- assign has_mirrors = true %}
+    {%- endif %}
+  {%- endfor %}
+  {%- comment %} slugify, not a remove: chain, because arch_name is a raw YAML key that feeds an HTML id and can hold any character. {% endcomment %}
+  {%- assign arch_name_id_safe = arch_name | slugify %}
   {%- assign arch_id_prefix = include.release_id | default: "download" | append: "-" | append: arch_name_id_safe | downcase | replace: " ", "-" %}
   <div class="{{ arch_col_class }}"{% if arch_orphan_style != "" %} style="{{ arch_orphan_style }}"{% endif %}>
     <div class="d-flex flex-column align-items-start p-3 pt-1">
@@ -178,31 +121,17 @@ content — see examples/components/download-alternative-images.md.
       {%- endif %}
 
       {%- if has_download_button or has_checksum_row or has_signature_row %}
-      {%- comment %} These rows share this gap-2 flex-column wrapper so
-      spacing between whichever rows are present is handled by the gap
-      alone — no per-row conditional mt-2 class needed (Bootstrap's spacing
-      scale maps 2 to the same 0.5rem for both mt-* and gap-*, so this is a
-      visual no-op versus a conditional-margin approach, just less
-      conditional logic). {% endcomment %}
+      {%- comment %} The wrapper's gap spaces whichever rows are present, so no row needs a conditional margin. {% endcomment %}
       <div class="w-100 mb-3 pb-3 border-bottom d-flex flex-column gap-2">
         {%- if has_download_button %}
-        {%- comment %} d-flex flex-wrap gap-2, not Bootstrap's joined
-        .btn-group, so the button reflows instead of overflowing into the
-        neighboring architecture box at narrow column widths (e.g. the
-        4-column arrangement). Reuses base/link.html.liquid, the theme's
-        canonical link/button component. {% endcomment %}
+        {%- comment %} flex-wrap, not .btn-group, so the button reflows instead of overflowing into the next box at narrow widths. {% endcomment %}
         <div class="d-flex flex-wrap gap-2" role="group" aria-label="Primary download actions">
-          {%- include base/link.html.liquid class="btn focus-ring btn-primary flex-fill text-start" role="button" name="Download" link=arch_data.download icon="fa-solid fa-download" icon_position="start" -%}
+          {%- include base/link.html.liquid class="btn focus-ring btn-primary flex-fill text-start" name="Download" link=arch_data.download icon="fa-solid fa-download" icon_position="start" -%}
         </div>
         {%- endif %}
 
         {%- if has_checksum_row %}
-        {%- comment %} checksum/signature and checksum_value/signature_value
-        are two views of the same file, merged into one input-group row
-        (rather than a link button plus a separate copy field) via
-        base/download_filevalue.html.liquid — see that file for the row's
-        own markup and its .link-primary / copy-value details.
-        {% endcomment %}
+        {%- comment %} The file link and its value are two views of one file, so they share one download_filevalue row. {% endcomment %}
         {%- assign checksum_display = arch_data.checksum_value | default: arch_data.checksum %}
         {%- include base/download_filevalue.html.liquid arch_name=arch_name field=arch_data.checksum field_value=checksum_display icon="fa-solid fa-shield-halved" label="checksum" id_prefix=arch_id_prefix -%}
         {%- endif %}
@@ -215,32 +144,15 @@ content — see examples/components/download-alternative-images.md.
       {%- endif %}
 
       {%- if has_mirrors %}
-      {%- comment %} These are outbound links to a mirror/registry, not
-      in-page navigation, so each opens in a new tab (target="_blank") with
-      the same trailing external-link icon as the Documentation section's
-      buttons (base/download_documentation.html.liquid), rather than the
-      Download button's leading icon above — the leading icon there signals
-      the primary action; the trailing external icon here signals "leaves
-      the site". {% endcomment %}
+      {%- comment %} Mirrors leave the site, so they open in a new tab with a trailing external-link icon. {% endcomment %}
       <div class="d-flex flex-wrap gap-2">
-        {%- if arch_data.mirrors.iso and arch_data.mirrors.iso != "" %}
-          {%- include base/link.html.liquid class="btn focus-ring btn-outline-primary flex-fill text-start" role="button" name="ISO Mirrors" link=arch_data.mirrors.iso target="_blank" icon="fa-solid fa-up-right-from-square" icon_position="end" -%}
-        {%- endif %}
-        {%- if arch_data.mirrors.rpm and arch_data.mirrors.rpm != "" %}
-          {%- include base/link.html.liquid class="btn focus-ring btn-outline-primary flex-fill text-start" role="button" name="RPM Mirrors" link=arch_data.mirrors.rpm target="_blank" icon="fa-solid fa-up-right-from-square" icon_position="end" -%}
-        {%- endif %}
-        {%- if arch_data.mirrors.cloud and arch_data.mirrors.cloud != "" %}
-          {%- include base/link.html.liquid class="btn focus-ring btn-outline-primary flex-fill text-start" role="button" name="Cloud Images" link=arch_data.mirrors.cloud target="_blank" icon="fa-solid fa-up-right-from-square" icon_position="end" -%}
-        {%- endif %}
-        {%- if arch_data.mirrors.containers and arch_data.mirrors.containers != "" %}
-          {%- include base/link.html.liquid class="btn focus-ring btn-outline-primary flex-fill text-start" role="button" name="Container Images" link=arch_data.mirrors.containers target="_blank" icon="fa-solid fa-up-right-from-square" icon_position="end" -%}
-        {%- endif %}
-        {%- if arch_data.mirrors.vagrant and arch_data.mirrors.vagrant != "" %}
-          {%- include base/link.html.liquid class="btn focus-ring btn-outline-primary flex-fill text-start" role="button" name="Vagrant Boxes" link=arch_data.mirrors.vagrant target="_blank" icon="fa-solid fa-up-right-from-square" icon_position="end" -%}
-        {%- endif %}
-        {%- if arch_data.mirrors.torrent and arch_data.mirrors.torrent != "" %}
-          {%- include base/link.html.liquid class="btn focus-ring btn-outline-primary flex-fill text-start" role="button" name="Torrent" link=arch_data.mirrors.torrent target="_blank" icon="fa-solid fa-up-right-from-square" icon_position="end" -%}
-        {%- endif %}
+        {%- for mirror_key in download_mirror_keys %}
+          {%- assign mirror_link  = arch_data.mirrors[mirror_key] %}
+          {%- assign mirror_label = download_mirror_labels[forloop.index0] %}
+          {%- if mirror_link and mirror_link != "" %}
+            {%- include base/link.html.liquid class="btn focus-ring btn-outline-primary flex-fill text-start" name=mirror_label link=mirror_link target="_blank" icon="fa-solid fa-up-right-from-square" icon_position="end" -%}
+          {%- endif %}
+        {%- endfor %}
       </div>
       {%- endif %}
     </div>