From 39452cf714d71410a90e914711402bcd19b6df8b Mon Sep 17 00:00:00 2001
From: Kovid Goyal <kovid@kovidgoyal.net>
Date: Mon, 9 Mar 2009 09:01:27 -0700
Subject: [PATCH] Implement #1996 (Trimming option in cbr conversion)

---
 src/calibre/ebooks/lrf/comic/convert_from.py |   6 +-
 src/calibre/gui2/dialogs/comicconf.ui        | 144 ++++++++++---------
 2 files changed, 81 insertions(+), 69 deletions(-)

diff --git a/src/calibre/ebooks/lrf/comic/convert_from.py b/src/calibre/ebooks/lrf/comic/convert_from.py
index 45254f7b87..50f5e1e72e 100755
--- a/src/calibre/ebooks/lrf/comic/convert_from.py
+++ b/src/calibre/ebooks/lrf/comic/convert_from.py
@@ -143,7 +143,8 @@ class PageProcessor(list):
                     MagickRotateImage(wand, pw, -90)
                     
                 # 25 percent fuzzy trim?
-                MagickTrimImage(wand, 25*65535/100)
+                if not self.opts.disable_trim:
+                    MagickTrimImage(wand, 25*65535/100)
                 MagickSetImagePage(wand, 0,0,0,0)   #Clear page after trim, like a "+repage"
                 # Do the Photoshop "Auto Levels" equivalent
                 if not self.opts.dont_normalize:
@@ -303,6 +304,9 @@ def config(defaults=None,output_format='lrf'):
               help=_('Maintain picture aspect ratio. Default is to fill the screen.'))
     c.add_opt('dont_sharpen', ['-s', '--disable-sharpen'], default=False,
               help=_('Disable sharpening.'))
