Impl choosing colors by color picker menu "C"

src/imgui_helper.hpp

@@ -23,10 +23,11 @@ namespace Tri {
                 state->get_height() - 20.0f));
             ImGui::SetNextWindowBgAlpha(0.7f);
             ImGui::Begin("Help Window", nullptr, ImGuiWindowFlags_NoDecoration);
-            ImGui::Text("This is the help window. Press \"H\" to toggle this window.");
+            ImGui::Text("This is the help window - Press \"H\" to toggle this window");
             ImGui::Text("Click anywhere to create triangles, one point at a time");
             ImGui::Text("Press \"U\" to undo. Clicking will remove all future undo history");
             ImGui::Text("Press \"R\" to undo.");
+            ImGui::Text("Press \"C\" to change colors");
             ImGui::End();
         }
     }
@@ -50,6 +51,14 @@ namespace Tri {
             ImGui::PopStyleVar();
         }
     }
+
+    inline void draw_color_picker(Tri::State *state) {
+        if(state->get_flags().test(2)) {
+            ImGui::Begin("Color Picker");
+            ImGui::ColorPicker4("Tri Color", state->get_color());
+            ImGui::End();
+        }
+    }
 }
 
 #endif

src/state.cpp

@@ -21,7 +21,8 @@ starting_help_alpha(1.0f),
 window(sf::VideoMode(800, 600), "Triangles", sf::Style::Titlebar | sf::Style::Close),
 trisIndex(0),
 currentTri_state(CurrentState::NONE),
-currentTri_maxState(CurrentState::NONE)
+currentTri_maxState(CurrentState::NONE),
+colorPickerColor{1.0f, 1.0f, 1.0f, 1.0f}
 {
     flags.set(1); // is running
     ImGui::SFML::Init(window);
@@ -93,8 +94,11 @@ void Tri::State::handle_events() {
                         break;
                     }
                 }
+            } else if(event.key.code == sf::Keyboard::C) {
+                flags.flip(2);
             }
         } else if(event.type == sf::Event::MouseButtonPressed) {
+            if(!flags.test(2)) {
                 switch(currentTri_state) {
                 case CurrentState::NONE:
                     currentTri[0] = sf::Vector2f(event.mouseButton.x, event.mouseButton.y);
@@ -122,7 +126,7 @@ void Tri::State::handle_events() {
                     tris.back().setPoint(0, currentTri[0]);
                     tris.back().setPoint(1, currentTri[1]);
                     tris.back().setPoint(2, currentTri[2]);
-                tris.back().setFillColor(sf::Color::White); // TODO use chosen color
+                    tris.back().setFillColor(pointCircle.getFillColor());
                     currentTri_state = CurrentState::NONE;
                     currentTri_maxState = CurrentState::NONE;
                     break;
@@ -130,6 +134,7 @@ void Tri::State::handle_events() {
             }
         }
     }
+}
 
 void Tri::State::update() {
     ImGui::SFML::Update(window, dt);
@@ -141,8 +146,18 @@ void Tri::State::update() {
         }
     }
 
+    if(flags.test(3)) {
+        flags.reset(3);
+        pointCircle.setFillColor(sf::Color(
+            (unsigned char)(255 * colorPickerColor[0]),
+            (unsigned char)(255 * colorPickerColor[1]),
+            (unsigned char)(255 * colorPickerColor[2]),
+            (unsigned char)(255 * colorPickerColor[3])));
+    }
+
     // Seems misleading, but imgui handles setting up the window during update
     Tri::draw_show_help(this);
+    Tri::draw_color_picker(this);
     Tri::draw_help(this);
 
     ImGui::EndFrame();
@@ -183,3 +198,8 @@ const Tri::State::BitsetType Tri::State::get_flags() const {
 float Tri::State::get_starting_help_alpha() const {
     return starting_help_alpha;
 }
+
+float* Tri::State::get_color() {
+    flags.set(3);
+    return colorPickerColor;
+}

src/state.hpp

@@ -18,6 +18,8 @@ namespace Tri {
         /*
          * 0 - display help
          * 1 - is running
+         * 2 - display color picker
+         * 3 - color picker color dirty
          */
         typedef std::bitset<64> BitsetType;
         BitsetType flags;
@@ -36,6 +38,8 @@ namespace Tri {
 
         sf::Event event;
 
+        float colorPickerColor[4];
+
     public:
         void handle_events();
         void update();
@@ -47,6 +51,8 @@ namespace Tri {
         const BitsetType get_flags() const;
 
         float get_starting_help_alpha() const;
+
+        float* get_color();
     };
 }