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 862ef5015181b3f592de86497aa40745e3ad784e
parent a50534bb5d2f55be7d2f3112fe979aad4e0d81a1
Author: Pi Agent <agent@pi.local>
Date:   Sat, 18 Jul 2026 13:12:37 +0200

review: drop duplicate From header, keep only Subject

The parsed commit message no longer carries the format-patch From: line,
so the per-patch metadata header only needs the Subject now. Render it at
the same size and black color as the surrounding commit message body
(13px / var(--fg)).

Diffstat:
M
extension/review/review.css
 | 
12
++++++------
M
extension/review/review.js
 | 
1
-

2 files changed, 6 insertions(+), 7 deletions(-)
diff --git a/extension/review/review.css b/extension/review/review.css
@@ -233,26 +233,26 @@ button.primary:hover { color: #fff; opacity: 0.9; }
 }
 
 .commit-message .patch-headers {
-  margin: 0 0 0.5rem;
-  font: 12px/1.5 ui-monospace, "Cascadia Mono", "Source Code Pro", monospace;
+  margin: 0 0 0.4rem;
+  font-family: ui-monospace, "Cascadia Mono", "Source Code Pro", monospace;
+  color: var(--fg);
 }
 .commit-message .patch-headers dt {
   display: inline;
-  color: var(--muted);
+  color: var(--fg);
   margin-right: 0.3em;
 }
 .commit-message .patch-headers dd {
   display: inline;
   margin: 0;
+  color: var(--fg);
 }
 .commit-message .patch-headers code {
-  font-family: ui-monospace, monospace;
+  font-family: inherit;
   color: var(--fg);
 }
 .commit-message .patch-headers > dt + dd::after { content: "\A"; white-space: pre; }
 
-.commit-message .commit-message-body { white-space: pre-wrap; }
-
 .file {
   margin-bottom: 1rem;
   border: 1px solid var(--border);
diff --git a/extension/review/review.js b/extension/review/review.js
@@ -129,7 +129,6 @@ function renderPatch() {
   // follows below.
   const headers = document.createElement("dl");
   headers.className = "patch-headers";
-  headerRow("From", patch.author || "", headers);
   headerRow("Subject", patch.subject || "", headers);
   msg.appendChild(headers);