fix: 🐛Fixed cursor not changing to click mode issue (#503) (#504)

SimformSolutionsPvtLtd · Feb 26, 2025 · 817e005 · 817e005
1 parent ea99366
commit 817e005
Show file tree

Hide file tree

Showing 4 changed files with 175 additions and 152 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,7 +1,10 @@
 ## [4.1.0] - (UnRelease)
 - Feature [#500](https://github.com/SimformSolutionsPvtLtd/flutter_showcaseview/issues/500) -
   Added `onDismiss` callback in `ShowCaseWidget` which will trigger whenever `onDismiss` method is
-  called
+  called.
+- Fixed [#503](https://github.com/SimformSolutionsPvtLtd/flutter_showcaseview/issues/503) - Cursor
+  not changing to click mode when it is hovering over the clickable widgets provided by this
+  package.
 
 ## [4.0.1]
 - Fixed [#493](https://github.com/SimformSolutionsPvtLtd/flutter_showcaseview/issues/493) - ShowCase.withWidget not showing issue 

diff --git a/lib/src/showcase.dart b/lib/src/showcase.dart
@@ -807,7 +807,10 @@ class _ShowcaseState extends State<Showcase> {
             showArrow: widget.showArrow,
             contentHeight: widget.height,
             contentWidth: widget.width,
-            onTooltipTap: _getOnTooltipTap,
+            onTooltipTap:
+                widget.disposeOnTap == true || widget.onToolTipClick != null
+                    ? _getOnTooltipTap
+                    : null,
             tooltipPadding: widget.tooltipPadding,
             disableMovingAnimation: widget.disableMovingAnimation ??
                 showCaseWidgetState.disableMovingAnimation,
@@ -908,7 +911,10 @@ class _TargetWidget extends StatelessWidget {
           ? IgnorePointer(
               child: targetWidgetContent(),
             )
-          : targetWidgetContent(),
+          : MouseRegion(
+              cursor: SystemMouseCursors.click,
+              child: targetWidgetContent(),
+            ),
     );
   }
 

diff --git a/lib/src/tooltip_action_button_widget.dart b/lib/src/tooltip_action_button_widget.dart
@@ -21,35 +21,38 @@ class TooltipActionButtonWidget extends StatelessWidget {
     final theme = Theme.of(context);
 
     return config.button ??
-        GestureDetector(
-          onTap: handleOnTap,
-          child: Container(
-            padding: config.padding,
-            decoration: BoxDecoration(
-              color: config.backgroundColor ?? theme.primaryColor,
-              borderRadius: config.borderRadius,
-              border: config.border,
-            ),
-            child: Row(
-              mainAxisSize: MainAxisSize.min,
-              children: [
-                if (config.leadIcon != null)
-                  Padding(
-                    padding: config.leadIcon?.padding ??
-                        const EdgeInsets.only(right: 5),
-                    child: config.leadIcon?.icon,
-                  ),
-                Text(
-                  config.name ?? config.type?.actionName ?? '',
-                  style: config.textStyle,
-                ),
-                if (config.tailIcon != null)
-                  Padding(
-                    padding: config.tailIcon?.padding ??
-                        const EdgeInsets.only(left: 5),
-                    child: config.tailIcon?.icon,
+        MouseRegion(
+          cursor: SystemMouseCursors.click,
+          child: GestureDetector(
+            onTap: handleOnTap,
+            child: Container(
+              padding: config.padding,
+              decoration: BoxDecoration(
+                color: config.backgroundColor ?? theme.primaryColor,
+                borderRadius: config.borderRadius,
+                border: config.border,
+              ),
+              child: Row(
+                mainAxisSize: MainAxisSize.min,
+                children: [
+                  if (config.leadIcon != null)
+                    Padding(
+                      padding: config.leadIcon?.padding ??
+                          const EdgeInsets.only(right: 5),
+                      child: config.leadIcon?.icon,
+                    ),
+                  Text(
+                    config.name ?? config.type?.actionName ?? '',
+                    style: config.textStyle,
                   ),
-              ],
+                  if (config.tailIcon != null)
+                    Padding(
+                      padding: config.tailIcon?.padding ??
+                          const EdgeInsets.only(left: 5),
+                      child: config.tailIcon?.icon,
+                    ),
+                ],
+              ),
             ),
           ),
         );