Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

roffit takes some characters that should be ignored #28

Closed
hydrargyrum opened this issue Sep 16, 2022 · 2 comments · Fixed by #29
Closed

roffit takes some characters that should be ignored #28

hydrargyrum opened this issue Sep 16, 2022 · 2 comments · Fixed by #29

Comments

@hydrargyrum
Copy link

Debian's manpages (for example ls) contain , and / that seem to have a markup role (italic?) around some words. Here's how it could render for example on debian site.

But roffit renders them literally:

<p class="level0"><span Class="bold">ls</span> [<span Class="emphasis">,OPTION/</span>]... [<span Class="emphasis">,FILE/</span>]... <a name="DESCRIPTION"></a><h2 class="nroffsh">DESCRIPTION</h2>
@bagder
Copy link
Owner

bagder commented Sep 16, 2022

It think it is \, and \/ that are special. I'm reading https://n-t-roff.github.io/heirloom/doctools/troff.pdf and on page 58 it says:

The \/ escape sequence inserts an italic correction, i.e. a small piece of horizontal motion (1/12 em) that should be
sufficient to separate an italic character from a following roman character. Similarly, \, adds a left italic correction,
always a zero motion, that should be sufficient to separate a roman character from an immediately following italic
character.

bagder added a commit that referenced this issue Sep 16, 2022
@bagder
roffit: strip italic corrections
6bd0f46 
The `\/` escape sequence inserts an italic correction, i.e. a small piece of
horizontal motion (1/12 em) that should be sufficient to separate an italic
character from a following roman character. Similarly, `\,`, adds a left
italic correction, always a zero motion, that should be sufficient to separate
a roman character from an immediately following italic character.

roffit now simply removes them

Fixes #28
@bagder bagder mentioned this issue Sep 16, 2022
Merged
@bagder
Copy link
Owner

bagder commented Sep 16, 2022

Any chance you can try #29 and see if it fixes the issue?

bagder added a commit that referenced this issue Sep 16, 2022
@bagder
roffit: strip italic corrections
93ccdbb 
The `\/` escape sequence inserts an italic correction, i.e. a small piece of
horizontal motion (1/12 em) that should be sufficient to separate an italic
character from a following roman character. Similarly, `\,`, adds a left
italic correction, always a zero motion, that should be sufficient to separate
a roman character from an immediately following italic character.

roffit now simply removes them

Reported-by: hydrargyrum on github
Fixes #28
bagder added a commit that referenced this issue Sep 16, 2022
@bagder
testpage: add test of \, and \/
19955b6 
Based on the #28 report
bagder added a commit that referenced this issue Sep 16, 2022
@bagder
roffit: strip italic corrections
b180f65 
The `\/` escape sequence inserts an italic correction, i.e. a small piece of
horizontal motion (1/12 em) that should be sufficient to separate an italic
character from a following roman character. Similarly, `\,`, adds a left
italic correction, always a zero motion, that should be sufficient to separate
a roman character from an immediately following italic character.

roffit now simply removes them

Reported-by: hydrargyrum on github
Fixes #28
bagder added a commit that referenced this issue Sep 16, 2022
@bagder
testpage: add test of \, and \/
674e2b7 
Based on the #28 report
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

roffit: strip italic corrections bagder/roffit
2 participants
@bagder @hydrargyrum