From 51148a729cb5d62332c3ac56717a7cb3b5b0210e Mon Sep 17 00:00:00 2001
From: YohananDiamond <mitonanan12@gmail.com>
Date: Mon, 31 Jul 2023 15:58:38 -0300
Subject: [PATCH] command palette: instrument and sample menus

won't be doing for wavetables because they don't have text and that
would be impossible to search through with a keyboard
---
 src/gui/commandPalette.cpp | 46 ++++++++++++++++++++++++++++++++++----
 src/gui/commandPalette.h   |  3 +++
 src/gui/doAction.cpp       |  6 +++++
 src/gui/gui.h              |  2 ++
 src/gui/guiConst.cpp       |  2 ++
 5 files changed, 55 insertions(+), 4 deletions(-)

diff --git a/src/gui/commandPalette.cpp b/src/gui/commandPalette.cpp
index d3744741b..af38fa1ac 100644
--- a/src/gui/commandPalette.cpp
+++ b/src/gui/commandPalette.cpp
@@ -76,6 +76,26 @@ void FurnaceGUI::drawPalette() {
       }
       break;
 
+    case CMDPAL_TYPE_INSTRUMENTS:
+      if (matchFuzzy("- None -",paletteQuery.c_str())) {
+        paletteSearchResults.push_back(0);
+      }
+      for (int i=0; i<e->song.insLen; i++) {
+        if (matchFuzzy(e->song.ins[i]->name.c_str(),paletteQuery.c_str())) {
+          paletteSearchResults.push_back(i+1); // because over here ins=0 is 'None'
+        }
+      }
+      break;
+
+    case CMDPAL_TYPE_SAMPLES:
+      for (int i=0; i<e->song.sampleLen; i++) {
+        logD("ins #%x: %s", i, e->song.sample[i]->name.c_str());
+        if (matchFuzzy(e->song.sample[i]->name.c_str(),paletteQuery.c_str())) {
+          paletteSearchResults.push_back(i);
+        }
+      }
+      break;
+
     default:
       logE("invalid command palette type");
       ImGui::CloseCurrentPopup();
@@ -109,13 +129,21 @@ void FurnaceGUI::drawPalette() {
       case CMDPAL_TYPE_RECENT:
         s=recentFile.at(id).c_str();
         break;
+      case CMDPAL_TYPE_INSTRUMENTS:
+        if (id==0) {
+          s="- None -";
+        } else {
+          s=e->song.ins[id-1]->name.c_str();
+        }
+        break;
+      case CMDPAL_TYPE_SAMPLES:
+        s=e->song.sample[id]->name.c_str();
+        break;
       default:
         logE("invalid command palette type");
         break;
       };
 
-      logD("~ %d, %d", curPaletteType, s == nullptr);
-
       if (ImGui::Selectable(s,current)) {
         curPaletteChoice=i;
         accepted=true;
@@ -147,24 +175,34 @@ void FurnaceGUI::drawPalette() {
       int i=paletteSearchResults[curPaletteChoice];
       switch (curPaletteType) {
       case CMDPAL_TYPE_MAIN:
-        resetPalette(this);
         doAction(i);
         if (i<GUI_ACTION_CMDPAL_MIN || GUI_ACTION_CMDPAL_MAX<i) {
+          // TODO: maybe instead of this, just re-open the dialog from the doAction code. it'll be much simpler than this corny mess I made
           ImGui::CloseCurrentPopup();
         }
         break;
 
       case CMDPAL_TYPE_RECENT:
-        resetPalette(this);
         openRecentFile(recentFile.at(i));
         ImGui::CloseCurrentPopup();
         break;
 
+      case CMDPAL_TYPE_INSTRUMENTS:
+        curIns=i-1;
+        ImGui::CloseCurrentPopup();
+        break;
+
+      case CMDPAL_TYPE_SAMPLES:
+        curSample=i;
+        ImGui::CloseCurrentPopup();
+        break;
+
       default:
         logE("invalid command palette type");
         ImGui::CloseCurrentPopup();
         break;
       };
+      resetPalette(this);
     }
   }
 }
diff --git a/src/gui/commandPalette.h b/src/gui/commandPalette.h
index 758c7979f..308b2b8fc 100644
--- a/src/gui/commandPalette.h
+++ b/src/gui/commandPalette.h
@@ -20,6 +20,9 @@
 enum CommandPaletteType {
   CMDPAL_TYPE_MAIN = 0,
   CMDPAL_TYPE_RECENT,
+  CMDPAL_TYPE_INSTRUMENTS,
+  CMDPAL_TYPE_SAMPLES,
+  // a menu to select wavetables is beyond scope (they can't be put as strings)
   // TODO: are there more?
 
   CMDPAL_TYPE_MAX,
diff --git a/src/gui/doAction.cpp b/src/gui/doAction.cpp
index 6cd1a6761..52af29215 100644
--- a/src/gui/doAction.cpp
+++ b/src/gui/doAction.cpp
@@ -192,6 +192,12 @@ void FurnaceGUI::doAction(int what) {
     case GUI_ACTION_CMDPAL_RECENT:
       curPaletteType=CMDPAL_TYPE_RECENT;
       break;
+    case GUI_ACTION_CMDPAL_INSTRUMENTS:
+      curPaletteType=CMDPAL_TYPE_INSTRUMENTS;
+      break;
+    case GUI_ACTION_CMDPAL_SAMPLES:
+      curPaletteType=CMDPAL_TYPE_SAMPLES;
+      break;
     case GUI_ACTION_WINDOW_EDIT_CONTROLS:
       nextWindow=GUI_WINDOW_EDIT_CONTROLS;
       break;
diff --git a/src/gui/gui.h b/src/gui/gui.h
index d321609e2..dbb0d6c06 100644
--- a/src/gui/gui.h
+++ b/src/gui/gui.h
@@ -499,6 +499,8 @@ enum FurnaceGUIActions {
   GUI_ACTION_COMMAND_PALETTE,
   GUI_ACTION_CMDPAL_MIN,
   GUI_ACTION_CMDPAL_RECENT,
+  GUI_ACTION_CMDPAL_INSTRUMENTS,
+  GUI_ACTION_CMDPAL_SAMPLES,
   GUI_ACTION_CMDPAL_MAX,
 
   GUI_ACTION_WINDOW_EDIT_CONTROLS,
diff --git a/src/gui/guiConst.cpp b/src/gui/guiConst.cpp
index dabd79e2a..7854a65af 100644
--- a/src/gui/guiConst.cpp
+++ b/src/gui/guiConst.cpp
@@ -560,6 +560,8 @@ const FurnaceGUIActionDef guiActions[GUI_ACTION_MAX]={
   D("COMMAND_PALETTE", "Command Palette", FURKMOD_CMD|SDLK_p),
   D("CMDPAL_MIN", "", NOT_AN_ACTION),
   D("CMDPAL_RECENT", "Recent files (Palette)", 0),
+  D("CMDPAL_INSTRUMENTS", "Instruments (Palette)", 0),
+  D("CMDPAL_SAMPLES", "Samples (Palette)", 0),
   D("CMDPAL_MAX", "", NOT_AN_ACTION),
 
   D("WINDOW_EDIT_CONTROLS", "Edit Controls", 0),