fix CurvedAnimation & overlayEntry memory leak

Author: zeqinjie

lib/src/custom/custom_dialog.dart

@@ -384,6 +384,7 @@ class CustomDialog extends BaseDialog {
       closeType: closeType,
     );
     customDialog.overlayEntry.remove();
+    customDialog.overlayEntry.dispose();
   }
 
   static DialogInfo? _getDialog({

lib/src/custom/custom_notify.dart

@@ -211,6 +211,7 @@ class CustomNotify extends BaseDialog {
       closeType: closeType,
     );
     customDialog.overlayEntry.remove();
+    customDialog.overlayEntry.dispose();
   }
 
   static NotifyInfo? _getDialog({String? tag}) {

lib/src/custom/toast/custom_toast.dart

@@ -145,6 +145,7 @@ class CustomToast extends BaseDialog {
       for (var item in toastQueue) {
         await ViewUtils.awaitPostFrame(onPostFrame: () {
           item.mainDialog.overlayEntry.remove();
+          item.mainDialog.overlayEntry.dispose();
         });
       }
       toastQueue.clear();
@@ -224,6 +225,7 @@ class CustomToast extends BaseDialog {
     Timer(time, () async {
       await mainDialog.dismiss();
       mainDialog.overlayEntry.remove();
+      mainDialog.overlayEntry.dispose();
     });
   }
 

lib/src/custom/toast/toast_tool.dart

@@ -38,6 +38,7 @@ class ToastTool {
     await curToast.mainDialog.dismiss();
     if (curToast.mainDialog.overlayEntry.mounted) {
       curToast.mainDialog.overlayEntry.remove();
+      curToast.mainDialog.overlayEntry.dispose();
     }
     await Future.delayed(SmartDialog.config.toast.intervalTime);
   }
@@ -51,6 +52,7 @@ class ToastTool {
     for (var element in toastQueue) {
       if (element.mainDialog.overlayEntry.mounted) {
         element.mainDialog.overlayEntry.remove();
+        element.mainDialog.overlayEntry.dispose();
       }
     }
     toastQueue.clear();

lib/src/widget/animation/fade_animation.dart

@@ -1,6 +1,6 @@
 import 'package:flutter/material.dart';
 
-class FadeAnimation extends StatelessWidget {
+class FadeAnimation extends StatefulWidget {
   const FadeAnimation({
     Key? key,
     required this.controller,
@@ -11,11 +11,33 @@ class FadeAnimation extends StatelessWidget {
 
   final Widget child;
 
+  @override
+  State<FadeAnimation> createState() => _FadeAnimationState();
+}
+
+class _FadeAnimationState extends State<FadeAnimation> {
+  late CurvedAnimation _curvedAnimation;
+
+  @override
+  void initState() {
+    super.initState();
+    _curvedAnimation = CurvedAnimation(
+      parent: widget.controller,
+      curve: Curves.linear,
+    );
+  }
+
+  @override
+  void dispose() {
+    _curvedAnimation.dispose();
+    super.dispose();
+  }
+
   @override
   Widget build(BuildContext context) {
     return FadeTransition(
-      opacity: CurvedAnimation(parent: controller, curve: Curves.linear),
-      child: child,
+      opacity: _curvedAnimation,
+      child: widget.child,
     );
   }
 }

lib/src/widget/animation/highlight_mask_animation.dart

@@ -3,7 +3,7 @@ import 'package:flutter_smart_dialog/flutter_smart_dialog.dart';
 
 import '../attach_dialog_widget.dart';
 
-class HighlightMaskAnimation extends StatelessWidget {
+class HighlightMaskAnimation extends StatefulWidget {
   const HighlightMaskAnimation({
     Key? key,
     required this.controller,
@@ -31,21 +31,43 @@ class HighlightMaskAnimation extends StatelessWidget {
 
   final List<SmartNonAnimationType> nonAnimationTypes;
 
+  @override
+  State<HighlightMaskAnimation> createState() => _HighlightMaskAnimationState();
+}
+
+class _HighlightMaskAnimationState extends State<HighlightMaskAnimation> {
+  late CurvedAnimation _curvedAnimation;
+
+  @override
+  void initState() {
+    super.initState();
+    _curvedAnimation = CurvedAnimation(
+      parent: widget.controller,
+      curve: Curves.linear,
+    );
+  }
+
+  @override
+  void dispose() {
+    _curvedAnimation.dispose();
+    super.dispose();
+  }
+
   @override
   Widget build(BuildContext context) {
     //handle mask
     late Widget mask;
-    if (usePenetrate) {
+    if (widget.usePenetrate) {
       mask = Container();
-    } else if (maskWidget != null) {
-      mask = maskWidget!;
-    } else if (highlightBuilder == null) {
-      mask = Container(color: maskColor);
+    } else if (widget.maskWidget != null) {
+      mask = widget.maskWidget!;
+    } else if (widget.highlightBuilder == null) {
+      mask = Container(color: widget.maskColor);
     } else {
       mask = ColorFiltered(
         colorFilter: ColorFilter.mode(
           // mask color
-          maskColor,
+          widget.maskColor,
           BlendMode.srcOut,
         ),
         child: Stack(children: [
@@ -58,17 +80,17 @@ class HighlightMaskAnimation extends StatelessWidget {
           ),
 
           //dissolve mask, highlight location
-          highlightBuilder!.call(targetOffset, targetSize)
+          widget.highlightBuilder!.call(widget.targetOffset, widget.targetSize)
         ]),
       );
     }
 
     Widget maskAnimation = FadeTransition(
-      opacity: CurvedAnimation(parent: controller, curve: Curves.linear),
+      opacity: _curvedAnimation,
       child: mask,
     );
-    if (highlightBuilder != null) {
-      for (var element in nonAnimationTypes) {
+    if (widget.highlightBuilder != null) {
+      for (var element in widget.nonAnimationTypes) {
         if (element == SmartNonAnimationType.highlightMask_nonAnimation) {
           maskAnimation = mask;
         }

lib/src/widget/animation/mask_animation.dart

@@ -1,6 +1,6 @@
 import 'package:flutter/material.dart';
 
-class MaskAnimation extends StatelessWidget {
+class MaskAnimation extends StatefulWidget {
   const MaskAnimation({
     Key? key,
     required this.controller,
@@ -17,13 +17,35 @@ class MaskAnimation extends StatelessWidget {
 
   final bool usePenetrate;
 
+  @override
+  State<MaskAnimation> createState() => _MaskAnimationState();
+}
+
+class _MaskAnimationState extends State<MaskAnimation> {
+  late CurvedAnimation _curvedAnimation;
+
+  @override
+  void initState() {
+    super.initState();
+    _curvedAnimation = CurvedAnimation(
+      parent: widget.controller,
+      curve: Curves.linear,
+    );
+  }
+
+  @override
+  void dispose() {
+    _curvedAnimation.dispose();
+    super.dispose();
+  }
+
   @override
   Widget build(BuildContext context) {
     return FadeTransition(
-      opacity: CurvedAnimation(parent: controller, curve: Curves.linear),
-      child: (maskWidget != null && !usePenetrate)
-          ? maskWidget
-          : Container(color: usePenetrate ? null : maskColor),
+      opacity: _curvedAnimation,
+      child: (widget.maskWidget != null && !widget.usePenetrate)
+          ? widget.maskWidget
+          : Container(color: widget.usePenetrate ? null : widget.maskColor),
     );
   }
 }

lib/src/widget/animation/scale_animation.dart

@@ -1,6 +1,6 @@
 import 'package:flutter/material.dart';
 
-class ScaleAnimation extends StatelessWidget {
+class ScaleAnimation extends StatefulWidget {
   const ScaleAnimation({
     Key? key,
     required this.controller,
@@ -14,12 +14,34 @@ class ScaleAnimation extends StatelessWidget {
 
   final Alignment? alignment;
 
+  @override
+  State<ScaleAnimation> createState() => _ScaleAnimationState();
+}
+
+class _ScaleAnimationState extends State<ScaleAnimation> {
+  late CurvedAnimation _curvedAnimation;
+
+  @override
+  void initState() {
+    super.initState();
+    _curvedAnimation = CurvedAnimation(
+      parent: widget.controller,
+      curve: Curves.linear,
+    );
+  }
+
+  @override
+  void dispose() {
+    _curvedAnimation.dispose();
+    super.dispose();
+  }
+
   @override
   Widget build(BuildContext context) {
     return ScaleTransition(
-      alignment: alignment ?? const Alignment(0, 0),
-      scale: CurvedAnimation(parent: controller, curve: Curves.linear),
-      child: child,
+      alignment: widget.alignment ?? const Alignment(0, 0),
+      scale: _curvedAnimation,
+      child: widget.child,
     );
   }
 }