Skip to content

Jagged table data - #318

Merged
andrew merged 1 commit into
git-pkgs:mainfrom
kokes:kokes/table-alignment
Sep 6, 2026
Merged

Jagged table data#318
andrew merged 1 commit into
git-pkgs:mainfrom
kokes:kokes/table-alignment

Conversation

@kokes

@kokes kokes commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

I noticed two issues in the main dashboard:

  1. the cache hits were not aligned vertically, so it was hard to read them
  2. the version specifications would flow into the 'x minutes ago' bit - making it hard to read
image

Here it is with the PR applied:

image

@kokes
kokes marked this pull request as ready for review September 4, 2026 15:57
@andrew
andrew requested a lite review from Copilot September 5, 2026 08:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The change is limited to template layout classes and addresses the reported UI issues without altering server-side behavior.

Pull request overview

Improves the main dashboard’s package lists layout to make numeric metadata easier to scan and to prevent long version strings from visually colliding with the “time ago”/size metadata.

Changes:

  • Aligns “cache hits” and “size” in Popular Packages using a fixed two-column grid with tabular numerals.
  • Prevents long @version strings in Recently Cached from flowing into the right-side metadata by constraining/truncating the left-side row content and adding spacing.
  • Ensures the two-column dashboard cards align from the top (items-start) to avoid uneven vertical alignment.
File summaries
File Description
internal/server/templates/pages/dashboard.html Adjusts Tailwind layout classes to align numeric columns and prevent version/metadata overlap in dashboard lists.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@andrew

andrew commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Thanks!

@andrew
andrew merged commit 94c11b4 into git-pkgs:main Sep 6, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants