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

Change default height from 100 to 1 #218

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pier
Copy link

@pier pier commented Jan 26, 2018

The default height of 100 causes unwanted results with the "blank" space between EAN13 and EAN5 add-on codes in PDF rendering app wkhtmltopdf (which uses the Webkit browser). This problem goes away when the default is changed to 1.

The default height of 100 causes unwanted results with the "blank" space between EAN13 and EAN5 add-on codes in PDF rendering app wkhtmltopdf (which uses the Webkit browser). This problem goes away when the default is changed to 1.
@huangjinshe
Copy link

Did you even succeed with wkhtmltopdf to generate the barcode image? I tried failed with it. My PDF file always display blank for the barcode image which generated by jsbarcode.

@pier
Copy link
Author

pier commented Jul 29, 2020

Apart from the issue with add-on barcodes I've had no problems rendering in wkhtmltopdf. Maybe try and add the --debug-javascript flag to the wkhtmltopdf command to see what could be the problem, or delay the javascript with --javascript-delay.

@huangjinshe
Copy link

huangjinshe commented Jul 29, 2020

I did tried -javascript-delay command, it's not working even I waited 10 seconds. Because I'm using another framework called openhtmltopdf to convert it, I'm not able to get the debug info by -debug-javascript, Could you help me check what is the problem if you have free time? It would be very helpful.

@huangjinshe
Copy link

Don't bother. I download the binary of openhtmltopdf, and debug the js, also find the problem.

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 this pull request may close these issues.

2 participants