use accurate method for filling colour options box

vsicurella · Jul 18, 2024 · b7328aa · b7328aa
1 parent b3f1c74
commit b7328aa
Showing 1 changed file with 23 additions and 19 deletions.
diff --git a/Source/mapping_editors/MultiSelectControls.cpp b/Source/mapping_editors/MultiSelectControls.cpp
@@ -210,32 +210,36 @@ void MultiSelectControls::completeMappingLoaded(const LumatoneLayout &mappingDat
 
 void MultiSelectControls::boardChanged(const LumatoneBoard &boardData)
 {
-    auto boardColours = boardData.getBoardColours();
-    auto colourOptions = colourDropdown->getColourOptions();
-    for (auto colour : boardColours)
-    {
-        if (colourOptions.contains(colour))
-            continue;
+    // auto boardColours = boardData.getBoardColours();
+    // auto colourOptions = colourDropdown->getColourOptions();
+    // for (auto colour : boardColours)
+    // {
+    //     if (colourOptions.contains(colour))
+    //         continue;
+
+    //     colourDropdown->addItem(colour.toDisplayString(false), colourDropdown->getNumItems() + 1);
+    // }
 
-        colourDropdown->addItem(colour.toDisplayString(false), colourDropdown->getNumItems() + 1);
-    }
+    completeMappingLoaded(*getMappingData());
 }
 
 void MultiSelectControls::keyChanged(int boardIndex, int keyIndex, const LumatoneKey &lumatoneKey)
 {
-    auto colourOptions = colourDropdown->getColourOptions();
-    if (!colourOptions.contains(lumatoneKey.getColour()))
-        colourDropdown->addItem(lumatoneKey.getColour().toDisplayString(false), colourDropdown->getNumItems() + 1);
+    // auto colourOptions = colourDropdown->getColourOptions();
+    // if (!colourOptions.contains(lumatoneKey.getColour()))
+    //     colourDropdown->addItem(lumatoneKey.getColour().toDisplayString(false), colourDropdown->getNumItems() + 1);
+    completeMappingLoaded(*getMappingData());
 }
 
 void MultiSelectControls::keySetChanged(juce::Array<MappedLumatoneKey> selection)
 {
-    auto colourOptions = colourDropdown->getColourOptions();
-    for (auto key : selection)
-    {
-        if (colourOptions.contains(key.getColour()))
-            continue;
-
-        colourDropdown->addItem(key.getColour().toDisplayString(false), colourDropdown->getNumItems() + 1);
-    }
+    // auto colourOptions = colourDropdown->getColourOptions();
+    // for (auto key : selection)
+    // {
+    //     if (colourOptions.contains(key.getColour()))
+    //         continue;
+
+    //     colourDropdown->addItem(key.getColour().toDisplayString(false), colourDropdown->getNumItems() + 1);
+    // }
+    completeMappingLoaded(*getMappingData());
 }