~ knotless Search/
↑↓ move↵ open⇧↵ all results

chadtmiller.com/knotless

git that lives in your Atmosphere account

knotless.dev
1 collaborator

knotless 0.13.0 Latest 2 packages, 1 image

1
10

A repository's website shows only its address

Merged
Merged chadtmiller.com asked to merge into main from chadtmiller.com/knotless:web/site-link-tooltip

What changes

A repository's website, under its description, is its address alone: the commit it was deployed from and when ("from 9836c4c, 19 hours ago") are no longer shown. The site then sits in the line of facts beside the repository's links and releases, instead of taking a line or two of its own. The sites guide no longer says the page shows the deploy.

Why

Someone visiting a repository wants its site, not the commit it was built from; that matters to its owner, who deployed it and can read it with knotless repo site <url> or from the repository's checks. The line took as much room as the address, and wrapped onto its own line. A tooltip was tried and dropped: it shows on no phone, and could not keep the commit's link.

How it was checked

just look repository --element '.repohead .facts', with --set giving the repository a site: the address on one line with the links and releases, on desktop and on a phone (attached). No recorded page has a site, so no shots move.

Changeset: none

🤖 Generated with Claude Code

Approved

This moves the deploy commit and time from a second span beside the site link into the link's title, under the full address. RepoSite is now a single <a class="site-link repo-site">, so it fits in the .facts line. I checked the pieces and found nothing that needs changing:

  • ago(iso, 'on ') matches the helper's signature, so older deploys read "on <date>".
  • The repo prop is dropped from RepoSite and from its one call site in RepoHeader.
  • The CSS is reduced to the max-width rule the link still needs.
  • The sites guide describes the tooltip.

The PR body already says phones lose the commit and time. I only read the diff and ago in format.ts; I didn't run the page or look at the screenshots.

Approved

This drops the deploy commit and time from the site link, as the new title says. RepoSite keeps only the address, with title={site.url} so a cut-short address can still be read in full. The unused ago import and the derived title are gone, and the sites guide and the CSS comment match the new behaviour. I found nothing that needs changing. I only read the diff since my last approval; I didn't run the page.

Merged as 8f2a83e