Broken ContextMenu about ebitenui

[mcarpenter622]

Hey, So I figured out what broke it. It was a change I made a while back. I just pushed a change that seems to clean things up quite a well!. Please check out the latest commit and let me know if you seen anything else. I am going to close this comment for now. Please open a new one or reach out on discord (see readme for link). Thanks for all the feedback. Having people like you in the community help make this a better library!
~Mark

[mat007]

If we change the example code to:

diff --git a/_examples/widget_demos/contextmenu/main.go b/_examples/widget_demos/contextmenu/main.go
index 6637df8..c8183c0 100644
--- a/_examples/widget_demos/contextmenu/main.go
+++ b/_examples/widget_demos/contextmenu/main.go
@@ -50,9 +50,15 @@ func main() {
                        widget.ButtonOpts.ClickedHandler(func(args *widget.ButtonClickedEventArgs) {
                                println(fmt.Sprintf("Option %d clicked", iter))
                                //Manually close the ContextMenuWindow
-                               //if button.GetWidget().ContextMenuWindow != nil {
-                               //      button.GetWidget().ContextMenuWindow.Close()
-                               //}
+                               if button.GetWidget().ContextMenuWindow != nil {
+                                       button.GetWidget().ContextMenuWindow.Close()
+                               }
+                       }),
+                       widget.ButtonOpts.PressedHandler(func(args *widget.ButtonPressedEventArgs) {
+                               println(fmt.Sprintf("Option %d pressed", iter))
+                       }),
+                       widget.ButtonOpts.ReleasedHandler(func(args *widget.ButtonReleasedEventArgs) {
+                               println(fmt.Sprintf("Option %d released", iter))
                        }),
                )
                contextMenu.AddChild(Option1)
@@ -84,7 +90,7 @@ func main() {
                        // Set how the context menu should be closed
                        // This defaults to "CLICK" so that it will close the contextmenu
                        // whenever the user clicks their mouse next
-                       //widget.WidgetOpts.ContextMenuCloseMode(widget.NONE),
+                       widget.WidgetOpts.ContextMenuCloseMode(widget.CLICK_OUT),
                ),

                // specify the images to use

i.e. use widget.CLICK_OUT instead of the default widget.CLICK, and manually close the menu, then it does work as expected.
Perhaps the default should be changed and the example updated? Maybe it’s not as clean an API as ultimately possible though…