3a83161b1c
One commit (`af07314`), three strands of browser-UI work against one design system. `docs/design-system.md` was updated to match the CSS, not the reverse. ## Library (Reader-facing) Findings came out of a two-axis design review of the library surface; the fixes are the P1/P2 set plus the cheap P3s. - **`.chrome` sticks at `top: 0`.** Search and the tab row were unreachable three screens into a 300-item library — exactly where they earn their keep. Everything above them (`.topbar`, `.keyrow`, `.recent`) still scrolls away on purpose: another 150px of permanent chrome on an 844px phone costs more than re-scrolling for an icon reminder. - **One `:focus-visible` ring** (`2px solid var(--paper)`, offset 2px) on the nine controls that defined none and fell back to the UA blue — a colour tuned for neither branch of this palette. `.searchbar` keeps its `:focus-within` border recolour as a resting cue but no longer stands in for the ring. - **Mono labels lift 10px → 11px** everywhere (nine rules). PRODUCT.md names night reading and glare as the usage scene; 10px small-caps was the one place taste overrode the brief. 11px is now a documented floor. - **A card in flight past 2s says `Saving…` and carries `aria-busy`.** htmx sets neither, so the wait — up to its own 15s timeout, and this app is used on a phone in dead zones — was silent in both the visual and the assistive channel. Deliberately `--mute`, not `--ember`: ember means "new chapter" and nothing else. - **Titles clamp at 3 lines**; `.is-new .title` takes `width: fit-content`, or `-webkit-box` stretches the ember underline past the text it is supposed to be sized to. - `.libswitch a` reaches a real 44px under `(pointer: coarse)` — padding plus an 11px line landed at 43. ## /admin - Overview routes into Lanes when a lane is unhealthy, prefixes each figure with its column word on the phone layout that drops the `thead`, labels state cells for a screen reader, and has an empty state where the sites table previously assumed rows. - The admin shell picks up the library's chrome: htmx 15s timeout, the shared `#notice` slot, `#sr-announce`, `filter.js`. - `admin_render_test.go` and `card_render_test.go` render the templates directly, so markup regressions in either surface fail without a browser. ## Login `DISCORD_GUILD_NAME` (optional) names the community on the login screen and in the refusal message, so a stranger knows which Discord to ask for an invite. Unset degrades to a generic label. Neither form names the numeric guild id — that was never actionable, and the gate still reveals nothing about whether a given guild exists. ## Handlers `maxChapterNum` (9999) now bounds **both** typed-chapter paths. `uiChapter` and `adminSeriesCorrectLatest` each parsed a `float64` with no ceiling, so a hand-rolled POST stored `1e308` and every later reader of that row — the poller's `HasNewChapter` comparison, the display string — inherited it. Matches the `max` on the card's chapter input. The API PUT path is deliberately untouched: it carries the userscripts' own scraped numbers, not typed input. ## Verification - `cd backend && go test ./...` green (Docker-backed `pgtest`). `TestChapterOverrideRejectsBadInput` gained `"10000"` and `"1e5"` — both parse fine as `float64`, so they only fail if the bound exists. - Visual: 390×844 dark + light, 1000px and 1440px (`zoom: 1.2`) desktop, against the real templates + real CSS. Measured `chromeTop = 0` at `scrollY 950`, `2px solid rgb(242,236,229)` rings, `content: "Saving…"` at `opacity: 1` after 2.4s, `aria-busy` `true` during / cleared after, `libswitchH = 44` in a `hasTouch` context, no horizontal overflow at either width. - `detect.mjs` on `templates/`: `[]`, exit 0. ## Note on shape The three strands landed as one commit because `admin_series.go` and `style.css` each carry hunks from more than one of them; splitting cleanly would have needed hunk-level surgery. Say the word if you want it split before merge. Reviewed-on: #177 Co-authored-by: Sulthan Zaki <sultankiki05@gmail.com> Co-committed-by: Sulthan Zaki <sultankiki05@gmail.com>
107 lines
6.0 KiB
HTML
107 lines
6.0 KiB
HTML
{{/* Per-Series page: one address per Series, keyed "<site>:<series_id>" so the
|
|
list row is one hop from it. Everything here is a Series-level fact plus
|
|
the anonymous Reader count. Check now lands in its own .dform below the
|
|
.detail-grid; the correction form is the grid's first column and the URL
|
|
repair the second (issue #151). The pending and corrected markers ride the
|
|
meta line with the other marks. */}}
|
|
{{define "series-detail"}}
|
|
<a class="ghost detail-back" href="/admin/series">← Series</a>
|
|
<h1 class="detail-title">{{.Title}}</h1>
|
|
<p class="detail-key">{{.Key}} · {{.Site}} · {{.Kind}}</p>
|
|
{{if .Cover}}<div class="cover"><img src="{{.Cover}}" alt="" loading="lazy"></div>
|
|
{{else}}<div class="cover" aria-hidden="true"><span>no cover</span></div>{{end}}
|
|
{{template "series-detail-meta" .}}
|
|
<div class="detail-grid">
|
|
<form class="dform" hx-post="/admin/series/{{.Key}}/latest" hx-target="#detail-meta" hx-swap="outerHTML" hx-indicator="closest .dform" hx-disabled-elt="find button, input">
|
|
<h3>Correct latest chapter</h3>
|
|
<div class="field">
|
|
<label class="sr-only" for="chapter-{{.Key}}">Latest chapter number</label>
|
|
<input id="chapter-{{.Key}}" type="number" name="chapter" step="any" placeholder="{{.Chapter}}" required aria-describedby="hint-latest-{{.Key}}">
|
|
<button type="submit" class="ghost">Save chapter</button>
|
|
</div>
|
|
<p class="hint" id="hint-latest-{{.Key}}">The next successful Poll overwrites this value. Use this only when the Poll is failing or the page is wrong.</p>
|
|
{{if .Unverified}}<p class="hint">{{.Unverified}}</p>{{end}}
|
|
<p class="error-inline" role="status" hidden></p>
|
|
</form>
|
|
<form class="dform" hx-post="/admin/series/{{.Key}}/series-url" hx-target="#detail-meta" hx-swap="outerHTML" hx-indicator="closest .dform" hx-disabled-elt="find button, input">
|
|
<h3>Repair series URL</h3>
|
|
<div class="field">
|
|
<label class="sr-only" for="url-{{.Key}}">Series page URL</label>
|
|
<input id="url-{{.Key}}" type="url" name="series_url" value="{{.URL}}" required aria-describedby="hint-url-{{.Key}}">
|
|
<button type="submit" class="ghost">Save URL</button>
|
|
</div>
|
|
<p class="hint" id="hint-url-{{.Key}}">The Poll fetches this address. Saving does not verify it — A Site-wide host change is a SQL migration, not per-series edits.</p>
|
|
<p class="error-inline" role="status" hidden></p>
|
|
</form>
|
|
</div>
|
|
{{if .CanPoll}}
|
|
<div class="dform">
|
|
<div class="field"><button type="button" class="ghost"
|
|
hx-post="/admin/series/{{.Key}}/poll" hx-target="#detail-meta" hx-swap="outerHTML"
|
|
hx-indicator="#detail-meta" hx-disabled-elt="this">Check now</button></div>
|
|
<p class="error-inline" role="status" hidden></p>
|
|
</div>
|
|
{{end}}
|
|
{{if .CanRemove}}
|
|
<div class="dform">
|
|
<div class="field"><button type="button" class="ghost danger"
|
|
aria-expanded="false" aria-controls="confirm-remove"
|
|
onclick="document.getElementById('confirm-remove').hidden = false; this.setAttribute('aria-expanded','true')">Remove</button></div>
|
|
<div class="confirm-row" id="confirm-remove" role="group" aria-live="polite" hidden>
|
|
<span>Remove “{{.Title}}”? Stored cover is lost.</span>
|
|
<div>
|
|
<button type="button" class="danger-solid"
|
|
hx-post="/admin/series/{{.Key}}/remove" hx-target="#detail-meta"
|
|
hx-indicator="#detail-meta" hx-disabled-elt="this">Remove</button>
|
|
<button type="button" onclick="document.getElementById('confirm-remove').hidden = true; document.querySelector('.dform .danger').setAttribute('aria-expanded','false')">Cancel</button>
|
|
</div>
|
|
</div>
|
|
<p class="error-inline" role="status" hidden></p>
|
|
</div>
|
|
{{end}}
|
|
{{if .Finished}}
|
|
<div class="dform">
|
|
<div class="field"><button type="button" class="ghost"
|
|
hx-post="/admin/series/{{.Key}}/unfinish" hx-target="#detail-meta" hx-swap="outerHTML"
|
|
hx-indicator="#detail-meta" hx-disabled-elt="this">Un-finish</button></div>
|
|
<p class="error-inline" role="status" hidden></p>
|
|
</div>
|
|
{{else}}
|
|
<div class="dform">
|
|
<div class="field"><button type="button" class="ghost" aria-expanded="false" aria-controls="confirm-finish"
|
|
onclick="document.getElementById('confirm-finish').hidden = false; this.setAttribute('aria-expanded','true')">Finish</button></div>
|
|
{{if .SiteCompleted}}<p class="hint">{{.SiteCompleted}}</p>{{end}}
|
|
<div class="confirm-row calm" id="confirm-finish" role="group" aria-live="polite" hidden>
|
|
<span>Mark this Series finished?</span>
|
|
<div>
|
|
<button type="button" class="go" hx-post="/admin/series/{{.Key}}/finish" hx-target="#detail-meta" hx-swap="outerHTML"
|
|
hx-indicator="#detail-meta" hx-disabled-elt="this">Finish</button>
|
|
<button type="button" onclick="document.getElementById('confirm-finish').hidden = true; document.querySelector('[aria-controls=confirm-finish]').setAttribute('aria-expanded','false')">Cancel</button>
|
|
</div>
|
|
</div>
|
|
<p class="error-inline" role="status" hidden></p>
|
|
</div>
|
|
{{end}}
|
|
{{end}}
|
|
|
|
{{/* series-detail-meta is the meta line, and the answer a Check now or
|
|
correction press on the detail page swaps into its place: the same marks,
|
|
re-rendered after the stamp so the pending and corrected markers — and
|
|
the provenance line beside the number — describe the value they sit
|
|
next to. */}}
|
|
{{define "series-detail-meta"}}
|
|
<div class="detail-meta" id="detail-meta" aria-live="polite">
|
|
<span>ch {{.Chapter}}</span>
|
|
{{if .Provenance}}<span>{{.Provenance}}</span>{{end}}
|
|
<span>checked {{.Checked}}</span>
|
|
<span>{{.Readers}} reader{{if ne .Readers 1}}s{{end}}</span>
|
|
{{if .Corrected}}<span class="mark">{{.Corrected}}</span>{{end}}
|
|
{{if .Pending}}<span class="mark">{{.Requested}}</span>{{end}}
|
|
{{if .FinishedSince}}<span class="mark">{{.FinishedSince}}</span>{{end}}
|
|
{{if .Unpollable}}<span class="mark">unpollable</span>{{end}}
|
|
{{if .NoCover}}<span class="mark">no cover</span>{{end}}
|
|
{{if .Orphan}}<span class="mark">orphan</span>{{end}}
|
|
{{if .SightingRaised}}<span class="mark">sighting-raised</span>{{end}}
|
|
</div>
|
|
{{end}}
|