Valentin Gagarin <valentin@gagarin.work> writes:
diff --git a/Documentation/_includes/footer_custom.html b/Documentation/_includes/footer_custom.html index 2477920..db29f82 100644 --- a/Documentation/_includes/footer_custom.html +++ b/Documentation/_includes/footer_custom.html @@ -2,9 +2,6 @@ SPDX-FileCopyrightText: 2020 Alyssa Ross <hi@alyssa.is> SPDX-License-Identifier: CC-BY-SA-4.0 OR GFDL-1.3-no-invariants-or-later {% endcomment %} -<p> -<a href="https://spectrum-os.org/git/spectrum/tree/Documentation/{{ page.path }}">Source code for this page</a> - <p> <small>Permission is granted to copy, distribute and/or modify this document under either the terms of the @@ -15,6 +12,9 @@ Documentation License, Version 1.3</a> or any later version published by the Free Software Foundation; with no Invariant Sections, no Front-Cover Texts, and no Back-Cover Texts.</small>
-<p> -<a href="/sitemap.html"><small>Sitemap</small></a> -<a href="/impressum.html" lang="de"><small>Impressum</small></a> +<p><small> +<a href="/sitemap.html">Sitemap</a> +· +<a href="https://spectrum-os.org/git/spectrum/tree/Documentation/{{ page.path }}">Source code for this page</a> +· +<a href="/impressum.html" lang="de">Impressum</a>
Needs a </small> here — it's not optional like </p> is per the spec. I can fix when applying if no other changes are needed. https://html.spec.whatwg.org/multipage/syntax.html#optional-tags (When I was writing the website content originally I liked that omitting the </p> tags helped keep the source clean, but it's an obscure HTML feature other contributors are unlikely to be familiar with, so I wouldn't object to changing to avoiding tag omissions even where permitted by spec.)
diff --git a/Documentation/_layouts/page.html b/Documentation/_layouts/page.html index 230b601..f2cf0f6 100644 --- a/Documentation/_layouts/page.html +++ b/Documentation/_layouts/page.html @@ -3,7 +3,7 @@ SPDX-License-Identifier: GPL-3.0-or-later SPDX-FileCopyrightText: 2026 Valentin Gagarin <valentin@gagarin.work> {% endcomment %} <!doctype html> -<html lang="en"> +<html lang="{{ page.lang | default: "en" }}">
<head>
@@ -17,13 +17,10 @@ SPDX-FileCopyrightText: 2026 Valentin Gagarin <valentin@gagarin.work>
<body>
-<nav aria-label="Breadcrumb"> - <ol> - <li><a href="/">Spectrum</a></li> - <li><a href="/doc/">Documentation</a></li> - <li aria-current="page">{{ page.title }}</li> - </ol> -</nav> +{% include breadcrumb.html %} + +{% assign ext = page.path | split: "." | last %} +{% if ext == "html" and page.no_heading != true %}<h1>{{ page.title }}</h1>{% endif %}
Ah, I've figured out the answer to my own question from last time. AsciiDoctor renders page titles as h1, so without the extension check it would be duplicated here.