Commit cbbaad8 h

Pat Ambrosio · 26-08-26 14:01

Close the gap under the section headings

Changes: 3 files +45 -6

Changes (Side-by-Side)

Raw

commit cbbaad87330b1ba428cb1a17881408856a067642
Author: Pat Ambrosio <cp.ambrosio@gmail.com>
Date:   Wed Aug 26 14:01:53 2026 +0800

    Close the gap under the section headings
    
    The tag heading sat about 22px above its links. It had copied "margin:
    0 0 1em" from the notes and references labels, but those sit above lists
    that bring their own leading -- over a single row of inline links the same
    margin was a visible empty band.
    
    The tag label now takes no bottom margin at all; its own line box is the
    gap. Notes and references drop from 1em to .25em for the same reason, and
    the citations list loses the top margin that was stacking underneath its
    heading -- references opened 14px below its label where notes opened at 4.
    
    All three now open their content at 4-5px, so REFERENCES, NOTES and TAGS
    read as one family rather than three spacings.
    
    Tests pin each of the three values. Checked by mutation -- restoring the
    1em margin, restoring the citations list gap -- both caught.
    
    Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
    Claude-Session: https://claude.ai/code/session_01WnSAstgnDYdKPwLGv1YZE5

diff --git a/share/jenny/layout/page.sh b/share/jenny/layout/page.sh
index 9f48148..e079730 100755
--- a/share/jenny/layout/page.sh
+++ b/share/jenny/layout/page.sh
@@ -635,17 +635,24 @@ cat << _EOF_
            them with classes and leaves naming to the theme, which keeps
            English out of the parser and lets a theme relabel or hide either
            by overriding one rule. Styled to match an h5. */
+        /* A quarter of the old 1em. These labels sit above lists that bring
+           their own leading, so a full line of margin on top of that left a
+           visible empty band under the word -- the same thing that made the
+           tag heading look detached from its links. */
         ul.fn-list::before,
         .citations::before { display: block; font-size: 1em; font-weight: normal;
                              text-transform: uppercase; color: var(--text-primary);
-                             margin: 0 0 1em; }
+                             margin: 0 0 .25em; }
         ul.fn-list { list-style: none; padding: 2em 0 0; margin-top: 2.5em; font-size: .9em; }
         ul.fn-list::before { content: "Notes"; }
 
         .citations { margin-top: 2.5em; padding-top: 2em; font-size: .9em; }
         .citations::before { content: "References"; }
         .citations hr { display: none; }
-        .citations ol { padding-left: 1.4em; }
+        /* The list's own top margin sat under the label as well, so
+           references opened with a 14px band where notes and tags had 4-5.
+           The label supplies the gap for all three now. */
+        .citations ol { padding-left: 1.4em; margin-top: 0; }
         .citations li { margin-bottom: .5em; }
         .fn-handle, .fn-text { display: table-cell; }
         .fn-handle { padding-right: 1ch; }
@@ -659,9 +666,13 @@ cat << _EOF_
            :has(a) because the div is emitted whether or not the post has tags;
            without it every untagged post grew a heading over nothing. */
         .tags { margin-top: 2.5em; padding-top: 1.5em; font-size: .9em; border-top: 2px solid var(--border-lighter); }
+        /* No bottom margin: notes and references set 1em because each sits
+           above a list with its own leading, and a row of inline links needs
+           none of that -- it left roughly 22px of empty band under the word.
+           The label's own line box is the gap. */
         .tags:has(a)::before { content: "Tags"; display: block; font-size: 1em;
                                font-weight: normal; text-transform: uppercase;
-                               color: var(--text-primary); margin: 0 0 1em; }
+                               color: var(--text-primary); }
         /* MMD metadata pairs are preserved in the output rather than
            stripped, so nothing the author wrote is lost. The default theme
            hides them: the title comes from the heading and tags render
diff --git a/share/jenny/layout/post.sh b/share/jenny/layout/post.sh
index fdf9d21..46bd819 100755
--- a/share/jenny/layout/post.sh
+++ b/share/jenny/layout/post.sh
@@ -399,17 +399,24 @@ cat << _EOF_
            them with classes and leaves naming to the theme, which keeps
            English out of the parser and lets a theme relabel or hide either
            by overriding one rule. Styled to match an h5. */
