Merge pull request 'Serve the home nav link from PocketBase, not a hard-coded copy' (#32) from fix/duplicate-start-nav-link into main

Reviewed-on: #32
2026-09-23 22:32:26 +02:00
commit 4b93434679
@@ -19,7 +19,6 @@ public class NavBuilderTests
            Page("board", "header", 2, "Vorstandsteam"),
            Page("home", "header", 1, "Start"),
            Page("imprint", "footer", 1, "Impressum"),
-            Page("hidden", "", 1, "Unterseite"),
        };

        var header = NavBuilder.Header(pages);
@@ -56,13 +55,4 @@ public class NavBuilderTests
        header[0].Href.ShouldBe("/");
        header[1].Href.ShouldBe("/board");
    }
-
-    [Fact]
-    public void Pages_without_a_location_never_appear_in_a_menu()
-    {
-        var pages = new[] { Page("hidden", "", 1, "Unterseite") };
-
-        NavBuilder.Header(pages).ShouldBeEmpty();
-        NavBuilder.Footer(pages).ShouldBeEmpty();
-    }
 }
@@ -85,9 +85,9 @@ public sealed class PocketBaseClientTests(PocketBaseFixture pocketBase)

        var home = pages.Single(p => p.Slug == "home");
        home.Embed.ShouldBe(["posts", "events"], ignoreOrder: true);
-        // Home has no menu slot: the brand is the home link, so a "Start" nav
-        // entry would be redundant. NavBuilder filters it out via this empty value.
-        home.Location.ShouldBe("");
+        // Home sits in the header like every other page (location is required); it
+        // is the single source of the home nav link.
+        home.Location.ShouldBe("header");
    }

    [Fact]
