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

Drop leading blanks #355

Closed
molmich opened this issue Nov 9, 2015 · 2 comments
Closed

Drop leading blanks #355

molmich opened this issue Nov 9, 2015 · 2 comments
Assignees
Labels

Comments

@molmich
Copy link

molmich commented Nov 9, 2015

if I generate PDF document, where I use blanks for better ASCIDOC format/look, this blanks are not deleted/dropped!

for example:

...
ATTENTION: This is an example of 
           leading blank.
...

has in PDF:

...
ATTENTION: This is an example of            leading blank.
...

is it possible to dron leading blanks?

@mojavelinux
Copy link
Member

Doh! That is definitely an issue. Unlike HTML, PDF treats repeating spaces as significant, whereas HTML consolidates them. So, we'll need to do that consolidation. Thanks for reporting!

@mojavelinux mojavelinux added this to the v1.5.0.beta.1 milestone Nov 9, 2015
@mojavelinux mojavelinux self-assigned this Nov 9, 2015
mojavelinux added a commit to mojavelinux/asciidoctor-pdf that referenced this issue May 7, 2016
- collapse all whitespace, not just endlines, when normalizing text
@mojavelinux
Copy link
Member

The problem here is that we are only replacing endlines with single spaces. We are not then collapsing the repeated spaces or tabs. The normalize step needs to be updated to collapse all consecutive whitespace characters into a single space (matching the rules of HTML).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants