Add an option to split the toolbar into two toolbars

src/calibre/gui2/__init__.py

@@ -50,6 +50,7 @@ gprefs.defaults['action-layout-context-menu-device'] = (
 gprefs.defaults['show_splash_screen'] = True
 gprefs.defaults['toolbar_icon_size'] = 'medium'
 gprefs.defaults['toolbar_text'] = 'auto'
+gprefs.defaults['show_child_bar'] = False
 
 # }}}
 

src/calibre/gui2/actions/__init__.py

@@ -71,6 +71,12 @@ class InterfaceAction(QObject):
     all_locations = frozenset(['toolbar', 'toolbar-device', 'context-menu',
         'context-menu-device'])
 
+    #: Type of action
+    #: 'current' means acts on the current view
+    #: 'global' means an action that does not act on the current view, but rather
+    #: on calibre as a whole
+    action_type = 'global'
+
     def __init__(self, parent, site_customization):
         QObject.__init__(self, parent)
         self.setObjectName(self.name)

src/calibre/gui2/actions/add.py

@@ -25,6 +25,7 @@ class AddAction(InterfaceAction):
     action_spec = (_('Add books'), 'add_book.png',
             _('Add books to the calibre library/device from files on your computer')
             , _('A'))
+    action_type = 'current'
 
     def genesis(self):
         self._add_filesystem_book = self.Dispatcher(self.__add_filesystem_book)

src/calibre/gui2/actions/add_to_library.py

@@ -13,6 +13,7 @@ class AddToLibraryAction(InterfaceAction):
     action_spec = (_('Add books to library'), 'add_book.png',
             _('Add books to your calibre library from the connected device'), None)
     dont_add_to = frozenset(['toolbar', 'context-menu'])
+    action_type = 'current'
 
     def genesis(self):
         self.qaction.triggered.connect(self.add_books_to_library)

src/calibre/gui2/actions/annotate.py

@@ -18,6 +18,7 @@ class FetchAnnotationsAction(InterfaceAction):
 
     name = 'Fetch Annotations'
     action_spec = (_('Fetch annotations (experimental)'), None, None, None)
+    action_type = 'current'
 
     def genesis(self):
         pass

src/calibre/gui2/actions/convert.py

@@ -21,6 +21,7 @@ class ConvertAction(InterfaceAction):
     name = 'Convert Books'
     action_spec = (_('Convert books'), 'convert.png', None, _('C'))
     dont_add_to = frozenset(['toolbar-device', 'context-menu-device'])
+    action_type = 'current'
 
     def genesis(self):
         cm = QMenu()

src/calibre/gui2/actions/copy_to_library.py

@@ -80,6 +80,7 @@ class CopyToLibraryAction(InterfaceAction):
             _('Copy selected books to the specified library'), None)
     popup_type = QToolButton.InstantPopup
     dont_add_to = frozenset(['toolbar-device', 'context-menu-device'])
+    action_type = 'current'
 
     def genesis(self):
         self.menu = QMenu(self.gui)

src/calibre/gui2/actions/delete.py

@@ -16,6 +16,7 @@ class DeleteAction(InterfaceAction):
 
     name = 'Remove Books'
     action_spec = (_('Remove books'), 'trash.png', None, _('Del'))
+    action_type = 'current'
 
     def genesis(self):
         self.qaction.triggered.connect(self.delete_books)

src/calibre/gui2/actions/edit_collections.py

@@ -13,6 +13,7 @@ class EditCollectionsAction(InterfaceAction):
     action_spec = (_('Manage collections'), None,
             _('Manage the collections on this device'), None)
     dont_add_to = frozenset(['toolbar', 'context-menu'])
+    action_type = 'current'
 
     def genesis(self):
         self.qaction.triggered.connect(self.edit_collections)

src/calibre/gui2/actions/edit_metadata.py

@@ -22,6 +22,7 @@ class EditMetadataAction(InterfaceAction):
 
     name = 'Edit Metadata'
     action_spec = (_('Edit metadata'), 'edit_input.png', None, _('E'))
