QA evidence: paste-lines.js correctly creates .gutline spans, but no CSS rule existed anywhere, so the spans rendered inline and all gutter numbers overlapped on one visual row (all children at top=280.6px, height 25px vs line-height 36.72px); gutter count (16) also mismatched rendered visual rows (19).
Fix: add .code .gutter .gutline { display: block; } to internal/web/static/app.css so number and blank fill spans stack one per visual row. Wrap-off textContent path unchanged.
Local: go build ./... and go test ./... green. QA to re-verify on palette-dev with test paste 4t4h22.
Revision of #167 after QA failed PR #179 (commit 0643ca4).
QA evidence: paste-lines.js correctly creates .gutline spans, but no CSS rule existed anywhere, so the spans rendered inline and all gutter numbers overlapped on one visual row (all children at top=280.6px, height 25px vs line-height 36.72px); gutter count (16) also mismatched rendered visual rows (19).
Fix: add `.code .gutter .gutline { display: block; }` to internal/web/static/app.css so number and blank fill spans stack one per visual row. Wrap-off textContent path unchanged.
Local: go build ./... and go test ./... green. QA to re-verify on palette-dev with test paste 4t4h22.
QA of PR #179 found paste-lines.js creates .gutline spans but no CSS rule
existed, so all gutter numbers rendered inline on one visual row. Stack
them one per row under .code .gutter; blank fill spans inherit it.
fen
merged commit d402113f19 into dev2026-09-10 16:37:57 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Revision of #167 after QA failed PR #179 (commit
0643ca4).QA evidence: paste-lines.js correctly creates .gutline spans, but no CSS rule existed anywhere, so the spans rendered inline and all gutter numbers overlapped on one visual row (all children at top=280.6px, height 25px vs line-height 36.72px); gutter count (16) also mismatched rendered visual rows (19).
Fix: add
.code .gutter .gutline { display: block; }to internal/web/static/app.css so number and blank fill spans stack one per visual row. Wrap-off textContent path unchanged.Local: go build ./... and go test ./... green. QA to re-verify on palette-dev with test paste 4t4h22.