feat: mvToggledOpenHandler can now trigger on both open and close events.

This helps circumvent the change introduced in #1280 and handle the close.

Author: v-ein

dearpygui/_dearpygui.pyi

@@ -5066,6 +5066,8 @@ DearPyGui::GetEntityParser(mvAppItemType type)
             MV_PARSER_ARG_CALLBACK)
         );
 
+        args.push_back({ mvPyDataType::Bool, "two_way", mvArgType::KEYWORD_ARG, "False", "Trigger on both 'opened' and 'closed' events, i.e. when the 'opened' state is toggled between the two values. If False, some containers will trigger it only on the 'opened' event." });
+
         setup.about = "Adds a togged open handler.";
         setup.category = { "Widgets", "Events" };
         break;

dearpygui/_dearpygui_RTD.py

@@ -26,7 +26,7 @@ ResetAppItemState(mvAppItemState& state)
     state.activated = false;
     state.deactivated = false;
     state.deactivatedAfterEdit = false;
-    state.toggledOpen = false;
+    state.toggledOpenPure = state.toggledOpen = false;
     state.mvRectSizeResized = false;
     state.scrolledX = state.scrolledY = false;
     state.isScrollingX = state.isScrollingY = false;
@@ -57,7 +57,7 @@ UpdateAppItemState(mvAppItemState& state)
     state.activated = ImGui::IsItemActivated();
     state.deactivated = ImGui::IsItemDeactivated();
     state.deactivatedAfterEdit = ImGui::IsItemDeactivatedAfterEdit();
-    state.toggledOpen = ImGui::IsItemToggledOpen();
+    state.toggledOpenPure = state.toggledOpen = ImGui::IsItemToggledOpen();
     state.rectMin = { ImGui::GetItemRectMin().x, ImGui::GetItemRectMin().y };
     state.rectMax = { ImGui::GetItemRectMax().x, ImGui::GetItemRectMax().y };
     state.rectSize = { ImGui::GetItemRectSize().x, ImGui::GetItemRectSize().y };

dearpygui/dearpygui.py

@@ -85,6 +85,10 @@ struct mvAppItemState
     b8         deactivated          = false;
     b8         deactivatedAfterEdit = false;
     b8         toggledOpen          = false;
+    // This one is not reset by mvCollapsingHeader and mvTreeNode; also, not all items
+    // initialize it (but mvCollapsingHeader and mvTreeNode do, and they in fact are
+    // the only ones that must do it).  Used as an extra flag by the two-way toggled open handler.
+    b8         toggledOpenPure      = false;
     b8         mvRectSizeResized    = false;
     b8         scrolledX            = false;
     b8         scrolledY            = false;

src/mvAppItem.cpp

@@ -343,12 +343,28 @@ void mvResizeHandler::customAction(void* data)
 void mvToggledOpenHandler::customAction(void* data)
 {
 	mvAppItemState* state = static_cast<mvAppItemState*>(data);
-	if (state->toggledOpen)
+	if (state->toggledOpen || twoWay && state->toggledOpenPure)
 	{
 		submitHandler(state->parent);
 	}
 }
 
+void mvToggledOpenHandler::handleSpecificKeywordArgs(PyObject* dict)
+{
+	if (dict == nullptr)
+		return;
+
+	if (PyObject* item = PyDict_GetItemString(dict, "two_way")) twoWay = ToBool(item);
+}
+
+void mvToggledOpenHandler::getSpecificConfiguration(PyObject* dict)
+{
+	if (dict == nullptr)
+		return;
+
+	PyDict_SetItemString(dict, "two_way", mvPyObject(ToPyBool(twoWay)));
+}
+
 void mvVisibleHandler::customAction(void* data)
 {
 	mvAppItemState* state = static_cast<mvAppItemState*>(data);

src/mvAppItemState.cpp

@@ -149,6 +149,11 @@ class mvToggledOpenHandler : public mvItemHandler
     explicit mvToggledOpenHandler(mvUUID uuid) : mvItemHandler(uuid) {}
     void draw(ImDrawList* drawlist, float x, float y) override {}
     void customAction(void* data = nullptr) override;
+    void handleSpecificKeywordArgs(PyObject* dict) override;
+    void getSpecificConfiguration(PyObject* dict) override;
+
+private:
+    bool twoWay = false;
 };
 
 class mvVisibleHandler : public mvItemHandler