+    action_type = 'current'
 
     def genesis(self):
         self.create_action(spec=(_('Merge book records'), 'merge_books.png',

src/calibre/gui2/actions/open.py

@@ -14,6 +14,7 @@ class OpenFolderAction(InterfaceAction):
     action_spec = (_('Open containing folder'), 'document_open.png', None,
             _('O'))
     dont_add_to = frozenset(['toolbar-device', 'context-menu-device'])
+    action_type = 'current'
 
     def genesis(self):
         self.qaction.triggered.connect(self.gui.iactions['View'].view_folder)

src/calibre/gui2/actions/save_to_disk.py

@@ -38,6 +38,7 @@ class SaveToDiskAction(InterfaceAction):
 
     name = "Save To Disk"
     action_spec = (_('Save to disk'), 'save.png', None, _('S'))
+    action_type = 'current'
 
     def genesis(self):
         self.qaction.triggered.connect(self.save_to_disk)

src/calibre/gui2/actions/show_book_details.py

@@ -16,6 +16,7 @@ class ShowBookDetailsAction(InterfaceAction):
     action_spec = (_('Show book details'), 'dialog_information.png', None,
             _('I'))
     dont_add_to = frozenset(['toolbar-device', 'context-menu-device'])
+    action_type = 'current'
 
     def genesis(self):
         self.qaction.triggered.connect(self.show_book_info)

src/calibre/gui2/actions/similar_books.py

@@ -16,6 +16,7 @@ class SimilarBooksAction(InterfaceAction):
     name = 'Similar Books'
     action_spec = (_('Similar books...'), None, None, None)
     popup_type = QToolButton.InstantPopup
+    action_type = 'current'
 
     def genesis(self):
         m = QMenu(self.gui)

src/calibre/gui2/actions/view.py

@@ -22,6 +22,7 @@ class ViewAction(InterfaceAction):
 
     name = 'View'
     action_spec = (_('View'), 'view.png', None, _('V'))
+    action_type = 'current'
 
     def genesis(self):
         self.persistent_files = []

src/calibre/gui2/layout.py

@@ -61,7 +61,7 @@ class LocationManager(QObject): # {{{
 
         ac('library', _('Library'), 'lt.png',
                 _('Show books in calibre library'))
-        ac('main', _('Reader'), 'reader.png',
+        ac('main', _('Device'), 'reader.png',
                 _('Show books in the main memory of the device'))
         ac('carda', _('Card A'), 'sd.png',
                 _('Show books in storage card A'))
@@ -197,11 +197,21 @@ class SearchBar(QWidget): # {{{
 
 # }}}
 
+class Spacer(QWidget):
+
+    def __init__(self, parent):
+        QWidget.__init__(self, parent)
+        self.l = QHBoxLayout()
+        self.setLayout(self.l)
+        self.l.addStretch(10)
+
+
 class ToolBar(QToolBar): # {{{
 
-    def __init__(self, donate, location_manager, parent):
+    def __init__(self, donate, location_manager, child_bar, parent):
         QToolBar.__init__(self, parent)
         self.gui = parent
+        self.child_bar = child_bar
         self.setContextMenuPolicy(Qt.PreventContextMenu)
         self.setMovable(False)
         self.setFloatable(False)
@@ -223,16 +233,19 @@ class ToolBar(QToolBar): # {{{
         sz = gprefs['toolbar_icon_size']
         sz = {'small':24, 'medium':48, 'large':64}[sz]
         self.setIconSize(QSize(sz, sz))
+        self.child_bar.setIconSize(QSize(sz, sz))
         style = Qt.ToolButtonTextUnderIcon
         if gprefs['toolbar_text'] == 'never':
             style = Qt.ToolButtonIconOnly
         self.setToolButtonStyle(style)
+        self.child_bar.setToolButtonStyle(style)
         self.donate_button.set_normal_icon_size(sz, sz)
 
     def contextMenuEvent(self, *args):
         pass
 
     def build_bar(self):
+        self.child_bar.setVisible(gprefs['show_child_bar'])
         self.showing_donate = False
         showing_device = self.location_manager.has_device
         actions = '-device' if showing_device else ''
@@ -244,10 +257,16 @@ class ToolBar(QToolBar): # {{{
                 m.setVisible(False)
 
         self.clear()
+        self.child_bar.clear()
         self.added_actions = []
+        self.spacers = [Spacer(self.child_bar), Spacer(self.child_bar),
+                Spacer(self), Spacer(self)]
+        self.child_bar.addWidget(self.spacers[0])
+        if gprefs['show_child_bar']:
+            self.addWidget(self.spacers[2])
 
         for what in actions:
-            if what is None:
+            if what is None and not gprefs['show_child_bar']:
                 self.addSeparator()
             elif what == 'Location Manager':
                 for ac in self.location_manager.available_actions:
@@ -262,12 +281,21 @@ class ToolBar(QToolBar): # {{{
                 self.showing_donate = True
             elif what in self.gui.iactions:
                 action = self.gui.iactions[what]
-                self.addAction(action.qaction)
+                bar = self
+                if action.action_type == 'current' and gprefs['show_child_bar']:
+                    bar = self.child_bar
+                bar.addAction(action.qaction)
                 self.added_actions.append(action.qaction)
                 self.setup_tool_button(action.qaction, action.popup_type)
 
+        self.child_bar.addWidget(self.spacers[1])
+        if gprefs['show_child_bar']:
+            self.addWidget(self.spacers[3])
+
     def setup_tool_button(self, ac, menu_mode=None):
         ch = self.widgetForAction(ac)
+        if ch is None:
+            ch = self.child_bar.widgetForAction(ac)
         ch.setCursor(Qt.PointingHandCursor)
         ch.setAutoRaise(True)
         if ac.menu() is not None and menu_mode is not None:
@@ -280,7 +308,8 @@ class ToolBar(QToolBar): # {{{
         if p == 'never':
             style = Qt.ToolButtonIconOnly
 
-        if p == 'auto' and self.preferred_width > self.width()+35:
+        if p == 'auto' and self.preferred_width > self.width()+35 and \
+                not gprefs['show_child_bar']:
             style = Qt.ToolButtonIconOnly
 
         self.setToolButtonStyle(style)
@@ -309,9 +338,11 @@ class MainWindowMixin(object): # {{{
         self.iactions['Fetch News'].init_scheduler(db)
 
         self.search_bar = SearchBar(self)
+        self.child_bar = QToolBar(self)
         self.tool_bar = ToolBar(self.donate_button,
-                self.location_manager, self)
+                self.location_manager, self.child_bar, self)
         self.addToolBar(Qt.TopToolBarArea, self.tool_bar)
+        self.addToolBar(Qt.BottomToolBarArea, self.child_bar)
 
         l = self.centralwidget.layout()
         l.addWidget(self.search_bar)

src/calibre/gui2/preferences/look_feel.py

@@ -46,6 +46,7 @@ class ConfigWidget(ConfigWidgetBase, Ui_Form):
         r('use_roman_numerals_for_series_number', config)
         r('separate_cover_flow', config, restart_required=True)
         r('search_as_you_type', config)
+        r('show_child_bar', gprefs)
 
         choices = [(_('Small'), 'small'), (_('Medium'), 'medium'),
             (_('Large'), 'large')]

src/calibre/gui2/preferences/look_feel.ui

@@ -173,6 +173,13 @@
         </property>
        </widget>
       </item>
+      <item row="2" column="0" colspan="2">
+       <widget class="QCheckBox" name="opt_show_child_bar">
+        <property name="text">
+         <string>&amp;Split the toolbar into two toolbars</string>
+        </property>
+       </widget>
+      </item>
      </layout>
     </widget>
    </item>