~ 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 long repository name on a phone starts its first line with the slash

Merged
Merged chadtmiller.com asked to merge into main from chadtmiller.com/knotless:web/repo-name-slash

What changes

On a phone, a long repository name now starts its first line with the /, as in / a-rather-long-repository-name-, and wraps the rest under it. Before, the slash stood alone at the left edge, centred beside the wrapped name. The slash and the name are now one run of text in the header, with the caret beside them as before. A name with no place to break wraps anywhere rather than running off the page.

A short name looks the same as it did.

Why

The phone fix for long names kept the slash with the name, but as a flex item of its own. So when the name wrapped, the slash stayed beside the whole block instead of starting its first line.

How it was checked

just look of the repository page on a phone, with the name set to a-rather-long-repository-name-for-phones. The pictures below are before and after. A short name was checked in desktop light and phone dark, and it is unchanged. The shots check compares every recorded page.

Shortcomings

The caret still sits at the right edge, centred against the two lines, when the name wraps.

🤖 Generated with Claude Code

long-name-whole-page.png

slash-long-after.png

Asked for changes

This wraps the slash and the repo name in one .repo-path span, so the slash starts the first line when a long name wraps on a phone. The CSS looks right to me.

The body has no Shots-moved: line, and I expect shots to fail without one. RepoHeader.svelte adds a span to the HTML of every page that shows the repo header, even when a short name looks the same. The shots check compares HTML, not pixels, so it will list those pages as moved. AGENTS.md says the check fails a pull that moves pages without naming them on a Shots-moved: line. Once shots finishes, add that line with the page names it reports, or all, then rerun the shots workflow. Editing the body alone doesn't move the check.

At the time of review, check, web, shots, test and e2e hadn't finished, so I haven't seen them pass.

Merged as cc646ab