From 6e8a7ce285b004c232f010a7960c4702b95710de Mon Sep 17 00:00:00 2001
From: John Schember <john@nachtimwald.com>
Date: Wed, 30 Dec 2009 18:14:20 -0500
Subject: [PATCH] PML input: Use text-indent instead of margin when converting
 \t and \T tags. PML output: preliminary changes to support print style output
 (the parser needs to be rewritten to support it in a cleaner manner).

---
 src/calibre/ebooks/pml/pmlconverter.py |  4 ++--
 src/calibre/ebooks/pml/pmlml.py        | 14 ++++++++++----
 2 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/src/calibre/ebooks/pml/pmlconverter.py b/src/calibre/ebooks/pml/pmlconverter.py
index 3484be5927..099a23c544 100644
--- a/src/calibre/ebooks/pml/pmlconverter.py
+++ b/src/calibre/ebooks/pml/pmlconverter.py
@@ -71,8 +71,8 @@ class PML_HTMLizer(object):
         'ra': ('<span id="r%s"></span><a href="#%s">', '</a>'),
         'c': ('<div style="text-align: center; margin: auto;">', '</div>'),
         'r': ('<div style="text-align: right;">', '</div>'),
-        't': ('<div style="margin-left: 5%;">', '</div>'),
-        'T': ('<div style="margin-left: %s;">', '</div>'),
+        't': ('<div style="text-indent: 1.5em;">', '</div>'),
+        'T': ('<div style="text-indent: %s;">', '</div>'),
         'i': ('<span style="font-style: italic;">', '</span>'),
         'u': ('<span style="text-decoration: underline;">', '</span>'),
         'd': ('<span style="text-decoration: line-through;">', '</span>'),
diff --git a/src/calibre/ebooks/pml/pmlml.py b/src/calibre/ebooks/pml/pmlml.py
index 77f858dbc9..2e00ad6ae3 100644
--- a/src/calibre/ebooks/pml/pmlml.py
+++ b/src/calibre/ebooks/pml/pmlml.py
@@ -204,10 +204,12 @@ class PMLMLizer(object):
         tag_count = 0
 
         # Are we in a paragraph block?
-        if tag in BLOCK_TAGS or style['display'] in BLOCK_STYLES:
+        if tag in BLOCK_TAGS:# or style['display'] in BLOCK_STYLES:
             if 'block' not in tag_stack:
                 tag_count += 1
                 tag_stack.append('block')
+                if self.opts.remove_paragraph_spacing:
+                    text.append('\\t')
 
         # Process tags that need special processing and that do not have inner
         # text. Usually these require an argument
@@ -287,10 +289,14 @@ class PMLMLizer(object):
             close_tag_list.insert(0, tag_stack.pop())
         text += self.close_tags(close_tag_list)
         if tag in SEPARATE_TAGS:
-            text.append('\n\n')
+            text.append('\n')
+            if not self.opts.remove_paragraph_spacing:
+                text.append('\n')
 
-        if 'block' not in tag_stack:
-            text.append('\n\n')
+        if 'block' not in tag_stack and text and text[-1] != '\n':
+            text.append('\n')
+            if not self.opts.remove_paragraph_spacing:
+                text.append('\n')
 
         #if style['page-break-after'] == 'always':
         #    text.append('\\p')