QA of PR #179 on palette-dev failed: paste-lines.js splits .codeline spans correctly, but app.css had no .gutline rule, so gutter number spans rendered inline (all at the same top) and numbers 6+ fell off when wrap was enabled.
Fix: .gutter .gutline { display: block; } — one gutter number per visual wrapped row, block-level per gutline.
Verified in a local build (headless chromium, wrap forced ON, long unbroken token on line 5):
1400x900: 20 gutlines for 16 logical lines (line 5 wraps into 5 rows), all display: block, strictly increasing tops, 36.7px each, zero gutter clipping, no horizontal scroll.
375x812: 23 gutlines (line 5 wraps into 8 rows), all block, strictly increasing, zero clipping, no horizontal scroll on document or code column.
go build ./... and go test ./... pass. Closes#167.
QA of PR #179 on palette-dev failed: paste-lines.js splits .codeline spans correctly, but app.css had no .gutline rule, so gutter number spans rendered inline (all at the same top) and numbers 6+ fell off when wrap was enabled.
Fix: `.gutter .gutline { display: block; }` — one gutter number per visual wrapped row, block-level per gutline.
Verified in a local build (headless chromium, wrap forced ON, long unbroken token on line 5):
- 1400x900: 20 gutlines for 16 logical lines (line 5 wraps into 5 rows), all `display: block`, strictly increasing tops, 36.7px each, zero gutter clipping, no horizontal scroll.
- 375x812: 23 gutlines (line 5 wraps into 8 rows), all block, strictly increasing, zero clipping, no horizontal scroll on document or code column.
`go build ./...` and `go test ./...` pass. Closes #167.
QA on dev showed .codeline spans wrap correctly but gutter .gutline spans
rendered inline (all at the same top), so numbers 6+ fell off with wrap on.
Add .gutter .gutline { display: block; } so each gutter number occupies its
own visual row, matching the one-number-per-visual-row contract of
paste-lines.js. Verified rendered output in a local build at 1400x900 and
375x812 with a long unbroken token on line 5 and wrap ON.
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.
QA of PR #179 on palette-dev failed: paste-lines.js splits .codeline spans correctly, but app.css had no .gutline rule, so gutter number spans rendered inline (all at the same top) and numbers 6+ fell off when wrap was enabled.
Fix:
.gutter .gutline { display: block; }— one gutter number per visual wrapped row, block-level per gutline.Verified in a local build (headless chromium, wrap forced ON, long unbroken token on line 5):
display: block, strictly increasing tops, 36.7px each, zero gutter clipping, no horizontal scroll.go build ./...andgo test ./...pass. Closes #167.QA on dev showed .codeline spans wrap correctly but gutter .gutline spans rendered inline (all at the same top), so numbers 6+ fell off with wrap on. Add .gutter .gutline { display: block; } so each gutter number occupies its own visual row, matching the one-number-per-visual-row contract of paste-lines.js. Verified rendered output in a local build at 1400x900 and 375x812 with a long unbroken token on line 5 and wrap ON.fen referenced this pull request2026-09-10 17:13:14 +00:00
Pull request closed