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

Invalid breaks in words with accents #1098

Closed
ogmios-voice opened this issue Jan 2, 2019 · 4 comments
Closed

Invalid breaks in words with accents #1098

ogmios-voice opened this issue Jan 2, 2019 · 4 comments

Comments

@ogmios-voice
Copy link

Converting the following asciidoc document to pdf (using asciidoctor-pdf-1.5.0.alpha.17.dev w/ prawn-2.2.2)

:pdf-page-size: a5

= test

aàbècìdòeùf gáhéiíjókúlým nâoêpîqôrûs tñuõvãw xäyëzïaöbücÿd
aabbccddeef gghhiijjkkllm nnooppqqrrs ttuuvvw xxyyzzaabbccd

results in an invalid break in the first row. The resulting block of the pdf looks like this (for me):

aàbècìdòeùf gáhéiíjókúlým nâoêpîqôrûs tñuõvãw xäyëzïa
öbüc d
aabbccddeef gghhiijjkkllm nnooppqqrrs ttuuvvw
xxyyzzaabbccd

(Note: the ÿ character is invisible in the result, but that is a font issue.)

But the breaks should be consistent, while the current position of the break depends on whether a given letter is accented or not. The correct output should be:

aàbècìdòeùf gáhéiíjókúlým nâoêpîqôrûs tñuõvãw
xäyëzïaöbücÿd
aabbccddeef gghhiijjkkllm nnooppqqrrs ttuuvvw
xxyyzzaabbccd

See also: asciidoctor/asciidoctor-pdf#979 (comment)

@ogmios-voice ogmios-voice changed the title Invalid breaks of words with accents Invalid breaks in words with accents Jan 2, 2019
@gettalong
Copy link
Member

Can you provide a simple Ruby script (no dependencies except Prawn) to reproduce the problem?

@ogmios-voice
Copy link
Author

Unfortunately I can't. I don't have too much ruby experience, no experience at all with Prawn, and I will lack any free time in the following period.

@gettalong
Copy link
Member

No problem! Just note that the issue resolution may take longer without such a script.

@pointlessone
Copy link
Member

I'm closing it in favour of #1295. I don't think fixing just accented characters would be useful or even possible without addressing the bigger issue.

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

No branches or pull requests

3 participants