Slight improvement to playcontrol and its tests

diff --git a/hand_tests/fake_player.py b/hand_tests/fake_player.py
index 78e6840..1a215ba 100644 (file)
--- a/hand_tests/fake_player.py
+++ b/hand_tests/fake_player.py
@@ -50,7 +50,7 @@ class FakePlayer(gobject.GObject):
                self._layout.pack_start(self._stopButton)
                self._layout.pack_start(self._canNavigate)
 
-               self._state = "play"
+               self._state = "stop"
 
        @property
        def toplevel(self):
@@ -69,13 +69,16 @@ class FakePlayer(gobject.GObject):
                return self._state
 
        def _state_change(self, widget, state):
+               _moduleLogger.info("User changed state")
                self.emit("state_change", state)
                self._state = state
 
        def _navigate_change(self, widget):
+               _moduleLogger.info("User changed nav")
                self.emit("navigate_change", self._canNavigate.get_active())
 
        def _title_change(self, widget):
+               _moduleLogger.info("User changed title")
                self.emit("title_change", self._title.get_text())
 
        def play(self):

diff --git a/hand_tests/test_controls.py b/hand_tests/test_controls.py
index fcfe8bc..be2c3e8 100755 (executable)
--- a/hand_tests/test_controls.py
+++ b/hand_tests/test_controls.py
@@ -30,4 +30,6 @@ if __name__ == "__main__":
        window.connect("destroy", lambda w: gtk.main_quit())
        window.show_all()
 
+       sp.refresh()
+
        gtk.main()

diff --git a/src/playcontrol.py b/src/playcontrol.py
index adcf27e..c0fa3b5 100644 (file)
--- a/src/playcontrol.py
+++ b/src/playcontrol.py
@@ -120,12 +120,15 @@ class PlayControl(object):
                if newState == "play":
                        self._pause.show()
                        self._play.hide()
+                       self.toplevel.show()
                elif newState == "pause":
                        self._pause.hide()
                        self._play.show()
+                       self.toplevel.show()
                elif newState == "stop":
                        self._pause.hide()
                        self._play.show()
+                       self.toplevel.hide()
 
        @misc_utils.log_exception(_moduleLogger)
        def _on_player_state_change(self, player, newState):