poslop/palette
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

Fix #167: blank gutter rows match mobile code line height #201

Closed
fen wants to merge 1 commits from fix-167-r3 into dev
pull from: fix-167-r3
merge into: :dev
Conversation 0 Commits 1 Files Changed 1
+7
fen commented 2026-09-10 17:12:27 +00:00
Collaborator
Copy Link
Copy Source

Follow-up to the already-merged #167 fixes (PRs #190/#192/#193 + remote fix-167 2149fcc).

The mobile media query overrides .code font-size to 13px but left the gutter rows at the desktop metric, so on narrow viewports blank gutter rows stacked taller than code rows and every number drifted off its wrapped line start.

  • Blank gutter rows now take an explicit height matching the code line-height: calc(var(--code-fs) * var(--code-lh)) on desktop, calc(13px * 1.7) under the 640px media query.

Verified in headless chromium (stdlib CDP) at 375x812 and 1200x900:

  • wrapped long lines: gutter count == visual row count, numbers 1..n at each line start, per-row alignment < 2px, no horizontal scroll
  • wrap off / toggle on+off / short paste: unchanged behavior
Follow-up to the already-merged #167 fixes (PRs #190/#192/#193 + remote fix-167 2149fcc). The mobile media query overrides `.code` font-size to 13px but left the gutter rows at the desktop metric, so on narrow viewports blank gutter rows stacked taller than code rows and every number drifted off its wrapped line start. - Blank gutter rows now take an explicit height matching the code line-height: `calc(var(--code-fs) * var(--code-lh))` on desktop, `calc(13px * 1.7)` under the 640px media query. Verified in headless chromium (stdlib CDP) at 375x812 and 1200x900: - wrapped long lines: gutter count == visual row count, numbers 1..n at each line start, per-row alignment < 2px, no horizontal scroll - wrap off / toggle on+off / short paste: unchanged behavior
fen added 1 commit 2026-09-10 17:12:27 +00:00
Fix #167: mobile gutter row height follows mobile code font-size
CI / docker (pull_request) Skipped
Details
CI / test (pull_request) Successful in 45s
Details
9d87211eda 
The mobile media query overrides .code font-size to 13px, so the blank
gutter row height must match (13px x 1.7), not the desktop metric.
fen closed this pull request 2026-09-10 17:13:20 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Assigned
Owner assigned: agent may pick this up
 Critical
critical severity (pentest triage)
 High
high severity (pentest triage)
 In Progress
Agent actively working on this
 Low
low severity (pentest triage)
 Medium
medium severity (pentest triage)
 Owner Review
Blocked on a human decision or design call - poslop must review before work proceeds
 QA
Implementation done, awaiting independent review agent
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
fen poslop
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: poslop/palette#201