openlp-core team mailing list archive

[Tomas Groth <tomasgroth@xxxxxxxx>]

Tomas Groth has proposed merging lp:~tomasgroth/openlp/bug1265368 into lp:openlp.

Requested reviews:
  Raoul Snyman (raoul-snyman)
Related bugs:
  Bug #1265368 in OpenLP: "[regression] Traceback in web remote"
  https://bugs.launchpad.net/openlp/+bug/1265368

For more details, see:
https://code.launchpad.net/~tomasgroth/openlp/bug1265368/+merge/232764

Forward ported fix from 2.0.
-- 
https://code.launchpad.net/~tomasgroth/openlp/bug1265368/+merge/232764
Your team OpenLP Core is subscribed to branch lp:openlp.

=== modified file 'openlp/core/ui/slidecontroller.py'
--- openlp/core/ui/slidecontroller.py	2014-07-12 23:47:53 +0000
+++ openlp/core/ui/slidecontroller.py	2014-08-29 20:12:13 +0000
@@ -374,7 +374,8 @@
                                                       triggers=self._slide_shortcut_activated) for s in shortcuts])
             self.shortcut_timer.timeout.connect(self._slide_shortcut_activated)
         # Signals
-        self.preview_widget.itemSelectionChanged.connect(self.on_slide_selected)
+        self.preview_widget.clicked.connect(self.on_slide_selected)
+        self.preview_widget.verticalHeader().sectionClicked.connect(self.on_slide_selected)
         if self.is_live:
             # Need to use event as called across threads and UI is updated
             QtCore.QObject.connect(self, QtCore.SIGNAL('slidecontroller_toggle_display'), self.toggle_display)