new color scheme on fretboard, maekes each ocatave appear in different color

8 files changed, 77 insertions, 13 deletions

diff --git a/noncore/multimedia/tonleiter/editinst.cpp b/noncore/multimedia/tonleiter/editinst.cpp
index 120c3ac..f172ef8 100644
--- a/noncore/multimedia/tonleiter/editinst.cpp
+++ b/noncore/multimedia/tonleiter/editinst.cpp
@@ -40,3 +40,3 @@ Menu::InstEditDialog::InstEditDialog(TonleiterData* data,QWidget* parent,const c
 
-    QScrollView* scrollview=new QScrollView(this);
+    scrollview=new QScrollView(this);
     scrollview->setVScrollBarMode(QScrollView::AlwaysOn);
@@ -48,3 +48,2 @@ Menu::InstEditDialog::InstEditDialog(TonleiterData* data,QWidget* parent,const c
     scrollview->setSizePolicy(QSizePolicy(QSizePolicy::MinimumExpanding,QSizePolicy::Maximum));
-    //scrollview->setSizePolicy(QSizePolicy(QSizePolicy::MinimumExpanding,QSizePolicy::MinimumExpanding));
     masterlayout->addWidget(scrollview);
@@ -94,2 +93,3 @@ void Menu::InstEditDialog::saveInstrument()
 {
+    //auto delete is set to true so widgets _should_ be deleted
     stringlist.clear();
@@ -99,3 +99,5 @@ void Menu::InstEditDialog::loadInstrument()
 {
+    stringlist.clear();
     Instrument inst=data->getInstrument(instid);
+    qDebug("load Instrument : %d strings",inst.noOfStrings());
     for(uint s=0;s<inst.noOfStrings();s++)
diff --git a/noncore/multimedia/tonleiter/editinst.h b/noncore/multimedia/tonleiter/editinst.h
index 948a2de..a90c9ec 100644
--- a/noncore/multimedia/tonleiter/editinst.h
+++ b/noncore/multimedia/tonleiter/editinst.h
@@ -29,2 +29,3 @@ namespace Menu
 
+        QScrollView* scrollview;
         QVBox* stringwidget;
diff --git a/noncore/multimedia/tonleiter/fretboard.cpp b/noncore/multimedia/tonleiter/fretboard.cpp
index 7936c83..7e63e01 100644
--- a/noncore/multimedia/tonleiter/fretboard.cpp
+++ b/noncore/multimedia/tonleiter/fretboard.cpp
@@ -55,2 +55,8 @@ void Graph::FretBoard::resizeEvent(QResizeEvent* re)
 //****************************************************************************
+void Graph::FretBoard::mouseReleaseEvent(QMouseEvent* me)
+{
+    Q_UNUSED(me);
+    emit pressed();
+}
+//****************************************************************************
 void Graph::FretBoard::paintBoard(QPainter* p)
@@ -123,5 +129,3 @@ void Graph::FretBoard::paintScale(QPainter* p)
     Scale scale=data->getScale(scaleid);
-
-    int colorstart=255%inst.noOfOctaves();
-    int colorintervall=(255-colorstart)/inst.noOfOctaves();
+    int baseoctave=Note::octaveOfBaseNote(data->getCurrentBaseNote(),inst.lowestNote());
 
@@ -138,9 +142,32 @@ void Graph::FretBoard::paintScale(QPainter* p)
 
-                int octave=Note::octaveOfBaseNote(data->getCurrentBaseNote(),note);
-                if(octave>5) octave=5;
+                //no more than six octaves can be visualised (there is a zero octave)
+                int octave=Note::octaveOfBaseNote(data->getCurrentBaseNote(),note)-baseoctave;
+                if(octave<0)
+                    qDebug("%d,%d",octave,baseoctave);
+                if(octave>5)
+                {
+                    qDebug("octave out of range");
+                    octave=5;
+                }
+
                 p->setPen(QColor(255,0,0));
-                int c=colorstart+octave*colorintervall;
-                QColor dotcolor(c,c,255);
-                p->setBrush(dotcolor);
+                int c= ( (note-12*baseoctave) - (12*octave+data->getCurrentBaseNote()) )*15;
+                if(c<0 || c>255)
+                    qDebug("%d = %d - ( %d + %d)",c,note,12*octave,data->getCurrentBaseNote());
+                QColor dotcolor(255,255,255);
+
+                if(octave==0)
+                    dotcolor=QColor(c,c,255);
+                else if(octave==1)
+                    dotcolor=QColor(c,255,c);
+                else if(octave==2)
+                    dotcolor=QColor(255,c,c);
+                else if(octave==3)
+                    dotcolor=QColor(255,255,c);
+                else if(octave==4)
+                    dotcolor=QColor(255,c,255);
+                else
+                    dotcolor=QColor(c,255,255);
 
+                p->setBrush(dotcolor);
                 p->drawEllipse(x-dotsize/2,y-dotsize/2,dotsize,dotsize);
diff --git a/noncore/multimedia/tonleiter/fretboard.h b/noncore/multimedia/tonleiter/fretboard.h
index 458c733..bb696fb 100644
--- a/noncore/multimedia/tonleiter/fretboard.h
+++ b/noncore/multimedia/tonleiter/fretboard.h
@@ -43,2 +43,4 @@ namespace Graph
         virtual ~FretBoard();
+    signals:
+        void pressed();
     private slots:
@@ -48,2 +50,3 @@ namespace Graph
         void resizeEvent(QResizeEvent* re);
+        void mouseReleaseEvent(QMouseEvent* me);
         void paintBoard(QPainter* p);
diff --git a/noncore/multimedia/tonleiter/mainwidget.cpp b/noncore/multimedia/tonleiter/mainwidget.cpp
index f09207b..b9c3d01 100644
--- a/noncore/multimedia/tonleiter/mainwidget.cpp
+++ b/noncore/multimedia/tonleiter/mainwidget.cpp
@@ -19,4 +19,7 @@ MainWidget::MainWidget(QWidget* parent,const char* name,WFlags f)
     fretboard=new Graph::FretBoard(data,mainwidget);
+    connect(fretboard,SIGNAL(pressed()),this,SLOT(fretboardPressed()));
     mainlayout->addWidget(fretboard);
 
+    showmenu=true;
+
 }
@@ -27,2 +30,16 @@ MainWidget::~MainWidget()
 //****************************************************************************
+void MainWidget::fretboardPressed()
+{
+    if(showmenu)
+    {
+        menuwidget->hide();
+        showmenu=false;
+    }
+    else
+    {
+        menuwidget->show();
+        showmenu=true;
+    }
+}
+//****************************************************************************
 //****************************************************************************
diff --git a/noncore/multimedia/tonleiter/mainwidget.h b/noncore/multimedia/tonleiter/mainwidget.h
index 9bb593c..5c0a1f8 100644
--- a/noncore/multimedia/tonleiter/mainwidget.h
+++ b/noncore/multimedia/tonleiter/mainwidget.h
@@ -16,2 +16,3 @@ private:
     Graph::FretBoard* fretboard;
+    bool showmenu;
 public:
@@ -19,2 +20,4 @@ public:
     virtual ~MainWidget();
+private slots:
+    void fretboardPressed();
 };
diff --git a/noncore/multimedia/tonleiter/tonleiterdatahelper.cpp b/noncore/multimedia/tonleiter/tonleiterdatahelper.cpp
index 156dba5..d904a71 100644
--- a/noncore/multimedia/tonleiter/tonleiterdatahelper.cpp
+++ b/noncore/multimedia/tonleiter/tonleiterdatahelper.cpp
@@ -78,5 +78,14 @@ int Instrument::noOfOctaves()
 {
-    int lowest=strings[0];
-    int highest=strings[strings.count()-1]+frets;
-    return (int) ceil((highest-lowest)/12.0);
+    return (int) ceil((highestNote()-lowestNote())/12.0);
+}
+//****************************************************************************
+int Instrument::lowestNote()
+{
+    return strings[0];
+
+}
+//****************************************************************************
+int Instrument::highestNote()
+{
+    return strings[strings.count()-1]+frets;
 }
diff --git a/noncore/multimedia/tonleiter/tonleiterdatahelper.h b/noncore/multimedia/tonleiter/tonleiterdatahelper.h
index e4153d7..061c781 100644
--- a/noncore/multimedia/tonleiter/tonleiterdatahelper.h
+++ b/noncore/multimedia/tonleiter/tonleiterdatahelper.h
@@ -34,2 +34,4 @@ namespace Data
         int noOfOctaves();
+        int lowestNote();
+        int highestNote();
     };