+    c.add_opt('disable_trim', ['--disable-trim'], default=False,
+              help=_('Disable trimming of comic pages. For some comics, '
+                     'trimming might remove content as well as borders.'))
     c.add_opt('landscape', ['-l', '--landscape'], default=False,
               help=_("Don't split landscape images into two portrait images"))
     c.add_opt('wide', ['-w', '--wide-aspect'], default=False,
diff --git a/src/calibre/gui2/dialogs/comicconf.ui b/src/calibre/gui2/dialogs/comicconf.ui
index 36af85764a..acab125d57 100644
--- a/src/calibre/gui2/dialogs/comicconf.ui
+++ b/src/calibre/gui2/dialogs/comicconf.ui
@@ -1,154 +1,162 @@
-<ui version="4.0" >
+<?xml version="1.0" encoding="UTF-8"?>
+<ui version="4.0">
  <class>Dialog</class>
- <widget class="QDialog" name="Dialog" >
-  <property name="geometry" >
+ <widget class="QDialog" name="Dialog">
+  <property name="geometry">
    <rect>
     <x>0</x>
     <y>0</y>
     <width>646</width>
-    <height>468</height>
+    <height>503</height>
    </rect>
   </property>
-  <property name="windowTitle" >
+  <property name="windowTitle">
    <string>Dialog</string>
   </property>
-  <property name="windowIcon" >
-   <iconset resource="../images.qrc" >
+  <property name="windowIcon">
+   <iconset resource="../images.qrc">
     <normaloff>:/images/convert.svg</normaloff>:/images/convert.svg</iconset>
   </property>
-  <layout class="QGridLayout" name="gridLayout" >
-   <item row="0" column="0" >
-    <widget class="QLabel" name="title_label" >
-     <property name="text" >
+  <layout class="QGridLayout" name="gridLayout">
+   <item row="0" column="0">
+    <widget class="QLabel" name="title_label">
+     <property name="text">
       <string>&amp;Title:</string>
      </property>
-     <property name="buddy" >
+     <property name="buddy">
       <cstring>opt_title</cstring>
      </property>
     </widget>
    </item>
-   <item row="0" column="1" >
-    <widget class="QLineEdit" name="opt_title" />
+   <item row="0" column="1">
+    <widget class="QLineEdit" name="opt_title"/>
    </item>
-   <item row="1" column="0" >
-    <widget class="QLabel" name="author_label" >
-     <property name="text" >
+   <item row="1" column="0">
+    <widget class="QLabel" name="author_label">
+     <property name="text">
       <string>&amp;Author(s):</string>
      </property>
-     <property name="buddy" >
+     <property name="buddy">
       <cstring>opt_author</cstring>
      </property>
     </widget>
    </item>
-   <item row="1" column="1" >
-    <widget class="QLineEdit" name="opt_author" />
+   <item row="1" column="1">
+    <widget class="QLineEdit" name="opt_author"/>
    </item>
-   <item row="2" column="0" >
-    <widget class="QLabel" name="label_3" >
-     <property name="text" >
+   <item row="2" column="0">
+    <widget class="QLabel" name="label_3">
+     <property name="text">
       <string>&amp;Number of Colors:</string>
      </property>
-     <property name="buddy" >
+     <property name="buddy">
       <cstring>opt_colors</cstring>
      </property>
     </widget>
    </item>
-   <item row="2" column="1" >
-    <widget class="QSpinBox" name="opt_colors" >
-     <property name="minimum" >
+   <item row="2" column="1">
+    <widget class="QSpinBox" name="opt_colors">
+     <property name="minimum">
       <number>8</number>
      </property>
-     <property name="maximum" >
+     <property name="maximum">
       <number>3200000</number>
      </property>
-     <property name="singleStep" >
+     <property name="singleStep">
       <number>8</number>
      </property>
     </widget>
    </item>
-   <item row="3" column="0" >
-    <widget class="QLabel" name="label_4" >
-     <property name="text" >
+   <item row="3" column="0">
+    <widget class="QLabel" name="label_4">
+     <property name="text">
       <string>&amp;Profile:</string>
      </property>
-     <property name="buddy" >
+     <property name="buddy">
       <cstring>opt_profile</cstring>
      </property>
     </widget>
    </item>
-   <item row="3" column="1" >
-    <widget class="QComboBox" name="opt_profile" />
+   <item row="3" column="1">
+    <widget class="QComboBox" name="opt_profile"/>
    </item>
-   <item row="4" column="0" >
-    <widget class="QCheckBox" name="opt_dont_normalize" >
-     <property name="text" >
+   <item row="4" column="0">
+    <widget class="QCheckBox" name="opt_dont_normalize">
+     <property name="text">
       <string>Disable &amp;normalize</string>
      </property>
     </widget>
    </item>
-   <item row="5" column="0" >
-    <widget class="QCheckBox" name="opt_keep_aspect_ratio" >
-     <property name="text" >
+   <item row="5" column="0">
+    <widget class="QCheckBox" name="opt_keep_aspect_ratio">
+     <property name="text">
       <string>Keep &amp;aspect ratio</string>
      </property>
     </widget>
    </item>
-   <item row="6" column="0" >
-    <widget class="QCheckBox" name="opt_dont_sharpen" >
-     <property name="text" >
+   <item row="6" column="0">
+    <widget class="QCheckBox" name="opt_dont_sharpen">
+     <property name="text">
       <string>Disable &amp;Sharpening</string>
      </property>
     </widget>
    </item>
-   <item row="8" column="0" >
-    <widget class="QCheckBox" name="opt_landscape" >
-     <property name="text" >
+   <item row="9" column="0">
+    <widget class="QCheckBox" name="opt_landscape">
+     <property name="text">
       <string>&amp;Landscape</string>
      </property>
     </widget>
    </item>
-   <item row="10" column="0" >
-    <widget class="QCheckBox" name="opt_no_sort" >
-     <property name="text" >
+   <item row="11" column="0">
+    <widget class="QCheckBox" name="opt_no_sort">
+     <property name="text">
       <string>Don't so&amp;rt</string>
      </property>
     </widget>
    </item>
-   <item row="12" column="1" >
-    <widget class="QDialogButtonBox" name="buttonBox" >
-     <property name="orientation" >
+   <item row="13" column="1">
+    <widget class="QDialogButtonBox" name="buttonBox">
+     <property name="orientation">
       <enum>Qt::Horizontal</enum>
      </property>
-     <property name="standardButtons" >
+     <property name="standardButtons">
       <set>QDialogButtonBox::Cancel|QDialogButtonBox::Ok</set>
      </property>
     </widget>
    </item>
-   <item row="9" column="0" >
-    <widget class="QCheckBox" name="opt_right2left" >
-     <property name="text" >
+   <item row="10" column="0">
+    <widget class="QCheckBox" name="opt_right2left">
+     <property name="text">
       <string>&amp;Right to left</string>
      </property>
     </widget>
    </item>
-   <item row="11" column="0" >
-    <widget class="QCheckBox" name="opt_despeckle" >
-     <property name="text" >
+   <item row="12" column="0">
+    <widget class="QCheckBox" name="opt_despeckle">
+     <property name="text">
       <string>De&amp;speckle</string>
      </property>
     </widget>
    </item>
-   <item row="7" column="0" >
-    <widget class="QCheckBox" name="opt_wide" >
-     <property name="text" >
+   <item row="8" column="0">
+    <widget class="QCheckBox" name="opt_wide">
+     <property name="text">
       <string>&amp;Wide</string>
      </property>
     </widget>
    </item>
+   <item row="7" column="0">
+    <widget class="QCheckBox" name="opt_disable_trim">
+     <property name="text">
+      <string>Disable &amp;Trimming</string>
+     </property>
+    </widget>
+   </item>
   </layout>
  </widget>
  <resources>
-  <include location="../images.qrc" />
+  <include location="../images.qrc"/>
  </resources>
  <connections>
   <connection>
@@ -157,11 +165,11 @@
    <receiver>Dialog</receiver>
    <slot>accept()</slot>
    <hints>
-    <hint type="sourcelabel" >
+    <hint type="sourcelabel">
      <x>248</x>
      <y>254</y>
     </hint>
-    <hint type="destinationlabel" >
+    <hint type="destinationlabel">
      <x>157</x>
      <y>274</y>
     </hint>
@@ -173,11 +181,11 @@
    <receiver>Dialog</receiver>
    <slot>reject()</slot>
    <hints>
-    <hint type="sourcelabel" >
+    <hint type="sourcelabel">
      <x>316</x>
      <y>260</y>
     </hint>
-    <hint type="destinationlabel" >
+    <hint type="destinationlabel">
      <x>286</x>
      <y>274</y>
     </hint>