From 1f319cd2d14b9f0486147c2e46f815e979432956 Mon Sep 17 00:00:00 2001
From: David Capello <davidcapello@gmail.com>
Date: Thu, 11 Feb 2016 21:39:22 -0300
Subject: [PATCH] Don't change MiniPlayButton state in preview window when we
 scroll it (fix #958)

---
 src/app/ui/preview_editor.cpp | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/src/app/ui/preview_editor.cpp b/src/app/ui/preview_editor.cpp
index 77d71c247..ad32a641d 100644
--- a/src/app/ui/preview_editor.cpp
+++ b/src/app/ui/preview_editor.cpp
@@ -21,7 +21,6 @@
 #include "app/ui/editor/editor.h"
 #include "app/ui/editor/editor_view.h"
 #include "app/ui/editor/navigate_state.h"
-#include "app/ui/editor/play_state.h"
 #include "app/ui/skin/skin_button.h"
 #include "app/ui/skin/skin_theme.h"
 #include "app/ui/status_bar.h"
@@ -102,8 +101,8 @@ public:
 
   bool isPlaying() const { return m_isPlaying; }
 
-  void stop() {
-    m_isPlaying = false;
+  void setPlaying(bool state) {
+    m_isPlaying = state;
     setupIcons();
   }
 
@@ -398,14 +397,9 @@ void PreviewEditorWindow::uncheckCenterButton()
 
 void PreviewEditorWindow::onStateChanged(Editor* editor)
 {
-  EditorStatePtr state = editor->getState();
-  PlayState* playState = (state ? dynamic_cast<PlayState*>(state.get()): nullptr);
-  if (!playState) {
-    // We have to switch the "play button" state to "play" because the
-    // editor animation has just stopped. This happens when we use
-    // "play once" option and the PlayState stops automatically.
-    m_playButton->stop();
-  }
+  // Sync editor playing state with MiniPlayButton state
+  if (m_playButton->isPlaying() != editor->isPlaying())
+    m_playButton->setPlaying(editor->isPlaying());
 }
 
 void PreviewEditorWindow::onScrollChanged(Editor* miniEditor)