Repository navigation
Identical text width and line breaks in the side-by-side view - #8
Merged
Merged
Conversation
…dentically Side by side, the new version's text column was up to 42px narrower than the old one: only the right clauses reserved the review gutter, only the right pane showed a scrollbar, and its divider took 1px. Unchanged paragraphs therefore wrapped differently on the two sides. Both sides now reserve the same gutter and scrollbar space (invisible on the left), and the divider is drawn without taking up width. A check in check.mjs guards these rules. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
3 of 4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Side by side, unchanged paragraphs could wrap differently on the two sides, because the new version's text column was up to 42px narrower:
Both sides now reserve the same gutter and scrollbar space (the left scrollbar is transparent; the panes scroll in sync), and the divider is an inset shadow.
check.mjsguards these rules.Row alignment (same top and height for every row) was already correct and is unchanged.
Test plan
python3 scripts/build.pyandnode scripts/check.mjs; the new check fails on the old CSS🤖 Generated with Claude Code