Fixed: Markers and synth windows don't open at all on some desktops like Lubuntu.

See ChangeLog for details.

3 files changed, 26 insertions, 13 deletions

diff --git a/muse2/muse/app.cpp b/muse2/muse/app.cpp
index e0fffd4d..45492889 100644
--- a/muse2/muse/app.cpp
+++ b/muse2/muse/app.cpp
@@ -1555,11 +1555,14 @@ void MusE::showMarker(bool flag)
             markerView = new MusEGui::MarkerView(this);
 
             connect(markerView, SIGNAL(closed()), SLOT(markerClosed()));
-            markerView->show();
+            // Nov 21, 2012 Hey this causes the thing not to open at all, EVER, on Lubuntu and some others!  
+            //markerView->show();  // ??? REMOVE Tim. Superfluous?
             toplevels.push_back(markerView);
             }
-      markerView->setVisible(flag);
-      viewMarkerAction->setChecked(flag);
+      if(markerView->isVisible() != flag)      
+        markerView->setVisible(flag);
+      if(viewMarkerAction->isChecked() != flag)
+        viewMarkerAction->setChecked(flag);   // ??? TEST: Recursion? Does this call toggleMarker if called from menu?  No. Why? It should. REMOVE Tim. Or keep.
       if (!flag)
         if (currentMenuSharingTopwin == markerView)
           setCurrentMenuSharingTopwin(NULL);
@@ -1573,7 +1576,8 @@ void MusE::showMarker(bool flag)
 
 void MusE::markerClosed()
       {
-      viewMarkerAction->setChecked(false);
+      if(viewMarkerAction->isChecked())
+        viewMarkerAction->setChecked(false); // ??? TEST: Recursion? Does this call toggleMarker? Yes. REMOVE Tim. Or keep.
       if (currentMenuSharingTopwin == markerView)
         setCurrentMenuSharingTopwin(NULL);
 
@@ -1609,8 +1613,10 @@ void MusE::toggleArranger(bool checked)
 
 void MusE::showArranger(bool flag)
       {
-      arrangerView->setVisible(flag);
-      viewArrangerAction->setChecked(flag);
+      if(arrangerView->isVisible() != flag)  
+        arrangerView->setVisible(flag);
+      if(viewArrangerAction->isChecked() != flag)
+        viewArrangerAction->setChecked(flag);
       if (!flag)
         if (currentMenuSharingTopwin == arrangerView)
           setCurrentMenuSharingTopwin(NULL);
@@ -1623,7 +1629,8 @@ void MusE::showArranger(bool flag)
 
 void MusE::arrangerClosed()
       {
-      viewArrangerAction->setChecked(false);
+      if(viewArrangerAction->isChecked())  
+        viewArrangerAction->setChecked(false);
       updateWindowMenu();
 
       // focus the last activated topwin which is not the arranger view
@@ -1656,8 +1663,10 @@ void MusE::toggleTransport(bool checked)
 
 void MusE::showTransport(bool flag)
       {
-      transport->setVisible(flag);
-      viewTransportAction->setChecked(flag);
+      if(transport->isVisible() != flag)
+        transport->setVisible(flag);
+      if(viewTransportAction->isChecked() != flag)
+        viewTransportAction->setChecked(flag);
       }
 
 //---------------------------------------------------------
diff --git a/muse2/muse/arranger/arrangerview.cpp b/muse2/muse/arranger/arrangerview.cpp
index a2024159..e91d05f6 100644
--- a/muse2/muse/arranger/arrangerview.cpp
+++ b/muse2/muse/arranger/arrangerview.cpp
@@ -359,8 +359,10 @@ ArrangerView::ArrangerView(QWidget* parent)
   // for certain window managers, e.g xfce, this window is
   // is displayed although not specifically set to show();
   // bug: 2811156     Softsynth GUI unclosable with XFCE4 (and a few others)
-  show();
-  hide();
+  // Nov 21, 2012 Hey this causes the thing not to open at all, EVER, on Lubuntu and some others!
+  // And we had a request to remove this from a knowledgable tester. REMOVE Tim.
+  ///show();
+  ///hide();
 }
 
 ArrangerView::~ArrangerView()
diff --git a/muse2/muse/marker/markerview.cpp b/muse2/muse/marker/markerview.cpp
index 5a5f9a2a..39d8e748 100644
--- a/muse2/muse/marker/markerview.cpp
+++ b/muse2/muse/marker/markerview.cpp
@@ -281,8 +281,10 @@ MarkerView::MarkerView(QWidget* parent)
       // for certain window managers, e.g xfce, this window is
       // is displayed although not specifically set to show();
       // bug: 2811156  	 Softsynth GUI unclosable with XFCE4 (and a few others)
-      show();
-      hide();
+      // Nov 21, 2012 Hey this causes the thing not to open at all, EVER, on Lubuntu and some others!
+      // And we had a request to remove this from a knowledgable tester. REMOVE Tim.
+      ///show();
+      ///hide();
       }
 
 //---------------------------------------------------------