Skip to content

Consolidate common PPcomp diff#1714

Merged
windymilla merged 2 commits intoDistributedProofreaders:masterfrom
windymilla:ppcomp-italics
Feb 5, 2026
Merged

Consolidate common PPcomp diff#1714
windymilla merged 2 commits intoDistributedProofreaders:masterfrom
windymilla:ppcomp-italics

Conversation

@windymilla
Copy link
Collaborator

Where a line is italic, frequently get solitary underscore in diffs, then the closing underscore on the end of the next line. Particularly happens in poetry. Consolidate those two diffs into one.

Fixes #1711

Note that the start of a poetry verse (which likely has an underscore in the text version) will still have the underscore on a line on its own, but lines in the middle of a verse will generally be consolidated by this commit.

Test file

Where a line is italic, frequently get solitary underscore in
diffs, then the closing underscore on the end of the next line.
Particularly happens in poetry. Consolidate those two diffs
into one.

Fixes DistributedProofreaders#1711
@windymilla windymilla requested a review from srjfoo February 4, 2026 15:35
Increment line number of underscore diff from italic verse
@windymilla windymilla merged commit a520f4d into DistributedProofreaders:master Feb 5, 2026
1 check passed
@windymilla windymilla deleted the ppcomp-italics branch February 5, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve PPcomp

2 participants