[#26]: Disable Play/Save buttons, when there's no active Level

JacobDomagala · Aug 1, 2021 · 9c9bd7d · 9c9bd7d
1 parent b6da491
commit 9c9bd7d
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 6 deletions.
diff --git a/editor/GUI/EditorGUI.cpp b/editor/GUI/EditorGUI.cpp
@@ -7,6 +7,7 @@
 #include <backends/imgui_impl_opengl3.h>
 #include <fmt/format.h>
 #include <imgui.h>
+#include <imgui_internal.h>
 
 namespace dgame {
 
@@ -158,7 +159,9 @@ EditorGUI::Render()
    ImGui::SetNextWindowPos({0, 0});
    ImGui::SetNextWindowSize(ImVec2(m_windowWidth, toolsWindowHeight));
    ImGui::Begin("Tools");
-   ImGui::PushStyleColor(ImGuiCol_Button, {0.45f, 0.0f, 0.2f, 0.8f});
+   ImGui::PushStyleColor(ImGuiCol_Button, ImVec4{0.45f, 0.0f, 0.2f, 0.8f});
+   ImGui::PushDisabled(m_currentLevel == nullptr);
+
    if (ImGui::Button("Play"))
    {
       m_parent.PlayLevel();
@@ -174,6 +177,8 @@ EditorGUI::Render()
          m_parent.SaveLevel(levelName);
       }
    }
+   ImGui::PopDisabled();
+
    ImGui::SameLine();
    if (ImGui::Button("Load"))
    {

diff --git a/engine/FileManager/FileManager.cpp b/engine/FileManager/FileManager.cpp
@@ -241,11 +241,15 @@ FileManager::FileDialog(const std::vector< std::pair< std::string, std::string >
    assert(output);
 
    std::string buffer(FILE_DIALOG_MAX_BUFFER, 0);
-   fgets(buffer.data(), FILE_DIALOG_MAX_BUFFER, output);
-
-   // fgets includes \n character at the end, remove it
-   // buffer.back() = '\0';
-   buffer[strcspn(buffer.c_str(), "\n")] = 0;
+   if (fgets(buffer.data(), FILE_DIALOG_MAX_BUFFER, output))
+   {
+      // fgets includes \n character at the end, remove it
+      buffer[strcspn(buffer.c_str(), "\n")] = 0;
+   }
+   else
+   {
+      buffer = "";
+   }
 
    pclose(output);