-
Notifications
You must be signed in to change notification settings - Fork 6
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
Dropping webkit #15
Comments
@nicoe been waiting for getting rid of the beast of webkit :) I have never used WeasyPrint before. Will give this a try. Have you used it before ? |
@aroraumang any thoughts ? |
@sharoonthomas Will give it a try ;) |
@sharoonthomas I just remember a presentation about it at pyconfr 2012 and the guy who wrote it was really impressive. It is definitively worth a try I think. BTW, while looking at the different reporting solution in tryton I also noticed that GrasBauer are using pisa (or xhtml2pdf), I don't know what it's worth though … |
wow, that is indeed a very nice tool! if you like i can prepare a pull-request to support both... ( @nicoe something like an enum to choose the used rendering engine would be cool or not?) ☀️ |
@hiaselhans Sure, why not? ;) |
@hiaselhans |
@aroraumang will prepare soon, as i find some time... @nicoe thanks for your response, i am not to judge about the need neither, just asking you as i've seen the nice modularization youve already done in this topic... |
Just to clarify: property fields will be replaced by function fields.
That's my idea too. |
@nicoe pr #25 is merged and there is weasy print support for reports that want to implement it. However, it would be nice to have this as a configuration option on reports which I think your new proposal for easily extending report engines build for version 3.5+ Creates issue #26 to handle the migration to new API for tryton 3.6 |
This is just a suggestion not a bug at all. So you can do whatever you want with it ;).
But I was wondering if you ever though about dropping webkit in favor of something like weasyprint : https://github.com/Kozea/WeasyPrint
The text was updated successfully, but these errors were encountered: