#17 fixes

Zusammengeführt
Tilo15 hat 1 Commits von clanker/goal/ui-ux nach clanker/web-stack vor 1 Tag zusammengeführt
2 geänderte Dateien mit 27 neuen und 1 gelöschten Zeilen
  1. 26 0
      src/pages/app.css
  2. 1 1
      src/pages/main.html

+ 26 - 0
src/pages/app.css

@@ -1514,3 +1514,29 @@ td .truncate.repo-url {
     background: var(--spry-bg-inset);
     color: var(--spry-fg-muted);
 }
+
+/* ==========================================================================
+   Goal 41 review fix-ups (post-T11 review feedback)
+   Two integration defects the review caught: cards had no breathing room
+   (spry's .card carries no padding of its own - padding belongs to
+   .card-header/-body/-footer, and this app puts content directly in .card),
+   and the account menu used .menu, which is spry's ALWAYS-EXPANDED
+   vertical menu variant - it forced the submenu open and bulky. Cards now
+   carry the framework's own --spry-card-padding; the account menu uses
+   .nav like every other dropdown (markup fix in main.html).
+   ========================================================================== */
+
+/* Default card padding: the framework's own card rhythm. Cards built
+   from the header/body/footer children keep the framework's chrome
+   (those children self-pad the same amount, so the parent's padding is
+   removed for them); every other card gets the padding directly.
+   Chosen over negative-margin bleed after a headless-browser probe
+   showed margins break the header/footer full-width chrome. */
+.card {
+    padding: var(--spry-card-padding);
+}
+.card:has(> .card-header),
+.card:has(> .card-body),
+.card:has(> .card-footer) {
+    padding: 0;
+}

+ 1 - 1
src/pages/main.html

@@ -60,7 +60,7 @@
             </ul>
             <!-- spry:nav-end -->
             <span class="nav-auth">
-                <ul class="menu" role="menu">
+                <ul class="nav" role="menu">
                     <li class="has-submenu">
                         <a href="/profile">Account</a>
                         <ul class="submenu" role="menu">