comparison src/MainWindow.cpp @ 291:cddada03e222

Fix to the button enabling problem from feature #835

     connect(this, SIGNAL(canPlay(bool)), m_showAudio, SLOT(setEnabled(bool)));
 
     m_playAudio = toolbar->addAction(il.load("speaker"), tr("Play Audio"));
     m_playAudio->setCheckable(true);
     connect(m_playAudio, SIGNAL(triggered()), this, SLOT(playAudioToggled()));
-    connect(this, SIGNAL(canPlay(bool)), m_playAudio, SLOT(setEnabled(bool)));
+    connect(this, SIGNAL(canPlayWaveform(bool)), m_playAudio, SLOT(setEnabled(bool)));
 
     toolbar->addWidget(m_gainAudio);
     toolbar->addWidget(m_panAudio);
 
     // Pitch (f0)

     connect(this, SIGNAL(canPlay(bool)), m_showPitch, SLOT(setEnabled(bool)));
 
     m_playPitch = toolbar->addAction(il.load("speaker"), tr("Play Pitch Track"));
     m_playPitch->setCheckable(true);
     connect(m_playPitch, SIGNAL(triggered()), this, SLOT(playPitchToggled()));
-    connect(this, SIGNAL(canPlay(bool)), m_playPitch, SLOT(setEnabled(bool))); // JTEST: this resets the enabled state of m_playPitch.
+    connect(this, SIGNAL(canPlayPitch(bool)), m_playPitch, SLOT(setEnabled(bool)));
 
     toolbar->addWidget(m_gainPitch);
     toolbar->addWidget(m_panPitch);
 
     // Notes

     connect(this, SIGNAL(canPlay(bool)), m_showNotes, SLOT(setEnabled(bool)));
 
     m_playNotes = toolbar->addAction(il.load("speaker"), tr("Play Notes"));
     m_playNotes->setCheckable(true);
     connect(m_playNotes, SIGNAL(triggered()), this, SLOT(playNotesToggled()));
-    connect(this, SIGNAL(canPlay(bool)), m_playNotes, SLOT(setEnabled(bool)));
+    connect(this, SIGNAL(canPlayNotes(bool)), m_playNotes, SLOT(setEnabled(bool)));
 
     toolbar->addWidget(m_gainNotes);
     toolbar->addWidget(m_panNotes);
 
     // Spectrogram

     Layer *currentLayer = 0;
 
     if (m_paneStack) currentPane = m_paneStack->getCurrentPane();
     if (currentPane) currentLayer = currentPane->getSelectedLayer();
 
+    bool haveMainModel =
+	(getMainModel() != 0);
+    bool havePlayTarget =
+	(m_playTarget != 0);
     bool haveCurrentPane =
         (currentPane != 0);
     bool haveCurrentLayer =
         (haveCurrentPane &&
          (currentLayer != 0));

     emit canChangePlaybackSpeed(true);
     int v = m_playSpeed->value();
     emit canSpeedUpPlayback(v < m_playSpeed->maximum());
     emit canSlowDownPlayback(v > m_playSpeed->minimum());
 
+    bool haveWaveform =
+        m_analyser->isVisible(Analyser::Audio) &&
+        m_analyser->getLayer(Analyser::Audio);
+
     bool havePitchTrack = 
         m_analyser->isVisible(Analyser::PitchTrack) &&
         m_analyser->getLayer(Analyser::PitchTrack);
 
     bool haveNotes = 

         m_analyser->getLayer(Analyser::Notes);
 
     emit canExportPitchTrack(havePitchTrack);
     emit canExportNotes(haveNotes);
     emit canSnapNotes(haveSelection && haveNotes);
+
+    emit canPlayWaveform(haveWaveform && haveMainModel && havePlayTarget);
+    emit canPlayPitch(havePitchTrack && haveMainModel && havePlayTarget);
+    emit canPlayNotes(haveNotes && haveMainModel && havePlayTarget);
 
     if (pitchCandidatesVisible) {
         m_showCandidatesAction->setText(tr("Hide Pitch Candidates"));
         m_showCandidatesAction->setStatusTip(tr("Remove the display of alternate pitch candidates for the selected region"));
     } else {

 void
 MainWindow::showAudioToggled()
 {
     m_analyser->toggleVisible(Analyser::Audio);
 
-    // JTEST
-    if (!m_analyser->isVisible(Analyser::Audio))
-    {
-        m_analyser->setAudible(Analyser::Audio,false);
-        m_playAudio->setChecked(false);
-        m_playAudio->setEnabled(false);
-    }
-    else
-    {
-        m_playAudio->setEnabled(true);
-    }
+    QSettings settings;
+    settings.beginGroup("MainWindow");
+
+    bool playOn = false;
+    if (m_analyser->isVisible(Analyser::Audio)) {
+        // just switched layer on; check whether playback was also on previously
+        playOn = settings.value("playaudiowas", true).toBool();
+    } else {
+        settings.setValue("playaudiowas", m_playAudio->isChecked());
+    }
+    m_analyser->setAudible(Analyser::Audio, playOn);
+    m_playAudio->setChecked(playOn);
+
+    settings.endGroup();
+
+    updateMenuStates();
 }
 
 void
 MainWindow::showPitchToggled()
 {
     m_analyser->toggleVisible(Analyser::PitchTrack);
 
-    // JTEST
-    if (!m_analyser->isVisible(Analyser::PitchTrack))
-    {
-        m_analyser->setAudible(Analyser::PitchTrack,false);
-        m_playPitch->setChecked(false);
-        m_playPitch->setEnabled(false);
-    }
-    else
-    {
-        m_playPitch->setEnabled(true);
-    }
+    QSettings settings;
+    settings.beginGroup("MainWindow");
+
+    bool playOn = false;
+    if (m_analyser->isVisible(Analyser::PitchTrack)) {
+        // just switched layer on; check whether playback was also on previously
+        playOn = settings.value("playpitchwas", true).toBool();
+    } else {
+        settings.setValue("playpitchwas", m_playPitch->isChecked());
+    }
+    m_analyser->setAudible(Analyser::PitchTrack, playOn);
+    m_playPitch->setChecked(playOn);
+
+    settings.endGroup();
+
+    updateMenuStates();
 }
 
 void
 MainWindow::showSpectToggled()
 {

 void
 MainWindow::showNotesToggled()
 {
     m_analyser->toggleVisible(Analyser::Notes);
 
-    // JTEST
-    if (!m_analyser->isVisible(Analyser::Notes))
-    {
-        m_analyser->setAudible(Analyser::Notes,false);
-        m_playNotes->setChecked(false);
-        m_playNotes->setEnabled(false);
-    }
-    else
-    {
-        m_playNotes->setEnabled(true);
-    }
+    QSettings settings;
+    settings.beginGroup("MainWindow");
+
+    bool playOn = false;
+    if (m_analyser->isVisible(Analyser::Notes)) {
+        // just switched layer on; check whether playback was also on previously
+        playOn = settings.value("playnoteswas", true).toBool();
+    } else {
+        settings.setValue("playnoteswas", m_playNotes->isChecked());
+    }
+    m_analyser->setAudible(Analyser::Notes, playOn);
+    m_playNotes->setChecked(playOn);
+
+    settings.endGroup();
+
+    updateMenuStates();
 }
 
 void
 MainWindow::playAudioToggled()
 {