Panels, after looking: a lone review group open, phone downloads #17

merged cat/panels-polish → main, opened by cat, 1 revision, updated Oct 19

commits1oldest firstcombined diff 3 files, +9 -3

1/1

Open a lone review group, drop insights' second title, restack downloads

ae9b981

-3

✓

wrap lines

discussion0 comments

No comments yet: forge comment 17 adds one.

1/1 Open a lone review group, drop insights' second title, restack downloads

cat, Oct 19, ae9b981. CI ✓ passed in 111s, signed by cat

A repo whose reviews are all merged showed one folded panel and nothing
else; a group that is all there is starts open. A repo's insights page
said "insights" again under its insights tab. And on a phone a release's
sha256 sums had a column three characters wide: a file's name and size
share a line there now, its sum the width under them.

3 files changed, +9 -3

M	site/repo/insights.njk	1	-
M	site/repo/releases.njk	7	+++++++
M	site/repo/reviews.njk	4	++--

M site/repo/insights.njk +0 -1

@@ -7,6 +7,5 @@ pagination:
77 {% import "ui.njk" as ui %}88 {% block scripts %}<script src="{{ assets.insights }}" defer></script>{% endblock %}99 {% block content %}10-{{ ui.heading('insights') }}1110 {% include "insights-body.njk" %}1211 {% endblock %}

M site/repo/releases.njk +7 -0

@@ -15,6 +15,13 @@ pagination:
1515 .downloads td.size { white-space: nowrap; text-align: right; font: var(--fw-regular) var(--text-xs) var(--mono); }1616 .downloads td.sum { font-size: var(--text-xs); overflow-wrap: anywhere; width: 100%; }1717 .downloads p { margin: var(--space-2) 0 var(--space-1); font-size: var(--text-sm); }18+/* Phones: a file's name and size on a line, its sha256 the width under them. */19+@media (max-width: 720px) {20+	.downloads tr { display: grid; grid-template-columns: 1fr auto; column-gap: var(--space-3); padding: var(--space-1) 0; }21+	.downloads td { padding: 0; }22+	.downloads td.file { white-space: normal; overflow-wrap: anywhere; }23+	.downloads td.sum { grid-column: 1 / -1; }24+}1825 {% endcss %}1926 {# Release notes: each tag's commits since the tag before it, under the2027    section Jev sorted them into (lib/jev-commits.ts). The wording is the

M site/repo/reviews.njk +2 -2

@@ -7,7 +7,7 @@ pagination:
77 {% import "ui.njk" as ui %}88 {% block content %}99 {# Reviews by status: a panel per group, folding. Long merged and closed10-   groups start folded. #}10+   groups start folded, unless one is all there is. #}1111 {% if not p.groups.length %}1212 <div class="panel"><div class="markdown">1313 <p>No reviews yet. Push a branch and it becomes one (branches imported with the repo don't):</p>@@ -19,7 +19,7 @@ forge push -t "Title"   # the branch and its CI results</pre>
1919 {% else %}2020 {% for g in p.groups %}2121 {% set head %}<h2>{{ g.status }}</h2><span class="n">{{ g.items.length }}</span>{% endset %}22-{% call ui.panel(head, 'rv-group ' + g.status, '', 'open' if (g.status == 'open' or g.status == 'draft' or g.items.length <= 10) else 'closed') %}{{ m.reviews(g.items, p.repo) }}{% endcall %}22+{% call ui.panel(head, 'rv-group ' + g.status, '', 'open' if (g.status == 'open' or g.status == 'draft' or g.items.length <= 10 or p.groups.length == 1) else 'closed') %}{{ m.reviews(g.items, p.repo) }}{% endcall %}2323 {% endfor %}2424 {% endif %}2525 {% endblock %}