ReptarX / pyfpdf

Automatically exported from code.google.com/p/pyfpdf
GNU Lesser General Public License v3.0
0 stars 0 forks source link

write_html use a fixed font size and family #16

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
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 commented 9 years ago
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

Attachments:

GoogleCodeExporter commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
Issue 15 has been merged into this issue.

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

GoogleCodeExporter commented 9 years ago
fixed in rev f75aef66ce00

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

GoogleCodeExporter commented 9 years ago
This issue was closed by revision f75aef66ce00.

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