@@ -220,14 +220,14 @@ public sealed class PocketBaseFixture : IAsyncLifetime
    private static async Task SeedRecordsAsync(HttpClient http)
    {
        // Pages that the route smoke tests reach through the navigation, the
-        // footer or the FAQ hub. Header pages carry a location+order; the FAQ
-        // sub-pages are plain content pages without a menu slot. All public.
-        // The home page has no menu slot (location ""): the site brand is the home
-        // link, so a separate "Start" nav entry would be redundant.
+        // footer or the FAQ hub. Every page carries a location ("header" or
+        // "footer") -- the field is required, matching production. All public.
+        // The home page sits in the header at order 1, like production; the brand
+        // links home too, but the nav entry is the single source of the home link.
        await CreateRecordAsync(http, "pages", new
        {
-            title = "Startseite", body = "# Willkommen", location = "",
-            order = 0, slug = "home", embed = new[] { "posts", "events" }, @public = true,
+            title = "Start", body = "# Willkommen", location = "header",
+            order = 1, slug = "home", embed = new[] { "posts", "events" }, @public = true,
        });
        await CreateRecordAsync(http, "pages", new
        {
@@ -253,10 +253,13 @@ public sealed class PocketBaseFixture : IAsyncLifetime
            title = "Downloads", body = "# Downloads", location = "header",
            order = 5, slug = "downloads", embed = Array.Empty<string>(), @public = true,
        });
+        // A plain content page reached only by its slug (not shown in the header),
+        // so the catch-all "/{slug}" ContentPage route stays covered. It carries a
+        // valid location because the field is required; footer keeps it low-key.
        await CreateRecordAsync(http, "pages", new
        {
-            title = "FAQ Mensa", body = "# Mensa", location = "",
-            order = 0, slug = "faq-lunch", embed = Array.Empty<string>(), @public = true,
+            title = "FAQ Mensa", body = "# Mensa", location = "footer",
+            order = 9, slug = "faq-lunch", embed = Array.Empty<string>(), @public = true,
        });
        await CreateRecordAsync(http, "pages", new
        {
@@ -20,10 +20,10 @@
            </label>

            <nav class="main-nav">
-                @* A home link that only shows in the collapsed mobile menu. On wide
-                   screens the brand is the home link, so this would be a redundant
-                   entry -- CSS hides it there (see .nav-home-mobile). *@
-                <NavLink class="nav-home-mobile" href="/" Match="NavLinkMatch.All">Start</NavLink>
+                @* The home link comes from PocketBase like every other entry (the
+                   "home" page carries location "header"), so it is rendered by the
+                   loop below -- no separate hard-coded home link, which would show
+                   up twice next to the PocketBase one. *@
                @foreach (var link in _header)
                {
                    <NavLink href="@link.Href" Match="@(link.Href == "/" ? NavLinkMatch.All : NavLinkMatch.Prefix)">@link.Title</NavLink>
@@ -60,12 +60,10 @@
    gap: 0.25rem 1.25rem;
 }

-/* The hamburger button, its state checkbox, and the mobile-only home link are
-   hidden on wide screens; the media query below turns them on for narrow ones.
-   (On wide screens the brand is the home link, so a "Start" entry is redundant.) */
+/* The hamburger button and its state checkbox are hidden on wide screens; the
+   media query below turns them on for narrow ones. */
 .nav-toggle,
-.nav-toggle-label,
-.nav-home-mobile {
+.nav-toggle-label {
    display: none;
 }

@@ -177,11 +175,6 @@
        margin-top: 0.5rem;
    }

-    /* The mobile-only home link reappears as the first entry in the menu. */
-    .nav-home-mobile {
-        display: block;
-    }
-
    /* Mobile per-link spacing/separators for .main-nav a are in the global app.css
       too, for the same scoped-attribute reason as above. */
 }
@@ -3,9 +3,9 @@ using Elternbeirat.Contracts;
 namespace Elternbeirat.Web.Features.Layout;

 /// <summary>
-/// Builds the header and footer navigation from the public pages. Pages carry a
-/// <see cref="Page.Location"/> ("header"/"footer"/"") and an <see cref="Page.Order"/>;
-/// pages without a location are sub-pages and never appear in a menu.
+/// Builds the header and footer navigation from the public pages. Every page
+/// carries a required <see cref="Page.Location"/> ("header" or "footer") and an
+/// <see cref="Page.Order"/>; this splits them into the two menus, each ordered.
 /// </summary>
 public static class NavBuilder
 {
@@ -21,7 +21,7 @@ migrate((app) => {
  const schema = JSON.parse(toString($os.readFile("/pb_migrations/collections_schema.json")))

  const records = {
-    pages: [{"body":"# Vorstandsteam\n\nDer Elternbeirat der IGMH vertritt die Interessen aller Eltern gegenüber der Schulleitung und dem Kollegium. Wir treffen uns regelmäßig, bringen Anliegen aus der Elternschaft ein und begleiten das Schulleben aktiv.\n\n## Vorstand\n\n- **Vorsitz** — Anke Berger\n- **Stellvertretung** — Murat Yılmaz\n- **Kasse** — Sabine Krause\n- **Schriftführung** — Thomas Heinrich\n\n## So erreichen Sie uns\n\nAm schnellsten per Mail über unsere [Kontaktseite](/contact). Zu unseren öffentlichen Sitzungen sind alle Eltern herzlich eingeladen — die Termine finden Sie unter [Termine](/events).","embed":[],"location":"header","order":2,"public":true,"slug":"board","title":"Vorstandsteam"},{"body":"# Förderverein\n\nDer Förderverein der IGMH unterstützt Projekte, für die im regulären Schulhaushalt kein Geld vorgesehen ist: von der Ausstattung der Bibliothek über Pausenspielgeräte bis zu Zuschüssen für Klassenfahrten, damit kein Kind aus finanziellen Gründen zurückbleibt.\n\n## Mitglied werden\n\nDer Jahresbeitrag beträgt 24 € — jeder Euro fließt direkt in die Arbeit an der Schule. Den Aufnahmeantrag finden Sie unter [Downloads](/downloads).\n\n## Spenden\n\nSpenden sind jederzeit willkommen und werden auf Wunsch mit einer Zuwendungsbestätigung quittiert. Sprechen Sie uns über die [Kontaktseite](/contact) an.","embed":[],"location":"header","order":3,"public":true,"slug":"patrons","title":"Förderverein"},{"body":"# Häufige Fragen\n\nHier beantworten wir die Fragen, die uns am häufigsten erreichen. Ausführliche Antworten gibt es auf den Unterseiten:\n\n- [Mensa und Mittagessen](/faq-mensa)\n- [Schließfächer](/faq-schliessfach)\n- [Eltern-Euro](/faq-elterneuro)\n- [Elternarbeit und Mitmachen](/faq-elternarbeit)\n\nIhre Frage ist nicht dabei? Schreiben Sie uns über die [Kontaktseite](/contact).","embed":[],"location":"header","order":6,"public":true,"slug":"faqs","title":"FAQ"},{"body":"# Downloads\n\nFormulare und Dokumente zum Herunterladen:\n\n- Aufnahmeantrag Förderverein\n- Einzugsermächtigung\n- Erstattungsformular Klassenkasse\n\nFehlt ein Dokument? Melden Sie sich über die [Kontaktseite](/contact).","embed":[],"location":"header","order":7,"public":true,"slug":"downloads","title":"Downloads"},{"body":"# Kontakt\n\nSie erreichen den Elternbeirat der IGMH per E-Mail:\n\n**elternbeirat@igmh.example**\n\nWir antworten in der Regel innerhalb weniger Tage. Bei dringenden schulischen Angelegenheiten wenden Sie sich bitte direkt an das Sekretariat der Schule.","embed":[],"location":"footer","order":1,"public":true,"slug":"contact","title":"Kontakt"},{"body":"# Impressum\n\nAngaben gemäß § 5 DDG\n\n**Elternbeirat der IGMH**\nVertreten durch den Vorstand\n\nKontakt: siehe [Kontaktseite](/contact)\n\nVerantwortlich für den Inhalt nach § 18 Abs. 2 MStV: der Vorstand des Elternbeirats.","embed":[],"location":"footer","order":2,"public":true,"slug":"imprint","title":"Impressum"},{"body":"# Datenschutz\n\nDer Schutz Ihrer Daten ist uns wichtig. Diese Website kommt ohne externe Dienste aus: keine Analysewerkzeuge, keine eingebundenen Karten oder Videos, keine Schriften von fremden Servern.\n\n## Server-Logs\n\nBeim Aufruf werden technisch notwendige Zugriffsdaten (IP-Adresse, Zeitpunkt, aufgerufene Seite) kurzzeitig verarbeitet, um den Betrieb sicherzustellen. Eine Zusammenführung mit anderen Daten findet nicht statt.\n\n## Ihre Rechte\n\nFragen zum Datenschutz richten Sie bitte an die [Kontaktseite](/contact).","embed":[],"location":"footer","order":3,"public":true,"slug":"privacy","title":"Datenschutz"},{"body":"# Mensa und Mittagessen\n\n## Wie melde ich mein Kind zum Mittagessen an?\n\nDie Anmeldung läuft über das Bestellsystem der Schule. Die Zugangsdaten erhalten Sie zu Schuljahresbeginn über die Klassenleitung.\n\n## Was kostet ein Essen?\n\nEin Mittagessen kostet aktuell 4,20 €. Für Familien mit Anspruch auf Leistungen nach dem Bildungs- und Teilhabepaket ist das Essen kostenfrei — sprechen Sie das Sekretariat an.\n\n## Gibt es vegetarische Gerichte?\n\nJa, täglich steht mindestens ein vegetarisches Gericht zur Wahl.\n\nWeitere Fragen? [Schreiben Sie uns.](/contact)","embed":[],"location":"","order":0,"public":true,"slug":"faq-mensa","title":"Mensa und Mittagessen"},{"body":"# Schließfächer\n\n## Wie bekommt mein Kind ein Schließfach?\n\nSchließfächer werden über einen externen Anbieter gemietet. Das Anmeldeformular liegt bei den [Downloads](/downloads).\n\n## Was kostet die Miete?\n\nDie Miete beträgt rund 15 € pro Schuljahr zuzüglich einer Kaution, die bei Rückgabe erstattet wird.\n\n## Der Schlüssel ist weg — was nun?\n\nWenden Sie sich an den Anbieter; die Kontaktdaten stehen auf dem Mietvertrag. Bei Fragen hilft auch der [Elternbeirat](/contact).","embed":[],"location":"","order":0,"public":true,"slug":"faq-schliessfach","title":"Schließfächer"},{"body":"# Eltern-Euro\n\n## Was ist der Eltern-Euro?\n\nDer ElterLine truncated
+    pages: [{"body":"# Vorstandsteam\n\nDer Elternbeirat der IGMH vertritt die Interessen aller Eltern gegenüber der Schulleitung und dem Kollegium. Wir treffen uns regelmäßig, bringen Anliegen aus der Elternschaft ein und begleiten das Schulleben aktiv.\n\n## Vorstand\n\n- **Vorsitz** — Anke Berger\n- **Stellvertretung** — Murat Yılmaz\n- **Kasse** — Sabine Krause\n- **Schriftführung** — Thomas Heinrich\n\n## So erreichen Sie uns\n\nAm schnellsten per Mail über unsere [Kontaktseite](/contact). Zu unseren öffentlichen Sitzungen sind alle Eltern herzlich eingeladen — die Termine finden Sie unter [Termine](/events).","embed":[],"location":"header","order":2,"public":true,"slug":"board","title":"Vorstandsteam"},{"body":"# Förderverein\n\nDer Förderverein der IGMH unterstützt Projekte, für die im regulären Schulhaushalt kein Geld vorgesehen ist: von der Ausstattung der Bibliothek über Pausenspielgeräte bis zu Zuschüssen für Klassenfahrten, damit kein Kind aus finanziellen Gründen zurückbleibt.\n\n## Mitglied werden\n\nDer Jahresbeitrag beträgt 24 € — jeder Euro fließt direkt in die Arbeit an der Schule. Den Aufnahmeantrag finden Sie unter [Downloads](/downloads).\n\n## Spenden\n\nSpenden sind jederzeit willkommen und werden auf Wunsch mit einer Zuwendungsbestätigung quittiert. Sprechen Sie uns über die [Kontaktseite](/contact) an.","embed":[],"location":"header","order":3,"public":true,"slug":"patrons","title":"Förderverein"},{"body":"# Häufige Fragen\n\nHier beantworten wir die Fragen, die uns am häufigsten erreichen.\n\nIhre Frage ist nicht dabei? Schreiben Sie uns über die [Kontaktseite](/contact).","embed":[],"location":"header","order":6,"public":true,"slug":"faqs","title":"FAQ"},{"body":"# Downloads\n\nFormulare und Dokumente zum Herunterladen:\n\n- Aufnahmeantrag Förderverein\n- Einzugsermächtigung\n- Erstattungsformular Klassenkasse\n\nFehlt ein Dokument? Melden Sie sich über die [Kontaktseite](/contact).","embed":[],"location":"header","order":7,"public":true,"slug":"downloads","title":"Downloads"},{"body":"# Kontakt\n\nSie erreichen den Elternbeirat der IGMH per E-Mail:\n\n**elternbeirat@igmh.example**\n\nWir antworten in der Regel innerhalb weniger Tage. Bei dringenden schulischen Angelegenheiten wenden Sie sich bitte direkt an das Sekretariat der Schule.","embed":[],"location":"footer","order":1,"public":true,"slug":"contact","title":"Kontakt"},{"body":"# Impressum\n\nAngaben gemäß § 5 DDG\n\n**Elternbeirat der IGMH**\nVertreten durch den Vorstand\n\nKontakt: siehe [Kontaktseite](/contact)\n\nVerantwortlich für den Inhalt nach § 18 Abs. 2 MStV: der Vorstand des Elternbeirats.","embed":[],"location":"footer","order":2,"public":true,"slug":"imprint","title":"Impressum"},{"body":"# Datenschutz\n\nDer Schutz Ihrer Daten ist uns wichtig. Diese Website kommt ohne externe Dienste aus: keine Analysewerkzeuge, keine eingebundenen Karten oder Videos, keine Schriften von fremden Servern.\n\n## Server-Logs\n\nBeim Aufruf werden technisch notwendige Zugriffsdaten (IP-Adresse, Zeitpunkt, aufgerufene Seite) kurzzeitig verarbeitet, um den Betrieb sicherzustellen. Eine Zusammenführung mit anderen Daten findet nicht statt.\n\n## Ihre Rechte\n\nFragen zum Datenschutz richten Sie bitte an die [Kontaktseite](/contact).","embed":[],"location":"footer","order":3,"public":true,"slug":"privacy","title":"Datenschutz"},{"body":"# Willkommen beim Elternbeirat der IGMH","embed":["posts","events"],"location":"header","order":1,"public":true,"slug":"home","title":"Start"},{"body":"# Beiträge","embed":[],"location":"header","order":4,"public":true,"slug":"posts","title":"Beiträge"},{"body":"# Termine","embed":[],"location":"header","order":5,"public":true,"slug":"events","title":"Termine"}]
 ,
    posts: [{"body":"## Neuer Vorstand gewählt\n\nAuf der Mitgliederversammlung wurde der Vorstand des Elternbeirats für das Schuljahr **2026/27** neu gewählt. Wir danken dem bisherigen Team für sein Engagement und freuen uns auf die gemeinsame Arbeit.\n\nDie neuen Ansprechpartnerinnen und Ansprechpartner finden Sie auf der Seite [Vorstandsteam](/board).","date":"2026-09-24 18:00:00.000Z","public":true,"slug":"new-board-2026","title":"Neuer Vorstand für das Schuljahr 2026/27 gewählt"},{"body":"## Herbstbasar am 22. November\n\nAm **22. November** findet wieder unser beliebter Herbstbasar statt. Es gibt selbst gebackene Kuchen, Bastelstände und ein Bühnenprogramm der Klassen.\n\nWer einen Stand betreuen oder etwas beisteuern möchte, meldet sich bitte bis zum 10. November über das [Kontaktformular](/contact) an. Wir freuen uns über jede helfende Hand!","date":"2026-09-18 09:30:00.000Z","public":true,"slug":"herbstbasar-2026","title":"Herbstbasar am 22. November – jetzt anmelden"},{"body":"## Mehr Auswahl beim Mittagessen\n\nAb den Herbstferien erweitert der Caterer das Angebot in der Mensa: Es gibt künftig **täglich ein zusätzliches vegetarisches Gericht** sowie eine größere Salatauswahl.\n\nRückmeldungen zum Essen nehmen wir gerne über den Elternbeirat auf – sprechen Sie uns an.","date":"2026-09-10 12:15:00.000Z","public":true,"slug":"mensa-mehr-auswahl","title":"Mehr Auswahl beim Mittagessen ab den Herbstferien"},{"body":"## Besuchen Sie unseren Stand\n\nAm kommenden **Elternsprechtag** ist der Elternbeirat mit einem Info-Stand im Foyer vertreten. Kommen Sie vorbei – wir beantworten Fragen rund um die Elternarbeit und freuen uns über neue Mitstreiterinnen und Mitstreiter.\n\nAlle Termine finden Sie unter [Termine](/events).","date":"2026-09-02 08:00:00.000Z","public":true,"slug":"elternsprechtag-stand","title":"Elternbeirat am Elternsprechtag – besuchen Sie unseren Stand"},{"body":"## Neue Spielgeräte für den Pausenhof\n\nDank der Einnahmen aus dem letzten Basar und der Unterstützung des [Fördervereins](/patrons) konnten neue Spielgeräte für den Pausenhof angeschafft werden.\n\nDie Kinder haben die Klettergerüste bereits begeistert in Beschlag genommen.","date":"2026-08-28 16:45:00.000Z","public":true,"slug":"neue-spielgeraete","title":"Neue Spielgeräte für den Pausenhof"}]
 ,
@@ -269,7 +269,7 @@
        "maxSelect": 1,
        "name": "location",
        "presentable": false,
-        "required": false,
+        "required": true,
        "system": false,
        "type": "select",
        "values": [