On the releases page there is a sidebar that needs a better design.

crueber commented

2026-09-04 16:29:49 +00:00

Owner

Here is the design that it is currently. Right now it looks pretty bare bones. Make it look better. Make sure you check when it has something in it too.

![image](/attachments/d16265d2-7293-4b24-a1c7-083614c17a80) Here is the design that it is currently. Right now it looks pretty bare bones. Make it look better. Make sure you check when it has something in it too.

image.png

5.9 KiB

crueber commented

2026-09-04 17:53:28 +00:00

Author

Owner

Starting work on fix/issue-35 (branch fix/issue-35 from origin/main, scratch worktree — main worktree left untouched). Plan: redesign the Latest sidebar as a latest-release card (tag, name, date, badges, key assets with download links, link to release) + shared Empty callout when none; main list untouched, no backend changes.

crueber referenced this issue from a commit

2026-09-04 17:59:55 +00:00

Fix #35: latest-release sidebar card + compact empty state (12_web_ui.md decisions)

crueber referenced this issue from a pull request that will close it,

2026-09-04 17:59:59 +00:00

Fix #35: releases latest-panel design #47

crueber commented

2026-09-04 18:00:03 +00:00

Author

Owner

Fixed by #47 (branch fix/issue-35) — Latest sidebar is now a release card (tag/badges/date/key asset downloads/detail links) or the shared Empty callout when none; verified empty + with-releases in both themes, zero JS errors. No backend changes; note: latest prereleases/drafts never reach the panel because the page calls latest without include_prereleases (badges kept for correctness).

Fixed by https://git.packden.us/crueber/walhub/pulls/47 (branch fix/issue-35) — Latest sidebar is now a release card (tag/badges/date/key asset downloads/detail links) or the shared Empty callout when none; verified empty + with-releases in both themes, zero JS errors. No backend changes; note: latest prereleases/drafts never reach the panel because the page calls latest without include_prereleases (badges kept for correctness).

crueber commented

2026-09-04 18:02:06 +00:00

Author

Owner

Review of PR #47 (775fefa, fix/issue-35) — latest-release panel redesign. Verified in scratch worktree /tmp/pr47 (since removed): full node suite 206/206 pass (incl. 5 new keyAssets cases), vite+esbuild bundle build clean. Findings, all checked, no blocking issues:

Panel composition (Releases.jsx:107-186): tag router-link, ReleaseBadges (draft/prerelease), name (hidden when empty), Published time with dateTime=ISO, first-3 asset links with fmtBytes sizes, +N-more and view-release detail links. Matches spec.
Loading vs empty (Releases.jsx:108-109): getLatest() starts undefined -> 'loading latest…' role=status; 404 maps to null -> compact Empty; non-404 errors stay on loading + tray error. No flash of wrong state. Main list untouched.
Empty compact (Empty.jsx:44, ui.css:89-92): classList merge only adds .empty-state-compact when compact is truthy; pulls/checks/issues call sites pass no compact prop -> unaffected.
keyAssets (lib/releases.js:15-19): first-limit shown + remainder extra; null/non-array -> empty; non-positive/non-finite -> all extra; fractional floors. 5 unit tests pin this (releases-panel.test.js).
Download links (Releases.jsx:160-168): plain (not router A) to repoClient.releaseAssetUrl + download attr + title tooltip — correct, same-origin static bytes.
No backend change needed — confirmed: internal/releases/http.go latestRelease serves wireRelease with full assets array, and 404-on-empty is already mapped client-side.
No new deps (solid-js, @solidjs/router, relative imports only); Tailwind @apply in existing ui.css layer. Dark+light via emerald-700/dark:emerald-400, muted, zinc borders — matches palette.
Keyboard/focus: all links are native anchors / router links under the global :focus-visible ring; no extra tab stops.
Doc decision (12_web_ui.md): accurate — describes card contents, keyAssets helper, loading status, compact Empty, and the no-backend-change rationale.

No fixes required. Recommendation: ready to merge.

Review of PR #47 (775fefa, fix/issue-35) — latest-release panel redesign. Verified in scratch worktree /tmp/pr47 (since removed): full node suite 206/206 pass (incl. 5 new keyAssets cases), vite+esbuild bundle build clean. Findings, all checked, no blocking issues: - Panel composition (Releases.jsx:107-186): tag router-link, ReleaseBadges (draft/prerelease), name (hidden when empty), Published time with dateTime=ISO, first-3 asset links with fmtBytes sizes, +N-more and view-release detail links. Matches spec. - Loading vs empty (Releases.jsx:108-109): getLatest() starts undefined -> 'loading latest…' role=status; 404 maps to null -> compact Empty; non-404 errors stay on loading + tray error. No flash of wrong state. Main list untouched. - Empty compact (Empty.jsx:44, ui.css:89-92): classList merge only adds .empty-state-compact when compact is truthy; pulls/checks/issues call sites pass no compact prop -> unaffected. - keyAssets (lib/releases.js:15-19): first-limit shown + remainder extra; null/non-array -> empty; non-positive/non-finite -> all extra; fractional floors. 5 unit tests pin this (releases-panel.test.js). - Download links (Releases.jsx:160-168): plain <a> (not router A) to repoClient.releaseAssetUrl + download attr + title tooltip — correct, same-origin static bytes. - No backend change needed — confirmed: internal/releases/http.go latestRelease serves wireRelease with full assets array, and 404-on-empty is already mapped client-side. - No new deps (solid-js, @solidjs/router, relative imports only); Tailwind @apply in existing ui.css layer. Dark+light via emerald-700/dark:emerald-400, muted, zinc borders — matches palette. - Keyboard/focus: all links are native anchors / router links under the global :focus-visible ring; no extra tab stops. - Doc decision (12_web_ui.md): accurate — describes card contents, keyAssets helper, loading status, compact Empty, and the no-backend-change rationale. No fixes required. Recommendation: ready to merge.

crueber closed this issue

2026-09-04 18:02:26 +00:00

crueber commented

2026-09-04 18:02:26 +00:00

Author

Owner

Fixed by PR #47 (review clean; 206/206 node tests), merged. Closing.

crueber referenced this issue from a commit

2026-09-04 18:02:26 +00:00

Merge pull request 'Fix #35: releases latest-panel design' (#47) from fix/issue-35 into main

crueber referenced this issue

2026-09-04 22:16:25 +00:00