From 4eabe68a356be4631d544340616cca4166ffa0a6 Mon Sep 17 00:00:00 2001
From: gl0w <gl0w@world>
Date: Thu, 10 Sep 2026 07:34:03 +0100
Subject: [PATCH] m6/table2: natural scroll tables + inline title buttons

---
 src/commits.rs   | 30 ++++++++++++++++
 src/forge.rs     | 93 +++++++++++++++++++++++++++++++++++++++++++++---
 src/releases.rs  | 35 ++++++++++++++++++
 static/style.css | 12 +++++++
 4 files changed, 166 insertions(+), 4 deletions(-)

diff --git a/src/commits.rs b/src/commits.rs
index 9c3b288..d5fa33e 100644
--- a/src/commits.rs
+++ b/src/commits.rs
@@ -322,6 +322,36 @@ mod tests {
         assert!(!out.join("commit").join(format!("{}.html", all[2])).exists());
     }
 
+    /// m6/table-scroll (f0rge #30): long subject + full sha render inside a
+    /// .table-scroll wrapper with td.title / td.sha cells.
+    #[test]
+    fn long_subject_scrolls_with_title_cell() {
+        let dir = mktemp("long-subject");
+        git(&dir, &["init", "-b", "main"]);
+        git(&dir, &["config", "user.name", "hu13-test"]);
+        git(&dir, &["config", "user.email", "hu13@test.local"]);
+        git(&dir, &["config", "commit.gpgsign", "false"]);
+        let long_subject = "s".repeat(200);
+        std::fs::write(dir.join("x.txt"), "x\n").unwrap();
+        git(&dir, &["add", "-A"]);
+        git(&dir, &["commit", "-m", &long_subject]);
+        let out = mktemp("out");
+        render(&dir, &out, "main", 100, "test", "<nav>", None).expect("render");
+        let list = std::fs::read_to_string(out.join("commits.html")).expect("list");
+        assert!(
+            list.contains("<div class=\"table-scroll\"><table>"),
+            "wrapper"
+        );
+        assert!(list.contains("<td class=\"title\">"), "title cell");
+        assert!(list.contains("<td class=\"sha\">"), "sha cell");
+        assert!(list.contains(&long_subject), "long subject kept");
+        let sha = shas(&dir)[0].clone();
+        assert!(
+            list.contains(&format!("/commit/{sha}.html")),
+            "full sha link"
+        );
+    }
+
     #[test]
     fn empty_log_renders_none_yet() {
         let dir = mktemp("empty");
diff --git a/src/forge.rs b/src/forge.rs
index 1f8af3e..3574c8a 100644
--- a/src/forge.rs
+++ b/src/forge.rs
@@ -947,8 +947,8 @@ fn render_issues_index(
     nav: &str,
     description: Option<&str>,
 ) -> Result<String, String> {
-    let mut body = String::from("<h1>Issues</h1>\n");
-    body.push_str("<p><a class=\"btn-new\" href=\"/new-issue.html\">+ New issue</a></p>\n");
+    let mut body =
+        String::from("<div class=\"page-head\"><h1>Issues</h1>\n<a class=\"btn-new\" href=\"/new-issue.html\">+ New issue</a></div>\n");
     if data.issues.is_empty() {
         body.push_str("<p class=\"muted\">No issues yet. <a href=\"/new-issue.html\">Suggest one</a> — no account needed.</p>\n");
     } else {
@@ -994,8 +994,8 @@ fn render_issues_index(
 }
 
 fn render_pulls_index(prs: &[Pr], site_name: &str, nav: &str, description: Option<&str>) -> String {
-    let mut body = String::from("<h1>Pulls</h1>\n");
-    body.push_str("<p><a class=\"btn-new\" href=\"/new-pr.html\">+ New PR prompt</a></p>\n");
+    let mut body =
+        String::from("<div class=\"page-head\"><h1>Pulls</h1>\n<a class=\"btn-new\" href=\"/new-pr.html\">+ New PR prompt</a></div>\n");
     if prs.is_empty() {
         body.push_str("<p class=\"muted\">No pull requests yet. <a href=\"/new-pr.html\">Suggest one</a> — no account needed.</p>\n");
     } else {
@@ -1566,4 +1566,89 @@ mod tests {
         assert!(!data.ci.steps.is_empty(), "real ci steps read");
         assert!(data.ci.latest.is_some(), "real ci latest run read");
     }
+
+    /// m6/table-scroll (f0rge #30): CSS keeps natural width + nowrap cells
+    /// with a wrapping title floor; hover tint via var.
+    #[test]
+    fn table_scroll_css_rules() {
+        let css = include_str!("../static/style.css");
+        assert!(css.contains("width:max-content"), "natural width");
+        assert!(css.contains("min-width:100%"), "full-bleed floor");
+        assert!(
+            css.contains(".table-scroll th,.table-scroll td") && css.contains("white-space:nowrap"),
+            "cells nowrap"
+        );
+        assert!(
+            css.contains(".table-scroll td.title") && css.contains("min-width:12rem"),
+            "title floor"
+        );
+        assert!(
+            css.contains("tr:hover td") && css.contains("var(--bg-th)"),
+            "hover tint"
+        );
+    }
+
+    /// m6/table-scroll (f0rge #31): h1 + button share one .page-head row.
+    #[test]
+    fn page_head_css_and_markup() {
+        let css = include_str!("../static/style.css");
+        assert!(css.contains(".page-head"), "page-head rule");
+        assert!(css.contains("align-items:center"), "inline row");
+        let repo = mktemp("page-head");
+        let out = mktemp("page-head-out");
+        render_loaded(&load(&repo), &out, "test", "<nav>", None).expect("render");
+        for (page, title, btn) in [
+            ("issues/index.html", "<h1>Issues</h1>", "+ New issue"),
+            ("pulls/index.html", "<h1>Pulls</h1>", "+ New PR prompt"),
+        ] {
+            let html = std::fs::read_to_string(out.join(page)).expect(page);
+            let head = html.find("<div class=\"page-head\">").expect("page-head");
+            let h1 = html.find(title).expect("h1");
+            let b = html.find(btn).expect("button");
+            let close = html[head..].find("</div>").expect("close") + head;
+            assert!(
+                h1 > head && b > head && h1 < close && b < close,
+                "{page}: h1+btn in page-head"
+            );
+        }
+        // Tasks has no submit button (no page-head needed).
+        let tasks = std::fs::read_to_string(out.join("tasks/index.html")).expect("tasks");
+        assert!(!tasks.contains("btn-new"), "tasks: no submit button");
+    }
+
+    /// m6/table-scroll (f0rge #30): 200-char title + long sha stay in a
+    /// .table-scroll wrapper with a td.title cell (scroll, never squash).
+    #[test]
+    fn long_title_and_sha_scroll() {
+        let long_title = "t".repeat(200);
+        let long_sha = "a".repeat(64);
+        let repo = mktemp("long-title");
+        write_forge(
+            &repo,
+            "issues/aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa/issue.md",
+            &format!(
+                "---\nid: aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa\ntitle: {long_title}\nstate: open\ncreated: 2026-09-10T03:00:00Z\n---\nbody\n"
+            ),
+        );
+        write_forge(
+            &repo,
+            "prs/bbbbbbbb-bbbb-bbbb-bbbb-bbbbbbbbbbbb/pr.md",
+            &format!(
+                "---\nid: bbbbbbbb-bbbb-bbbb-bbbb-bbbbbbbbbbbb\ntitle: {long_title}\nstate: open\nbase: main\nhead: {long_sha}\ncreated: 2026-09-10T03:00:00Z\n---\ndesc\n"
+            ),
+        );
+        let out = mktemp("long-title-out");
+        render_loaded(&load(&repo), &out, "test", "<nav>", None).expect("render");
+        for page in ["issues/index.html", "pulls/index.html"] {
+            let html = std::fs::read_to_string(out.join(page)).expect(page);
+            assert!(
+                html.contains("<div class=\"table-scroll\"><table>"),
+                "{page}: wrapper"
+            );
+            assert!(html.contains("<td class=\"title\">"), "{page}: title cell");
+            assert!(html.contains(&long_title), "{page}: long title kept");
+        }
+        let pulls = std::fs::read_to_string(out.join("pulls/index.html")).expect("pulls");
+        assert!(pulls.contains(&long_sha), "long head sha kept");
+    }
 }
diff --git a/src/releases.rs b/src/releases.rs
index 10d39c7..ad238d9 100644
--- a/src/releases.rs
+++ b/src/releases.rs
@@ -462,6 +462,41 @@ mod tests {
         assert_eq!(r.display_title(), "v1.0");
     }
 
+    /// m6/table-scroll (f0rge #30): 200-char release title renders in a
+    /// .table-scroll wrapper with a td.title cell.
+    #[test]
+    fn long_release_title_scrolls_with_title_cell() {
+        let long_title = "r".repeat(200);
+        let rel = Release {
+            version: "v9.9.9".to_string(),
+            title: Some(long_title.clone()),
+            created: "2026-09-10T03:00:00Z".to_string(),
+            sha: Some("a".repeat(64)),
+            body: "notes\n".to_string(),
+            artifacts: Vec::new(),
+        };
+        let repo = std::env::temp_dir().join(format!(
+            "hu13-rel-long-{}-{}",
+            std::process::id(),
+            std::time::SystemTime::now()
+                .duration_since(std::time::UNIX_EPOCH)
+                .map(|d| d.as_nanos())
+                .unwrap_or(0)
+        ));
+        std::fs::create_dir_all(&repo).unwrap();
+        let out = repo.join("out");
+        std::fs::create_dir_all(&out).unwrap();
+        render_loaded(&[rel], &repo, &out, "test", "<nav>", None).expect("render");
+        let index =
+            std::fs::read_to_string(out.join("releases").join("index.html")).expect("index");
+        assert!(
+            index.contains("<div class=\"table-scroll\"><table>"),
+            "wrapper"
+        );
+        assert!(index.contains("<td class=\"title\">"), "title cell");
+        assert!(index.contains(&long_title), "long title kept");
+    }
+
     #[test]
     fn missing_dir_is_empty() {
         let dir = std::env::temp_dir().join(format!(
diff --git a/static/style.css b/static/style.css
index ad2b9da..5234011 100644
--- a/static/style.css
+++ b/static/style.css
@@ -157,3 +157,15 @@ a.ln:hover,a.ln:focus{text-decoration:underline}
 .clone-block h3{margin:1rem 0 .25rem}
 .clone-block p{margin:.5rem 0}
 .clone-cmd code{user-select:all;overflow-wrap:anywhere;word-break:break-all}
+/* List tables scroll, never squash (m6/table-scroll, f0rge #30): natural
+   width inside .table-scroll; cells nowrap except title cells which wrap
+   with a floor so rows scroll horizontally. Hover tint via var. */
+.table-scroll table{width:max-content;min-width:100%}
+.table-scroll th,.table-scroll td{white-space:nowrap}
+.table-scroll td.title{white-space:normal;overflow-wrap:anywhere;min-width:12rem}
+.table-scroll tbody tr:hover td{background:var(--bg-th)}
+/* Page titles with inline buttons (m6/table-scroll, f0rge #31): h1 +
+   button share one flex row; wrap allowed on 360px but stays adjacent. */
+.page-head{display:flex;flex-wrap:wrap;align-items:center;gap:.5rem 1rem}
+.page-head h1{margin:0}
+.page-head .btn-new{flex:none}
-- 
2.55.0