Merge pull request #74 from icapps/bugfix/#73-fix-touch-feedback-ios

#73: Fixed the touch feedback on iOS

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## [0.6.4] - 2022-02-22
+### Fixed
+
+- iOS Touchfeedback when 2 TouchFeedbackIOS widgets are on top of eachother
+
 ## [0.6.3] - 2022-01-26
 ### Added
 

lib/src/widget/touch_feedback/touch_feedback.dart

@@ -90,13 +90,23 @@ class TouchFeedBackIOS extends StatefulWidget {
 class _TouchFeedBackIOSState extends State<TouchFeedBackIOS> {
   static const touchScale = 0.98;
   static const defaultScale = 1.0;
+
+  static final _touchPoints = <UniqueKey, Offset>{};
+  final _key = UniqueKey();
+
   var touched = false;
 
+  @override
+  void dispose() {
+    _touchPoints.remove(_key);
+    super.dispose();
+  }
+
   @override
   Widget build(BuildContext context) {
     return GestureDetector(
       excludeFromSemantics: widget.onClick == null,
-      onTapDown: (details) => _setTouched(true),
+      onTapDown: _onTapDown,
       onTap: widget.onClick,
       onTapCancel: () => _setTouched(false),
       onTapUp: (details) => _setTouched(false),
@@ -115,7 +125,15 @@ class _TouchFeedBackIOSState extends State<TouchFeedBackIOS> {
   }
 
   void _setTouched(bool touched) {
-    if (widget.onClick == null) return;
+    if (!touched) _touchPoints.remove(_key);
+    if (widget.onClick == null || this.touched == touched) return;
     setState(() => this.touched = touched);
   }
+
+  void _onTapDown(TapDownDetails details) {
+    final touchPosition = details.globalPosition;
+    if (_touchPoints.containsValue(touchPosition)) return;
+    _touchPoints[_key] = touchPosition;
+    _setTouched(true);
+  }
 }

pubspec.yaml

@@ -4,7 +4,7 @@ homepage: https://github.com/icapps/flutter-icapps-architecture
 repository: https://github.com/icapps/flutter-icapps-architecture
 issue_tracker: https://github.com/icapps/flutter-icapps-architecture/issues
 
-version: 0.6.3
+version: 0.6.4
 
 environment:
   sdk: '>=2.14.0 <3.0.0'

test/util/logging/logging_test.dart

@@ -181,7 +181,7 @@ void testWithLogger() {
       expect(
           messages[4].lines[4],
           matches(
-              '\\d+:\\d+:\\d+\\.\\d+\\s+#2   StackZoneSpecification._registerUnaryCallback.<anonymous closure> \\(package:stack_trace/src/stack_zone_specification.dart\\)'));
+              '\\d+:\\d+:\\d+\\.\\d+\\s+#2   StackZoneSpecification._registerUnaryCallback.<anonymous closure> \\(package:stack_trace/src/stack_zone_specification.dart:125:47\\)'));
       expect(messages[4].lines[5],
           matches('\\d+:\\d+:\\d+\\.\\d+\\s+#3   <asynchronous suspension>'));
       expect(messages[4].lines[0],