Make Tag Browser visible by default and freely resizable

src/calibre/gui2/main.ui

@@ -301,8 +301,11 @@
       <widget class="QWidget" name="library">
        <layout class="QVBoxLayout" name="verticalLayout_2">
         <item>
-         <layout class="QHBoxLayout" name="horizontalLayout">
-          <item>
+         <widget class="QSplitter" name="horizontal_splitter">
+          <property name="orientation">
+           <enum>Qt::Horizontal</enum>
+          </property>
+          <widget class="QWidget" name="layoutWidget">
            <layout class="QVBoxLayout" name="verticalLayout">
             <item>
              <widget class="TagsView" name="tags_view">
@@ -363,7 +366,7 @@
               <item>
                <widget class="QLabel" name="restriction_label">
                 <property name="text">
-                 <string>&amp;Restrict display to:</string>
+                 <string>&amp;Restrict to:</string>
                 </property>
                 <property name="buddy">
                  <cstring>search_restriction</cstring>
@@ -386,42 +389,40 @@
              </layout>
             </item>
            </layout>
-          </item>
-          <item>
-           <widget class="BooksView" name="library_view">
-            <property name="sizePolicy">
-             <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
-              <horstretch>100</horstretch>
-              <verstretch>10</verstretch>
-             </sizepolicy>
-            </property>
-            <property name="acceptDrops">
-             <bool>true</bool>
-            </property>
-            <property name="dragEnabled">
-             <bool>true</bool>
-            </property>
-            <property name="dragDropOverwriteMode">
-             <bool>false</bool>
-            </property>
-            <property name="dragDropMode">
-             <enum>QAbstractItemView::DragDrop</enum>
-            </property>
-            <property name="alternatingRowColors">
-             <bool>true</bool>
-            </property>
-            <property name="selectionBehavior">
-             <enum>QAbstractItemView::SelectRows</enum>
-            </property>
-            <property name="showGrid">
-             <bool>false</bool>
-            </property>
-            <property name="wordWrap">
-             <bool>false</bool>
-            </property>
-           </widget>
-          </item>
-         </layout>
+          </widget>
+          <widget class="BooksView" name="library_view">
+           <property name="sizePolicy">
+            <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
+             <horstretch>100</horstretch>
+             <verstretch>10</verstretch>
+            </sizepolicy>
+           </property>
+           <property name="acceptDrops">
+            <bool>true</bool>
+           </property>
+           <property name="dragEnabled">
+            <bool>true</bool>
+           </property>
+           <property name="dragDropOverwriteMode">
+            <bool>false</bool>
+           </property>
+           <property name="dragDropMode">
+            <enum>QAbstractItemView::DragDrop</enum>
+           </property>
+           <property name="alternatingRowColors">
+            <bool>true</bool>
+           </property>
+           <property name="selectionBehavior">
+            <enum>QAbstractItemView::SelectRows</enum>
+           </property>
+           <property name="showGrid">
+            <bool>false</bool>
+           </property>
+           <property name="wordWrap">
+            <bool>false</bool>
+           </property>
+          </widget>
+         </widget>
         </item>
        </layout>
       </widget>

src/calibre/gui2/status.py

@@ -205,19 +205,6 @@ class CoverFlowButton(QToolButton):
         self.setDisabled(True)
         self.setToolTip(_('<p>Browsing books by their covers is disabled.<br>Import of pictureflow module failed:<br>')+reason)
 
-class TagViewButton(QToolButton):
-
-    def __init__(self, parent=None):
-        QToolButton.__init__(self, parent)
-        self.setIconSize(QSize(80, 80))
-        self.setIcon(QIcon(I('tags.svg')))
-        self.setToolTip(_('Click to browse books by tags'))
-        self.setSizePolicy(QSizePolicy(QSizePolicy.Preferred, QSizePolicy.Expanding))
-        self.setCursor(Qt.PointingHandCursor)
-        self.setCheckable(True)
-        self.setChecked(False)
-        self.setAutoRaise(True)
-
 
 class StatusBar(QStatusBar):
 
@@ -227,9 +214,7 @@ class StatusBar(QStatusBar):
         self.notifier = get_notifier(systray)
         self.movie_button = MovieButton(jobs_dialog)
         self.cover_flow_button = CoverFlowButton()
-        self.tag_view_button = TagViewButton()
         self.addPermanentWidget(self.cover_flow_button)
-        self.addPermanentWidget(self.tag_view_button)
         self.addPermanentWidget(self.movie_button)
         self.book_info = BookInfoDisplay(self.clearMessage)
         self.book_info.setAcceptDrops(True)

src/calibre/gui2/ui.py

@@ -575,8 +575,6 @@ class Main(MainWindow, Ui_MainWindow, DeviceGUI):
         self.connect(self.tags_view,
                 SIGNAL('tags_marked(PyQt_PyObject, PyQt_PyObject)'),
                      self.saved_search.clear_to_help)
-        self.connect(self.status_bar.tag_view_button,
-                SIGNAL('toggled(bool)'), self.toggle_tags_view)
         self.connect(self.search,
                 SIGNAL('search(PyQt_PyObject, PyQt_PyObject)'),
                      self.tags_view.model().reinit)
@@ -674,8 +672,10 @@ class Main(MainWindow, Ui_MainWindow, DeviceGUI):
         if self.cover_flow is not None and dynamic.get('cover_flow_visible', False):
             self.status_bar.cover_flow_button.toggle()
 
-        if dynamic.get('tag_view_visible', False):
-            self.status_bar.tag_view_button.toggle()
+        tb_state = dynamic.get('tag_browser_state', None)
+        if tb_state is not None:
+            self.horizontal_splitter.restoreState(tb_state)
+        self.toggle_tags_view(True)
 
         self._add_filesystem_book = Dispatcher(self.__add_filesystem_book)
         v = self.library_view
@@ -2331,7 +2331,6 @@ class Main(MainWindow, Ui_MainWindow, DeviceGUI):
             self.view_menu.actions()[1].setEnabled(True)
             self.action_open_containing_folder.setEnabled(True)
             self.action_sync.setEnabled(True)
-            self.status_bar.tag_view_button.setEnabled(True)
             self.status_bar.cover_flow_button.setEnabled(True)
             for action in list(self.delete_menu.actions())[1:]:
                 action.setEnabled(True)
@@ -2342,7 +2341,6 @@ class Main(MainWindow, Ui_MainWindow, DeviceGUI):
             self.view_menu.actions()[1].setEnabled(False)
             self.action_open_containing_folder.setEnabled(False)
             self.action_sync.setEnabled(False)
-            self.status_bar.tag_view_button.setEnabled(False)
             self.status_bar.cover_flow_button.setEnabled(False)
             for action in list(self.delete_menu.actions())[1:]:
                 action.setEnabled(False)
@@ -2459,8 +2457,9 @@ class Main(MainWindow, Ui_MainWindow, DeviceGUI):
     def write_settings(self):
         config.set('main_window_geometry', self.saveGeometry())
         dynamic.set('sort_history', self.library_view.model().sort_history)
-        dynamic.set('tag_view_visible', self.tags_view.isVisible())
         dynamic.set('cover_flow_visible', self.cover_flow.isVisible())
+        dynamic.set('tag_browser_state',
+                str(self.horizontal_splitter.saveState()))
         self.library_view.write_settings()
         if self.device_connected:
             self.save_device_view_settings()