Fixed track list midi port popup menu - too many ports listed.

Seperated empty ports into submenu.

2 files changed, 30 insertions, 8 deletions

diff --git a/muse2/muse/arranger/tlist.cpp b/muse2/muse/arranger/tlist.cpp
index 98c4a595..87179575 100644
--- a/muse2/muse/arranger/tlist.cpp
+++ b/muse2/muse/arranger/tlist.cpp
@@ -602,7 +602,7 @@ void TList::portsPopupMenu(MusECore::Track* t, int x, int y)
                   else   
                     port = track->outPort();
                     
-                  QMenu* p = MusECore::midiPortsPopup(0, port);
+                  QMenu* p = MusECore::midiPortsPopup(this, port);     // 0, port);
                   QAction* act = p->exec(mapToGlobal(QPoint(x, y)), 0);
                   if (act) {
                         int n = act->data().toInt();
diff --git a/muse2/muse/midiport.cpp b/muse2/muse/midiport.cpp
index a8a248cb..49934d7b 100644
--- a/muse2/muse/midiport.cpp
+++ b/muse2/muse/midiport.cpp
@@ -37,6 +37,7 @@
 #include "synth.h"
 #include "app.h"
 #include "song.h"
+#include "menutitleitem.h"
 
 //#ifdef DSSI_SUPPORT
 //#include "dssihost.h"
@@ -316,20 +317,41 @@ int MidiPort::portno() const
 //   midiPortsPopup
 //---------------------------------------------------------
 
-//QPopupMenu* midiPortsPopup(QWidget* parent)
 QMenu* midiPortsPopup(QWidget* parent, int checkPort)
       {
       QMenu* p = new QMenu(parent);
+      p->addAction(new MusEGui::MenuTitleItem("Port / Device", p));
+      QMenu* subp = 0;
+      QAction *act = 0;
+      QString name;
       for (int i = 0; i < MIDI_PORTS; ++i) {
             MidiPort* port = &MusEGlobal::midiPorts[i];
-            QString name;
             name.sprintf("%d:%s", port->portno()+1, port->portname().toLatin1().constData());
-	    QAction *act = p->addAction(name);
-	    act->setData(i);
-            
-            if(i == checkPort)
-              act->setChecked(true);
+            if(port->device() || (i == checkPort))   
+            {  
+              act = p->addAction(name);
+              act->setData(i);
+              act->setCheckable(true);
+              act->setChecked(i == checkPort);
+            }  
+
+            if(!port->device())
+            {
+              if(!subp)                  // No submenu yet? Create it now.
+              {
+                subp = new QMenu(p);
+                subp->setTitle(subp->tr("Empty"));
+                subp->addAction(new MusEGui::MenuTitleItem("Empty Ports", subp));
+              }  
+              //act = subp->addAction(name);               // No need for all those "<None>" names. 
+              act = subp->addAction(QString().setNum(i+1));
+              act->setData(i);
+              act->setCheckable(true);
+              act->setChecked(i == checkPort);
+            }  
           }  
+      if(subp)
+        p->addMenu(subp);
       return p;
       }