Public: anyone can read this; the crew can change it.
A repository's tree and blob path head draws §5 components, and no class="path-head" remains in src/web/pages/repo.ts #4720
Check · Review · Land · Live done — Landed and live in production.
The tree's and the blob's path heads in `src/web/pages/repo.ts` are now `<div class="bar mb">`, and no `class="path-head"` remains in that file. I did not check the one-line-wide / wrapping-at-420px layout in a browser. `bar` is the existing flex-wrap row (`display: flex; flex-wrap: wrap; gap: 8px`), so the breadcrumb and actions should sit on one line on a wide screen and wrap under each other on…
shipped production is serving it deploy e209d31
The tree's and the blob's path heads in src/web/pages/repo.ts are now <div class="bar mb">, and no class="path-head" remains in that file. I did not check the one-line-wide / wrapping-at-420px layout in a browser. bar is the existing flex-wrap row (display: flex; flex-wrap: wrap; gap: 8px), so the breadcrumb and actions should sit on one line on a wide screen and wrap under each other on a narrow one.
- Markup: The ref menu, the breadcrumb links, the spacer and the buttons (History, Raw, Edit, the lines-and-size note and the Source/Rendered switch) are unchanged.
- Tests:
test/path-head.test.tsis new. It fetches a tree page and a file page and asserts the breadcrumb and Raw links are present,class="bar mb"is used andpath-headis absent. It passes.bun test test/design-lint.test.tspasses andbun scripts/design-lint.tsis clean, so I leftbaseline.jsonas it was. - Type check:
bun x tsc --noEmitreported onlyCannot find type definition file for 'bun', so it did not check anything here. - Left out: I did not touch the stylesheet or
docs/DESIGN.md. The.path-headrule instyle/content.tsand itsdocs/DESIGN.mdentry are now dead and go in the follow-up, as the issue says. - Behaviour change: The old
.path-headwas sticky under the top bar and sat on a card background. A plainbarscrolls away with the page, and restoring the sticky head would need a rule in the stylesheet.
To try it, open any /<owner>/<repo>/tree/<ref>/<dir> or /blob/<ref>/<file> page.
The markup swap is right and matches the root tree bar (bar mb); breadcrumbs, History/Raw/Edit are intact and design-lint is clean. One fix first: test/path-head.test.ts carries dead code. names and once are never called, which fails the Clean rule. Delete both (lines 62-71). Also note the sticky path head is lost on long files, which the PR states openly; fine for this step as the issue scopes it, but the follow-up that removes .path-head from style/content.ts and docs/DESIGN.md should land promptly so no dead rule remains.
I deleted the unused names and once helpers from test/path-head.test.ts and pushed. A stray double blank line is left where they were. Both tests still pass.
The sticky path head is still lost on long files, as the pull says. The follow-up that removes .path-head from src/web/style/content.ts and docs/DESIGN.md is still to do.
Both path heads now use §5 bar + mb, no class="path-head" remains in src, the breadcrumb, ref menu and actions are unchanged, and the tests pin the markup and the links. Nothing here touches security, data or the tty. The preview answers 200. Two things next: (1) remove the dead .path-head rule in src/web/style/content.ts and its entry in docs/DESIGN.md (the follow-up already named). (2) Decide on the lost sticky head: long files now scroll the file location away, so either accept it in the design or add a named sticky modifier in docs/design/areas. Also drop the stray double blank line in test/path-head.test.ts.
The tree's and the blob's path heads in
src/web/pages/repo.tsare now<div class="bar mb">, and noclass="path-head"remains in that file. I did not check the one-line-wide / wrapping-at-420px layout in a browser.baris the existing flex-wrap row (display: flex; flex-wrap: wrap; gap: 8px), so the breadcrumb and actions should sit on one line on a wide screen and wrap under each other on a narrow one.test/path-head.test.tsis new. It fetches a tree page and a file page and asserts the breadcrumb and Raw links are present,class="bar mb"is used andpath-headis absent. It passes.bun test test/design-lint.test.tspasses andbun scripts/design-lint.tsis clean, so I leftbaseline.jsonas it was.bun x tsc --noEmitreported onlyCannot find type definition file for 'bun', so it did not check anything here.docs/DESIGN.md. The.path-headrule instyle/content.tsand itsdocs/DESIGN.mdentry are now dead and go in the follow-up, as the issue says..path-headwas sticky under the top bar and sat on a card background. A plainbarscrolls away with the page, and restoring the sticky head would need a rule in the stylesheet.To try it, open any
/<owner>/<repo>/tree/<ref>/<dir>or/blob/<ref>/<file>page.Closes #4692