Fix #963559 (Keyboard shortcuts for text formatting not working in 0.8.44)

src/calibre/gui2/comments_editor.py

@@ -9,10 +9,10 @@
 
 from lxml import html
 
-from PyQt4.Qt import QApplication, QFontInfo, QSize, QWidget, QPlainTextEdit, \
-    QToolBar, QVBoxLayout, QAction, QIcon, Qt, QTabWidget, QUrl, \
-    QSyntaxHighlighter, QColor, QChar, QColorDialog, QMenu, QInputDialog, \
-    QHBoxLayout
+from PyQt4.Qt import (QApplication, QFontInfo, QSize, QWidget, QPlainTextEdit,
+    QToolBar, QVBoxLayout, QAction, QIcon, Qt, QTabWidget, QUrl,
+    QSyntaxHighlighter, QColor, QChar, QColorDialog, QMenu, QInputDialog,
+    QHBoxLayout, QKeySequence)
 from PyQt4.QtWebKit import QWebView, QWebPage
 
 from calibre.ebooks.chardet import xml_to_unicode
@@ -32,6 +32,7 @@ def __init__(self, wac, icon, text, checkable, view):
                 type=Qt.QueuedConnection)
         self.page_action.changed.connect(self.update_state,
                 type=Qt.QueuedConnection)
+        self.update_state()
 
     @property
     def page_action(self):
@@ -66,6 +67,12 @@ def __init__(self, parent=None):
 
         self.comments_pat = re.compile(r'<!--.*?-->', re.DOTALL)
 
+        extra_shortcuts = {
+                'ToggleBold': 'Bold',
+                'ToggleItalic': 'Italic',
+                'ToggleUnderline': 'Underline',
+        }
+
         for wac, name, icon, text, checkable in [
                 ('ToggleBold', 'bold', 'format-text-bold', _('Bold'), True),
                 ('ToggleItalic', 'italic', 'format-text-italic', _('Italic'),
@@ -106,6 +113,9 @@ def __init__(self, parent=None):
             ]:
             ac = PageAction(wac, icon, text, checkable, self)
             setattr(self, 'action_'+name, ac)
+            ss = extra_shortcuts.get(wac, None)
+            if ss:
+                ac.setShortcut(QKeySequence(getattr(QKeySequence, ss)))
 
         self.action_color = QAction(QIcon(I('format-text-color')), _('Foreground color'),
                 self)