Browse Source

PlotHistogram, PlotLines: use ButtonBehavior() to be idiomatic. (#7935, #3072)

ocornut 1 year ago
parent
commit
776813416b
2 changed files with 4 additions and 3 deletions
  1. 2 2
      docs/CHANGELOG.txt
  2. 2 1
      imgui_widgets.cpp

+ 2 - 2
docs/CHANGELOG.txt

@@ -92,8 +92,8 @@ Other changes:
 - TextLink(), TextLinkOpenURL(): change mouse cursor to Hand shape when hovered. (#7885, #7660)
 - Tooltips, Drag and Drop: made it possible to override BeginTooltip() position while inside
   a drag and drop source or target: a SetNextWindowPos() call won't be overriden. (#6973)
-- PlotHistogram, PlotLines: register item ID in a more idiomatic manner, fixes preventing
-  e.g. GetItemID() and other ID-based helper to work. (#7935, #3072)
+- PlotHistogram, PlotLines: register item ID and use button behavior in a more idiomatic manner,
+  fixes preventing e.g. GetItemID() and other ID-based helper to work. (#7935, #3072)
 - Style: added PushStyleVarX(), PushStyleVarY() helpers to modify only one component of a ImVec2 var.
 - Fonts: made it possible to use PushFont()/PopFont() calls accross Begin() calls. (#3224, #3875, #6398, #7903)
 - Backends:

+ 2 - 1
imgui_widgets.cpp

@@ -8226,7 +8226,8 @@ int ImGui::PlotEx(ImGuiPlotType plot_type, const char* label, float (*values_get
     ItemSize(total_bb, style.FramePadding.y);
     if (!ItemAdd(total_bb, id, &frame_bb, ImGuiItemFlags_NoNav))
         return -1;
-    const bool hovered = ItemHoverable(frame_bb, id, g.LastItemData.InFlags);
+    bool hovered;
+    ButtonBehavior(frame_bb, id, &hovered, NULL);
 
     // Determine scale from values if not specified
     if (scale_min == FLT_MAX || scale_max == FLT_MAX)