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

write_html use a fixed font size and family #16

Closed
GoogleCodeExporter opened this issue Mar 25, 2015 · 7 comments
Closed

write_html use a fixed font size and family #16

GoogleCodeExporter opened this issue Mar 25, 2015 · 7 comments

Comments

@GoogleCodeExporter
Copy link

When rendering a html using "write_html", the font size and family of the 
rendered part in the pdf is always the same. This is because the 
line 42 of html.py
self.set_font("times", 12)

What is the expected output? What do you see instead?

I'd expect the html rendered to take into account the font changes inserted in 
the html text.


Using the latest pyfpdf version included in web2py.

Regards.

Original issue reported on code.google.com by jredr...@gmail.com on 15 Sep 2011 at 12:45

@GoogleCodeExporter
Copy link
Author

Attached patch to fix the problem. There was also some other bugs in the fonts 
management after translating it from its php equivalent code.

Hope this helps.
Regards.

Original comment by jredr...@gmail.com on 15 Sep 2011 at 4:23

  • Added labels: ****
  • Removed labels: ****

Attachments:

@GoogleCodeExporter
Copy link
Author

Ok, if you want I can give you commit access so you can patch the files 
directly in the repository, do you agree?

Original comment by reingart@gmail.com on 15 Sep 2011 at 11:48

  • Changed state: Accepted
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Of course, I agree. I'm using pyfpdf, so I'm glad to help if I can improve it.
Regards.

Original comment by jredr...@gmail.com on 16 Sep 2011 at 6:18

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Sorry for the delay, you are a commiter now, fell free to commit patches ;-)

Original comment by reingart@gmail.com on 3 Oct 2011 at 5:38

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Issue 15 has been merged into this issue.

Original comment by reingart@gmail.com on 3 Oct 2011 at 5:48

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

fixed in rev f75aef66ce00

Original comment by reingart@gmail.com on 6 Aug 2012 at 7:43

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was closed by revision f75aef66ce00.

Original comment by reingart@gmail.com on 6 Aug 2012 at 8:16

  • Added labels: ****
  • Removed labels: ****

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

No branches or pull requests

1 participant