From 43fba1faa9240b969aa2537182e15e77165bc556 Mon Sep 17 00:00:00 2001 From: Kovid Goyal Date: Thu, 5 Jul 2018 02:20:09 +0530 Subject: [PATCH] E-book viewer: Add a shortcut (Ctrl+Alt_B) to toggle the bookmarks panel. Fixes #1780097 [keyboard shortcut for 'show/hide bookmarks' in ebook viewer](https://bugs.launchpad.net/calibre/+bug/1780097) --- src/calibre/gui2/viewer/keys.py | 3 +++ src/calibre/gui2/viewer/main.py | 6 ++++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/calibre/gui2/viewer/keys.py b/src/calibre/gui2/viewer/keys.py index 3cea69dc17..178aee985a 100644 --- a/src/calibre/gui2/viewer/keys.py +++ b/src/calibre/gui2/viewer/keys.py @@ -90,6 +90,9 @@ SHORTCUTS = { 'Bookmark': (['Ctrl+B'], _('Bookmark the current location')), + 'Toggle bookmarks': (['Ctrl+Alt+B'], + _('Show/hide bookmarks')), + 'Reload': (['Ctrl+R', 'F5'], _('Reload the current book')), diff --git a/src/calibre/gui2/viewer/main.py b/src/calibre/gui2/viewer/main.py index ef4fa59f05..2eb70877e2 100644 --- a/src/calibre/gui2/viewer/main.py +++ b/src/calibre/gui2/viewer/main.py @@ -949,8 +949,9 @@ class EbookViewer(MainWindow): def build_bookmarks_menu(self, bookmarks): self.bookmarks_menu.clear() sc = _(' or ').join(self.view.shortcuts.get_shortcuts('Bookmark')) - self.bookmarks_menu.addAction(_("Bookmark this location [%s]") % sc, self.bookmark) - self.bookmarks_menu.addAction(_("Show/hide bookmarks"), self.bookmarks_dock.toggleViewAction().trigger) + tc = _(' or ').join(self.view.shortcuts.get_shortcuts('Toggle bookmarks')) + self.bookmarks_menu.addAction(_("Bookmark this location") + '\t' + sc, self.bookmark) + self.bookmarks_menu.addAction(_("Show/hide bookmarks") + '\t' + tc, self.bookmarks_dock.toggleViewAction().trigger) self.bookmarks_menu.addSeparator() current_page = None self.existing_bookmarks = [] @@ -1143,6 +1144,7 @@ class EbookViewer(MainWindow): 'Lookup word': self.view.dictionary_action, 'Next occurrence': self.view.search_action, 'Bookmark': bac, + 'Toggle bookmarks': self.bookmarks_dock.toggleViewAction(), 'Reload': self.action_reload, 'Table of Contents': self.action_table_of_contents, 'Print': self.action_print,