From 779dad005ec6600f9fca3ab5772540ce84f2c87f Mon Sep 17 00:00:00 2001
From: Kovid Goyal <kovid@kovidgoyal.net>
Date: Fri, 25 Jun 2010 16:18:22 -0600
Subject: [PATCH] Fix #5975 (Set a margin for bottom/top has no effect)

---
 src/calibre/ebooks/pdf/writer.py | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/calibre/ebooks/pdf/writer.py b/src/calibre/ebooks/pdf/writer.py
index ff624f6831..03519a2cbb 100644
--- a/src/calibre/ebooks/pdf/writer.py
+++ b/src/calibre/ebooks/pdf/writer.py
@@ -154,20 +154,24 @@ class PDFWriter(QObject): # {{{
 
         self.view.load(QUrl.fromLocalFile(item))
 
-    def get_printer(self):
+    def get_printer(self, set_horz_margins=False):
         printer = get_pdf_printer()
         printer.setPaperSize(QSizeF(self.size[0] * 10, self.size[1] * 10), QPrinter.Millimeter)
-        printer.setPageMargins(0, 0, 0, 0, QPrinter.Point)
+        if set_horz_margins:
+            printer.setPageMargins(0., self.opts.margin_top, 0.,
+                    self.opts.margin_bottom, QPrinter.Point)
+        else:
+            printer.setPageMargins(0, 0, 0, 0, QPrinter.Point)
         printer.setOrientation(orientation(self.opts.orientation))
         printer.setOutputFormat(QPrinter.PdfFormat)
-        printer.setFullPage(True)
+        printer.setFullPage(not set_horz_margins)
         return printer
 
     def _render_html(self, ok):
         if ok:
             item_path = os.path.join(self.tmp_path, '%i.pdf' % len(self.combine_queue))
             self.logger.debug('\tRendering item %s as %i' % (os.path.basename(str(self.view.url().toLocalFile())), len(self.combine_queue)))
-            printer = self.get_printer()
+            printer = self.get_printer(set_horz_margins=True)
             printer.setOutputFileName(item_path)
             self.view.print_(printer)
         self._render_book()