Skip to content

Preserve multiple mid-line space in autogen /$#1718

Merged
windymilla merged 1 commit intoDistributedProofreaders:masterfrom
windymilla:dollar
Feb 9, 2026
Merged

Preserve multiple mid-line space in autogen /$#1718
windymilla merged 1 commit intoDistributedProofreaders:masterfrom
windymilla:dollar

Conversation

@windymilla
Copy link
Copy Markdown
Collaborator

In GG1, /$ markup not only indents using an HTML span with margin, but also replaces pairs of mid-line spaces with   . Without that, HTML compresses the multiple spaces to one, which is not what is wanted.

This makes GG2 do the same. Fixes #1717

Testing notes:

  • dollar.txt - test file from issue
  • Run autogen.
  • Compare with output of GG1.
  • HTML code layout is different, but essentials that affect reader's view of the text should be the same

In GG1, `/$` markup not only indents using an HTML span
with margin, but also replaces pairs of mid-line spaces with
`  `. Without that, HTML compresses the multiple
spaces to one, which is not what is wanted.

This makes GG2 do the same. Fixes DistributedProofreaders#1717
@windymilla windymilla requested a review from srjfoo February 9, 2026 18:35
@windymilla windymilla merged commit d43ab05 into DistributedProofreaders:master Feb 9, 2026
1 check passed
@windymilla windymilla deleted the dollar branch February 9, 2026 19:47
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.

/$...$/ should preserve multiple whitespace in HTML

2 participants