repos / git-pr

commit

163b995

parent

caaef17

author

Eric Bower

date

2025-12-02 08:59:38 -0500 EST

design: patchset tweaks

 1@@ -20,6 +20,10 @@ details {
 2   margin-bottom: 0;
 3 }
 4 
 5+.text-2xl {
 6+  text-transform: lowercase;
 7+}
 8+
 9 .pill-success {
10   border: 1px solid var(--success);
11   color: var(--success);
12@@ -60,12 +64,17 @@ details {
13   height: max-content;
14 }
15 
16+.mb-0 {
17+  margin-bottom: 0;
18+}
19+
20 .patch-file {
21   position: sticky;
22   top: 0;
23   left: 0;
24   padding: var(--grid-height) 0;
25   background-color: var(--bg-color);
26+  margin: 0;
27 }
28 
29 .details-min {
30@@ -74,6 +83,10 @@ details {
31   padding: 0;
32 }
33 
34+.chroma {
35+  margin: 0;
36+}
37+
38 .chroma .gi {
39   font-weight: normal !important;
40 }

 1@@ -2,7 +2,7 @@
 2 <div class="group">
 3   <div class="flex gap-2 collapse">
 4     <div class="group patchset-list" style="width: 350px;">
 5-      <h2 class="text-xl mt">
 6+      <h2 class="text-xl">
 7         Patchset <code>ps-{{.Patchset.ID}}</code>
 8       </h2>
 9 
10@@ -23,7 +23,7 @@
11           </div>
12 
13           {{range $patch.PatchFiles}}
14-            <div class="flex justify-between items-center">
15+            <div class="flex justify-between items-center text-sm">
16               {{if .NewName}}
17               <a class="flex-1 word-break-word mono" href="#patch-{{$patch.ID}}-{{.NewName}}">{{.NewName}}</a>
18               {{else}}
19@@ -42,26 +42,26 @@
20         </div>
21       {{end}}
22 
23-      <div><a href="#top">Back to top</a></div>
24+      <div><a href="#top" class="text-sm">Back to top</a></div>
25     </div>
26 
27     <div class="max-w flex-1">
28       {{range $patch := .Patches}}
29         <div class="group" id="{{$patch.Url}}">
30-          <h3 class="text-lg text-transform-none mono"><a href="#{{$patch.Url}}">{{$patch.Title}}</a></h3>
31+          <div class="box">
32+            <h3 class="text-lg text-transform-none mono mb-0"><a href="#{{$patch.Url}}">{{$patch.Title}}</a></h3>
33+          </div>
34 
35           {{if $patch.Body}}<pre class="w-full">{{$patch.Body}}</pre>{{end}}
36 
37           {{range $patch.PatchFiles}}
38             {{if .NewName}}
39             <details class="details-min" open="true" id="patch-{{$patch.ID}}-{{.NewName}}">
40-              <summary class="group-h patch-file">
41-                <span class="mono">{{.NewName}}</span>
42+              <summary class="patch-file">
43+                <code class="pill-success">+{{.Adds}}</code>
44+                <code class="pill-admin">-{{.Dels}}</code>
45+                <span class="mono ml">{{.NewName}}</span>
46                 <a href="#patch-{{$patch.ID}}-{{.NewName}}" class="word-break-word">link</a>
47-                <div class="flex gap">
48-                  <code class="pill-success">+{{.Adds}}</code>
49-                  <code class="pill-admin">-{{.Dels}}</code>
50-                </div>
51               </summary>
52               {{if .IsBinary}}
53               <div><pre>Binaries are not rendered as diffs.</pre></div>
54@@ -71,13 +71,11 @@
55             </details>
56             {{else}}
57             <details class="details-min" open="true" id="patch-{{$patch.ID}}-{{.OldName}}">
58-              <summary class="group-h patch-file">
59-                <span class="mono">{{.OldName}}</span>
60+              <summary class="patch-file">
61+                <code class="pill-success">+{{.Adds}}</code>
62+                <code class="pill-admin">-{{.Dels}}</code>
63+                <span class="mono ml">{{.OldName}}</span>
64                 <a href="#patch-{{$patch.ID}}-{{.OldName}}" class="word-break-word">link</a>
65-                <div class="flex gap">
66-                  <code class="pill-success">+{{.Adds}}</code>
67-                  <code class="pill-admin">-{{.Dels}}</code>
68-                </div>
69               </summary>
70               {{if .IsBinary}}
71               <div><pre>Binaries are not rendered as diffs.</pre></div>