tjue/kokosbananas
0
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity

UI tweaks #10

Closed
arne wants to merge 4 commits from ui-tweaks into main
pull from: ui-tweaks
merge into: tjue:main
Conversation 0 Commits 4 Files changed 13 +285 -232
arne commented 2026-06-26 12:15:09 +02:00
Owner
Copy link

Small styling fixes on the feed-generation show page, surfaced in review. Sibling of #9 (both based on catalog-feed-generation, independent). Each matches the existing design system; quick to eyeball.

  • Explicit checkbox stacked vertically — used class="row", but label is flex-direction:column and .row doesn't override it, so the checkbox sat above its label text. Switched to the purpose-built .check class (row + centered).
  • Dead feed-card accent — --accent:var(--sky) was set on a plain .card, which never reads --accent, so the sky tint never rendered. Added card--wash to match the rendition-card pattern (partials.html).
  • Feed cards in a column-less grid — bare .grid rendered one card per row; added an auto-fit minmax template so platforms sit side by side and stack on narrow viewports.
  • Border-radius used a spacing token — cover-art thumbnail had border-radius:var(--s2); switched to the radius token --r-sm.

Build + go test ./internal/web green (templates parse + render).

🤖 Generated with Claude Code

Small styling fixes on the feed-generation show page, surfaced in review. Sibling of #9 (both based on `catalog-feed-generation`, independent). Each matches the existing design system; quick to eyeball. - **Explicit checkbox stacked vertically** — used `class="row"`, but `label` is `flex-direction:column` and `.row` doesn't override it, so the checkbox sat *above* its label text. Switched to the purpose-built `.check` class (row + centered). - **Dead feed-card accent** — `--accent:var(--sky)` was set on a plain `.card`, which never reads `--accent`, so the sky tint never rendered. Added `card--wash` to match the rendition-card pattern (`partials.html`). - **Feed cards in a column-less grid** — bare `.grid` rendered one card per row; added an auto-fit `minmax` template so platforms sit side by side and stack on narrow viewports. - **Border-radius used a spacing token** — cover-art thumbnail had `border-radius:var(--s2)`; switched to the radius token `--r-sm`. Build + `go test ./internal/web` green (templates parse + render). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Four small fixes to the feed-generation show page, matching the
established design system:

- Explicit-content checkbox used class="row", but `label` is
  flex-direction:column and `.row` doesn't override it, so the box stacked
  above its text. Use the purpose-built `.check` class (row + centered).
- Feed cards set `--accent:var(--sky)` on a plain `.card`, which never
  reads `--accent` — the sky tint never rendered. Add `card--wash` to
  match the rendition-card pattern (partials.html), so the accent shows.
- Feed cards sat in a column-less `.grid` (one per row). Give the grid an
  auto-fit minmax template so platforms lay out side by side, stacking on
  narrow viewports.
- Cover-art thumbnail used a spacing token (`--s2`) for border-radius;
  switch to the radius token `--r-sm`.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
arne changed target branch from catalog-feed-generation to main 2026-06-26 12:46:53 +02:00 
URL structure:
- Move the program/episode admin from /katalog/program/* to top-level
  /program/* (routes, redirects, templates, tests). Auth unchanged:
  selectiveAuth already gates any non-public prefix.

Landing:
- Add a "Programmer" widget listing all shows, linking to each directly
  (best-effort; a store error just hides the list).

Navbar (global, via the head partial):
- Show the current show's title as a clickable pill (links back to the
  show); add a distinct grape pill for the episode title when in scope.
- Drop the now-redundant per-handler Tag wiring for show-scoped pages.

Templates:
- Drop heroes/breadcrumbs on show, episode, and "ny episode" pages; the
  navbar pills carry that context now.
- Move cover-art upload from the show page to the (edit-mode) program
  form; the show page now opens on the action buttons.
- Episode sections: drop the "N · " prefixes; rename "Renditioner" ->
  "Renderinger" (display text only).

Credits:
- Translate roles to Norwegian (display-only map; DB keys/feeds unchanged).
- Add a reusable credit_line component; render credits as dense pills.
- Collapse the work/edition split (level is derived from role server-side)
  and put add/remove behind a no-JS "Rediger" <details> disclosure.

Forms:
- Auto-grow textareas via CSS field-sizing, with a tiny JS fallback for
  browsers without it (e.g. Firefox).
- Apply input styling to email/url/date fields that were missed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Credits:
- Make the whole person pill a link (was just the name) with a single
  hover state across the pill.
- Replace the read-only credit view in place when the "Rediger" disclosure
  opens (CSS :has), and order the toggle below its panel so the credits
  stay above the edit button.

Feeds:
- Feed cards span the full line (single-column grid) instead of two-up.

Copy-to-clipboard:
- navigator.clipboard only exists in a secure context, so it failed over
  plain HTTP on iOS. Fall back to a hidden-textarea + execCommand with the
  selection dance iOS needs; keep the async API on HTTPS.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
navigator.clipboard works on iPad over HTTPS, which is how production is
served, so the hidden-textarea/execCommand fallback isn't needed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
arne referenced this pull request from a commit 2026-06-28 22:32:09 +02:00
UI changes (#13)
arne closed this pull request 2026-06-28 22:32:15 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
tjue/kokosbananas!10
No description provided.