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.

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

Asciidoc: add an option to prevent line wrapping #242

Closed
jnavila opened this issue May 28, 2020 · 1 comment · Fixed by #247
Closed

Asciidoc: add an option to prevent line wrapping #242

jnavila opened this issue May 28, 2020 · 1 comment · Fixed by #247

Comments

@jnavila
Copy link
Collaborator

jnavila commented May 28, 2020

If a paragraph of a list item contains for instance "see point A. Bla bla bla" and the wrapping occurs like that:

  see point 
  A. Bla bla bla

Then the second line can be interpreted as a sub-list start. Disabling wrapping would ensure that such occurence does not happen.

If asciidoc output is aimed at being read directly, wrapping might still be an option. But for machine processing, it's safer to disable it.

@mquinson
Copy link
Owner

That's an important issue, because I don't see any good solution. The less bad would be to prevent rewrapping all together as you propose, and I don't see how we could wrap the content while avoiding such issue.

I guess that a "neverwrap" as in the Text module is the way to go, at the end.

Thanks,

jnavila added a commit to jnavila/po4a that referenced this issue Jun 3, 2020
This option disables line wrapping when it would normally occur in
order to prevent introduction of asciidoc formatting errors due to not
managed new line prefix that could be wrongly interpreted by asciidoc.

Fixes mquinson#242
jnavila added a commit to jnavila/po4a that referenced this issue Sep 20, 2020
This change was performed in order to prevent introduction of asciidoc
formatting errors due to not managed new line prefix that could be
wrongly interpreted by asciidoc.

Add forcewrap option to re-enable line wrapping when it used to occur,
in case users of previous versions require it.

Fixes mquinson#242
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 a pull request may close this issue.

2 participants