Fix triggered both on_click and on_long_press events (#2914)

Author: isaffathir

packages/flet/lib/src/controls/container.dart

@@ -153,23 +153,16 @@ class ContainerControl extends StatelessWidget with FletStoreMixin {
               // Dummy callback to enable widget
               // see https://github.com/flutter/flutter/issues/50116#issuecomment-582047374
               // and https://github.com/flutter/flutter/blob/eed80afe2c641fb14b82a22279d2d78c19661787/packages/flutter/lib/src/material/ink_well.dart#L1125-L1129
-              onTap: onHover ? () {} : null,
-              onTapDown: onClick || url != ""
-                  ? (details) {
+              onTap: onClick || url != ""
+                  ? () {
                       debugPrint("Container ${control.id} clicked!");
                       if (url != "") {
                         openWebBrowser(url, webWindowName: urlTarget);
                       }
                       if (onClick) {
                         backend.triggerControlEvent(
                             control.id,
-                            "click",
-                            json.encode(ContainerTapEvent(
-                                    localX: details.localPosition.dx,
-                                    localY: details.localPosition.dy,
-                                    globalX: details.globalPosition.dx,
-                                    globalY: details.globalPosition.dy)
-                                .toJson()));
+                            "click");
                       }
                     }
                   : null,
@@ -201,7 +194,7 @@ class ContainerControl extends StatelessWidget with FletStoreMixin {
                 width: control.attrDouble("width"),
                 height: control.attrDouble("height"),
                 margin: parseEdgeInsets(control, "margin"),
-                clipBehavior: Clip.none,
+                clipBehavior: clipBehavior,
                 decoration: boxDecor,
                 child: ink,
               )
@@ -269,22 +262,16 @@ class ContainerControl extends StatelessWidget with FletStoreMixin {
                   }
                 : null,
             child: GestureDetector(
-              onTapDown: onClick || url != ""
-                  ? (details) {
+              onTap: onClick || url != ""
+                  ? () {
                       debugPrint("Container ${control.id} clicked!");
                       if (url != "") {
                         openWebBrowser(url, webWindowName: urlTarget);
                       }
                       if (onClick) {
                         backend.triggerControlEvent(
                             control.id,
-                            "click",
-                            json.encode(ContainerTapEvent(
-                                    localX: details.localPosition.dx,
-                                    localY: details.localPosition.dy,
-                                    globalX: details.globalPosition.dx,
-                                    globalY: details.globalPosition.dy)
-                                .toJson()));
+                            "click");
                       }
                     }
                   : null,

sdk/python/packages/flet-core/src/flet_core/container.py

@@ -474,11 +474,11 @@ def theme_mode(self, value: Optional[ThemeMode]):
     # on_click
     @property
     def on_click(self):
-        return self.__on_click
+        return self._get_event_handler("click")
 
     @on_click.setter
     def on_click(self, handler):
-        self.__on_click.subscribe(handler)
+        self._add_event_handler("click", handler)
         self._set_attr("onClick", True if handler is not None else None)
 
     # on_long_press