+        /* A quarter of the old 1em. These labels sit above lists that bring
+           their own leading, so a full line of margin on top of that left a
+           visible empty band under the word -- the same thing that made the
+           tag heading look detached from its links. */
         ul.fn-list::before,
         .citations::before { display: block; font-size: 1em; font-weight: normal;
                              text-transform: uppercase; color: var(--text-primary);
-                             margin: 0 0 1em; }
+                             margin: 0 0 .25em; }
         ul.fn-list { list-style: none; padding: 2em 0 0; margin-top: 2.5em; font-size: .9em; }
         ul.fn-list::before { content: "Notes"; }
 
         .citations { margin-top: 2.5em; padding-top: 2em; font-size: .9em; }
         .citations::before { content: "References"; }
         .citations hr { display: none; }
-        .citations ol { padding-left: 1.4em; }
+        /* The list's own top margin sat under the label as well, so
+           references opened with a 14px band where notes and tags had 4-5.
+           The label supplies the gap for all three now. */
+        .citations ol { padding-left: 1.4em; margin-top: 0; }
         .citations li { margin-bottom: .5em; }
         .fn-handle, .fn-text { display: table-cell; }
         .fn-handle { padding-right: 1ch; }
@@ -423,9 +430,13 @@ cat << _EOF_
            :has(a) because the div is emitted whether or not the post has tags;
            without it every untagged post grew a heading over nothing. */
         .tags { margin-top: 2.5em; padding-top: 1.5em; font-size: .9em; border-top: 2px solid var(--border-lighter); }
+        /* No bottom margin: notes and references set 1em because each sits
+           above a list with its own leading, and a row of inline links needs
+           none of that -- it left roughly 22px of empty band under the word.
+           The label's own line box is the gap. */
         .tags:has(a)::before { content: "Tags"; display: block; font-size: 1em;
                                font-weight: normal; text-transform: uppercase;
-                               color: var(--text-primary); margin: 0 0 1em; }
+                               color: var(--text-primary); }
         /* MMD metadata pairs are preserved in the output rather than
            stripped, so nothing the author wrote is lost. The default theme
            hides them: the title comes from the heading and tags render
diff --git a/test/regressions.sh b/test/regressions.sh
index 668f21f..f154f68 100755
--- a/test/regressions.sh
+++ b/test/regressions.sh
@@ -893,6 +893,23 @@ assert_file_contains "$tl_post" ':has(a)::before'
 it "pages carry the same rule"
 assert_file_contains "$tl/.dist/plain.html" '.tags:has(a)::before { content: "Tags";'
 
+it "a section heading sits close to what it labels"
+# 1em of margin on top of a list's own leading left a visible empty band under
+# the word -- about 22px under the tag row. The three headings now open their
+# content at 4-5px apiece.
+assert_file_contains "$tl_post" 'margin: 0 0 .25em; }'
+
+it "the tag label leans on its own line box for the gap"
+# A row of inline links has no list leading to account for, so it needs no
+# bottom margin at all.
+tag_rule=$("$SED" -n '/\.tags:has(a)::before/,/}/p' "$tl_post")
+assert_not_contains "$tag_rule" 'margin'
+
+it "the references list adds no second gap"
+# The <ol> brought its own top margin, so references opened 14px below its
+# heading where notes and tags opened at 4.
+assert_file_contains "$tl_post" '.citations ol { padding-left: 1.4em; margin-top: 0; }'
+
 it "the build is clean"
 assert_eq "$stderr_output" ""