thunderbird-patch-review

Easy-to-use patch review interface for Thunderbird

Contribute: ~marcc/thunderbird-review-plugin@lists.sr.ht

git clone git://mccd.space/thunderbird-patch-review

commit 27552a85b1df73487f327ccc9da214dadacdf224
parent 7e7527f2aa32b3bbea1c218a9b6ded0376b13fe5
Author: Pi Agent <agent@pi.local>
Date:   Sat, 18 Jul 2026 22:28:51 +0200

review: move the patch pills below the top bar, above the subject

Move #series-nav out of the toolbar and into #content as its first child\n(sitting above the patch box). Pills are left-aligned at the same\nhorizontal edge as the subject and hunks. renderPatch now clears #patch,\na dedicated child below the nav, so the nav persists across re-renders.

Diffstat:

Mextension/review/review.css | 6+++++-


Mextension/review/review.html | 6++++--


Mextension/review/review.js | 2+-


3 files changed, 10 insertions(+), 4 deletions(-)
diff --git a/extension/review/review.css b/extension/review/review.css
@@ -58,13 +58,17 @@ body {
 #series-nav {
   display: flex;
   align-items: center;
+  justify-content: flex-start;
   gap: 0.4rem;
   flex-wrap: wrap;
+  margin: 0.6rem 0;
 }
 
 #series-nav .series-title {
   font-weight: 600;
   white-space: nowrap;
+  margin-bottom: 0.2rem;
+  flex: 1 0 100%;
 }
 
 #actions {
@@ -199,7 +203,7 @@ button.primary:hover { color: #fff; opacity: 0.9; }
 }
 
 .patch-box {
-  margin: 0.6rem 0 1rem;
+  margin: 0 0 1rem;
   border: 1px solid var(--border);
   border-radius: 6px;
   background: var(--surface);
diff --git a/extension/review/review.html b/extension/review/review.html
@@ -7,7 +7,6 @@
 </head>
 <body>
   <header id="topbar">
-    <nav id="series-nav" aria-label="Patch series"></nav>
     <div id="actions">
       <div class="cluster" id="repo-cluster">
         <button id="repo-path" type="button" title="Click to choose the repository directory">no repository chosen</button>
@@ -32,7 +31,10 @@
     </div>
   </header>
 
-  <main id="content"></main>
+  <main id="content">
+    <nav id="series-nav" aria-label="Patch series"></nav>
+    <div id="patch"></div>
+  </main>
 
   <div id="send-confirm" hidden>
     <div class="modal-card">
diff --git a/extension/review/review.js b/extension/review/review.js
@@ -114,7 +114,7 @@ function renderNav() {
 }
 
 function renderPatch() {
-  const content = $("#content");
+  const content = $("#patch");
   content.textContent = "";
   const i = state.current;
   const patch = state.series[i];