From 8ff4ff2aa00190a3f81462d81355a80e3f9ef913 Mon Sep 17 00:00:00 2001
From: Kovid Goyal <kovid@kovidgoyal.net>
Date: Sun, 5 May 2013 09:40:30 +0530
Subject: [PATCH] Add a button to clear the viewer search history in the viewer
 Preferences, under Miscellaneous

---
 src/calibre/gui2/__init__.py      |  2 +-
 src/calibre/gui2/viewer/config.py | 15 ++++++++--
 src/calibre/gui2/viewer/config.ui | 47 ++++++++++++++++++-------------
 src/calibre/gui2/viewer/main.py   |  3 ++
 4 files changed, 43 insertions(+), 24 deletions(-)

diff --git a/src/calibre/gui2/__init__.py b/src/calibre/gui2/__init__.py
index 3a532ec2c0..5fcde65ff5 100644
--- a/src/calibre/gui2/__init__.py
+++ b/src/calibre/gui2/__init__.py
@@ -187,7 +187,7 @@ def _config(): # {{{
     c.add_opt('shortcuts_search_history', default=[],
         help='Search history for the keyboard preferences')
     c.add_opt('jobs_search_history', default=[],
-        help='Search history for the keyboard preferences')
+        help='Search history for the tweaks preferences')
     c.add_opt('tweaks_search_history', default=[],
         help='Search history for tweaks')
     c.add_opt('worker_limit', default=6,
diff --git a/src/calibre/gui2/viewer/config.py b/src/calibre/gui2/viewer/config.py
index a744a2c1f9..4132149842 100644
--- a/src/calibre/gui2/viewer/config.py
+++ b/src/calibre/gui2/viewer/config.py
@@ -139,12 +139,20 @@ class ConfigDialog(QDialog, Ui_Dialog):
         self.load_options(opts)
         self.init_load_themes()
 
+        self.clear_search_history_button.clicked.connect(self.clear_search_history)
+
+    def clear_search_history(self):
+        from calibre.gui2 import config
+        config['viewer_search_history'] = []
+
     def save_theme(self):
         themename, ok = QInputDialog.getText(self, _('Theme name'),
                 _('Choose a name for this theme'))
-        if not ok: return
+        if not ok:
+            return
         themename = unicode(themename).strip()
-        if not themename: return
+        if not themename:
+            return
         c = config('')
         c.add_opt('theme_name_xxx', default=themename)
         self.save_options(c)
@@ -247,7 +255,8 @@ class ConfigDialog(QDialog, Ui_Dialog):
     def update_sample_colors(self):
         for x in ('text', 'background'):
             val = getattr(self, 'current_%s_color'%x)
-            if not val: val = 'inherit' if x == 'text' else 'transparent'
+            if not val:
+                val = 'inherit' if x == 'text' else 'transparent'
             ss = 'QLabel { %s: %s }'%('background-color' if x == 'background'
                     else 'color', val)
             getattr(self, '%s_color_sample'%x).setStyleSheet(ss)
diff --git a/src/calibre/gui2/viewer/config.ui b/src/calibre/gui2/viewer/config.ui
index b146b571a2..1dce8e1a9e 100644
--- a/src/calibre/gui2/viewer/config.ui
+++ b/src/calibre/gui2/viewer/config.ui
@@ -68,7 +68,7 @@ QToolBox::tab:hover {
             <x>0</x>
             <y>0</y>
             <width>811</width>
-            <height>352</height>
+            <height>380</height>
            </rect>
           </property>
           <attribute name="label">
@@ -240,8 +240,8 @@ QToolBox::tab:hover {
            <rect>
             <x>0</x>
             <y>0</y>
-            <width>397</width>
-            <height>232</height>
+            <width>811</width>
+            <height>380</height>
            </rect>
           </property>
           <attribute name="label">
@@ -370,8 +370,8 @@ QToolBox::tab:hover {
            <rect>
             <x>0</x>
             <y>0</y>
-            <width>352</width>
-            <height>176</height>
+            <width>811</width>
+            <height>380</height>
            </rect>
           </property>
           <attribute name="label">
@@ -446,8 +446,8 @@ QToolBox::tab:hover {
            <rect>
             <x>0</x>
             <y>0</y>
-            <width>351</width>
-            <height>76</height>
+            <width>811</width>
+            <height>380</height>
            </rect>
           </property>
           <attribute name="label">
@@ -525,8 +525,8 @@ QToolBox::tab:hover {
            <rect>
             <x>0</x>
             <y>0</y>
-            <width>410</width>
-            <height>120</height>
+            <width>811</width>
+            <height>380</height>
            </rect>
           </property>
           <attribute name="label">
@@ -596,8 +596,8 @@ QToolBox::tab:hover {
            <rect>
             <x>0</x>
             <y>0</y>
-            <width>352</width>
-            <height>151</height>
+            <width>811</width>
+            <height>380</height>
            </rect>
           </property>
           <attribute name="label">
@@ -628,27 +628,34 @@ QToolBox::tab:hover {
              </property>
             </widget>
            </item>
-           <item row="3" column="0" colspan="2">
+           <item row="5" column="0">
+            <widget class="QPushButton" name="clear_search_history_button">
+             <property name="text">
+              <string>Clear search history</string>
+             </property>
+            </widget>
+           </item>
+           <item row="2" column="0">
+            <widget class="QCheckBox" name="opt_show_controls">
+             <property name="text">
+              <string>Show &amp;controls in the viewer window</string>
+             </property>
+            </widget>
+           </item>
+           <item row="3" column="0">
             <widget class="QCheckBox" name="opt_remember_window_size">
              <property name="text">
               <string>Remember last used &amp;window size and layout</string>
              </property>
             </widget>
            </item>
-           <item row="4" column="0" colspan="2">
+           <item row="4" column="0">
             <widget class="QCheckBox" name="opt_remember_current_page">
              <property name="text">
               <string>Remember the &amp;current page when quitting</string>
              </property>
             </widget>
            </item>
-           <item row="2" column="0" colspan="2">
-            <widget class="QCheckBox" name="opt_show_controls">
-             <property name="text">
-              <string>Show &amp;controls in the viewer window</string>
-             </property>
-            </widget>
-           </item>
           </layout>
          </widget>
         </widget>
diff --git a/src/calibre/gui2/viewer/main.py b/src/calibre/gui2/viewer/main.py
index 4587a6542b..3b63d51c15 100644
--- a/src/calibre/gui2/viewer/main.py
+++ b/src/calibre/gui2/viewer/main.py
@@ -940,6 +940,9 @@ class EbookViewer(MainWindow, Ui_EbookViewer):
     def do_config(self):
         self.view.config(self)
         self.load_theme_menu()
+        from calibre.gui2 import config
+        if not config['viewer_search_history']:
+            self.search.clear_history()
 
     def bookmark(self, *args):
         num = 1