From e09255f96ef7aef58bea873a3c922e6d19b8b6c6 Mon Sep 17 00:00:00 2001
From: Lucas Oliveira <62367544+tilucasoli@users.noreply.github.com>
Date: Thu, 11 Jul 2024 14:22:43 -0300
Subject: [PATCH] refactor: use timer instead of future<void> (#374)

* change future to Timer

* Update pressable_widget.dart
---
 .../widget_state/pressable_widget.dart        | 29 +++++++++++++++----
 .../pressable/pressable_widget_test.dart      | 25 ++++++++++++++++
 2 files changed, 48 insertions(+), 6 deletions(-)

diff --git a/packages/mix/lib/src/widgets/widget_state/pressable_widget.dart b/packages/mix/lib/src/widgets/widget_state/pressable_widget.dart
index e113b801b..ba37929f7 100644
--- a/packages/mix/lib/src/widgets/widget_state/pressable_widget.dart
+++ b/packages/mix/lib/src/widgets/widget_state/pressable_widget.dart
@@ -1,3 +1,5 @@
+import 'dart:async';
+
 import 'package:flutter/gestures.dart';
 import 'package:flutter/material.dart';
 
@@ -277,15 +279,30 @@ abstract class _MixStateWidgetBuilderState<T extends _MixStateWidgetBuilder>
     setState(fn);
   }
 
-  Future<void> unpressAfterDelay(int initialPressCount) async {
-    final delay = widget.unpressDelay;
+  Timer? _timer;
 
-    if (delay != Duration.zero) {
-      await Future.delayed(delay);
+  @override
+  void dispose() {
+    _timer?.cancel();
+    super.dispose();
+  }
+
+  void unpressAfterDelay(int initialPressCount) {
+    void unpressCallback() {
+      if (_isPressed && _pressCount == initialPressCount) {
+        updateState(() => _isPressed = false);
+      }
     }
 
-    if (_isPressed && _pressCount == initialPressCount) {
-      updateState(() => _isPressed = false);
+    _timer?.cancel();
+    _timer = null;
+
+    final delay = widget.unpressDelay;
+
+    if (delay != Duration.zero) {
+      _timer = Timer(delay, unpressCallback);
+    } else {
+      unpressCallback();
     }
   }
 
diff --git a/packages/mix/test/src/widgets/pressable/pressable_widget_test.dart b/packages/mix/test/src/widgets/pressable/pressable_widget_test.dart
index 5015eb66b..5ea8f43e6 100644
--- a/packages/mix/test/src/widgets/pressable/pressable_widget_test.dart
+++ b/packages/mix/test/src/widgets/pressable/pressable_widget_test.dart
@@ -131,6 +131,31 @@ void main() {
     });
   });
 
+  testWidgets('Pressable cancel timer on dispose', (WidgetTester tester) async {
+    var wasPressed = false;
+
+    await tester.pumpWidget(
+      MaterialApp(
+        home: Scaffold(
+          body: Pressable(
+            autofocus: true,
+            unpressDelay: const Duration(minutes: 1),
+            onPress: () {
+              wasPressed = !wasPressed;
+            },
+            child: const Text('Tap me'),
+          ),
+        ),
+      ),
+    );
+
+    await tester.sendKeyEvent(LogicalKeyboardKey.enter);
+    expect(wasPressed, isTrue);
+
+    await tester.sendKeyEvent(LogicalKeyboardKey.enter);
+    expect(wasPressed, isFalse);
+  });
+
   group('PressableBox', () {
     testWidgets(
       'must be clickable when enable is setted to true',