comparison framework/MainWindowBase.cpp @ 128:09e6016c5436

* Add playback controls to data edit dialog
* Make data edit dialog close properly when layer is removed from view
* More fixes to playback tracking & selection model in data edit dialog
* Remove edit button from data edit dialog for now

 
     if (!m_playTarget) {
         emit hideSplash();
 
         if (factory->isAutoCallbackTarget(targetName)) {
+            QMessageBox::warning
 	    (this, tr("Couldn't open audio device"),
 	     tr("<b>No audio available</b><p>Could not open an audio device for playback.<p>Automatic audio device detection failed. Audio playback will not be available during this session.</p>"),
 	     QMessageBox::Ok);
         } else {
             QMessageBox::warning

     }
 
     QString title = layer->getLayerPresentationName();
 
     ModelDataTableDialog *dialog = new ModelDataTableDialog(tabular, title, this);
-    dialog->setAttribute(Qt::WA_DeleteOnClose); // see below
-
+    dialog->setAttribute(Qt::WA_DeleteOnClose);
+    
+    connectLayerEditDialog(dialog);
+
+    m_layerDataDialogMap[layer] = dialog;
+    m_viewDataDialogMap[pane].insert(dialog);
+
+    dialog->show();
+}
+
+void
+MainWindowBase::connectLayerEditDialog(ModelDataTableDialog *dialog)
+{
     connect(m_viewManager,
             SIGNAL(globalCentreFrameChanged(unsigned long)),
             dialog,
             SLOT(userScrolledToFrame(unsigned long)));
 

 
     connect(dialog,
             SIGNAL(scrollToFrame(unsigned long)),
             m_viewManager,
             SLOT(setGlobalCentreFrame(unsigned long)));
-    
-    m_layerDataDialogMap[layer] = dialog;
-    m_viewDataDialogMap[pane].insert(dialog);
-
-    dialog->show();
-}
+}    
 
 void
 MainWindowBase::previousPane()
 {
     if (!m_paneStack) return;

 }
 
 void
 MainWindowBase::layerAboutToBeDeleted(Layer *layer)
 {
-    if (m_layerDataDialogMap.find(layer) != m_layerDataDialogMap.end()) {
-
-        ModelDataTableDialog *dialog = m_layerDataDialogMap[layer];
-
-        for (ViewDataDialogMap::iterator vi = m_viewDataDialogMap.begin();
-             vi != m_viewDataDialogMap.end(); ++vi) {
-            vi->second.erase(dialog);
-        }
-
-        m_layerDataDialogMap.erase(layer);
-        delete dialog;
-    }
-
-//    std::cerr << "MainWindowBase::layerAboutToBeDeleted(" << layer << ")" << std::endl;
+    std::cerr << "MainWindowBase::layerAboutToBeDeleted(" << layer << ")" << std::endl;
+
+    removeLayerEditDialog(layer);
+
     if (m_timeRulerLayer && (layer == m_timeRulerLayer)) {
 //	std::cerr << "(this is the time ruler layer)" << std::endl;
 	m_timeRulerLayer = 0;
     }
 }
 
 void
 MainWindowBase::layerInAView(Layer *layer, bool inAView)
 {
-//    std::cerr << "MainWindowBase::layerInAView(" << layer << "," << inAView << ")" << std::endl;
+    std::cerr << "MainWindowBase::layerInAView(" << layer << "," << inAView << ")" << std::endl;
+
+    if (!inAView) removeLayerEditDialog(layer);
 
     // Check whether we need to add or remove model from play source
     Model *model = layer->getModel();
     if (model) {
         if (inAView) {

 
     updateMenuStates();
 }
 
 void
+MainWindowBase::removeLayerEditDialog(Layer *layer)
+{
+    if (m_layerDataDialogMap.find(layer) != m_layerDataDialogMap.end()) {
+
+        ModelDataTableDialog *dialog = m_layerDataDialogMap[layer];
+
+        for (ViewDataDialogMap::iterator vi = m_viewDataDialogMap.begin();
+             vi != m_viewDataDialogMap.end(); ++vi) {
+            vi->second.erase(dialog);
+        }
+
+        m_layerDataDialogMap.erase(layer);
+        delete dialog;
+    }
+}
+
+void
 MainWindowBase::modelAdded(Model *model)
 {
 //    std::cerr << "MainWindowBase::modelAdded(" << model << ")" << std::endl;
     m_playSource->addModel(model);
 }