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

Jump to bottom

WeasyPrint crashes when there's a local() with a font name with a space in it and no quotes around in CSS #487

Closed
0xDEADCAT opened this issue Jul 20, 2017 · 2 comments
Closed

WeasyPrint crashes when there's a local() with a font name with a space in it and no quotes around in CSS #487

0xDEADCAT opened this issue Jul 20, 2017 · 2 comments
Labels
bug Existing features not working as expected crash Problems preventing documents from being rendered
Milestone
v0.40

Comments

@0xDEADCAT
Copy link

minimum test case:
@font-face { font-family: "Test"; src: local(Test Test) }
@liZe liZe added bug Existing features not working as expected crash Problems preventing documents from being rendered labels Jul 20, 2017
@liZe
Copy link
Member

liZe commented Jul 20, 2017

It's also broken with local() for a slightly different reason.

liZe added a commit that referenced this issue Jul 20, 2017
@liZe
Fix the local(Test Test) syntax in @font-face src
1fd38bd 
Related to #487.
@liZe liZe closed this as completed in a9a99f6 Jul 20, 2017
@liZe
Copy link
Member

liZe commented Jul 20, 2017

The first use case was broken when WeasyPrint was ported to tinycss2. The second one was broken since the beginning. Thanks for reporting!

@liZe liZe added this to the v0.40 milestone Jul 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Existing features not working as expected crash Problems preventing documents from being rendered
Projects
None yet
Milestone
v0.40
Development

No branches or pull requests
2 participants
@liZe @0xDEADCAT