Merge pull request #17 from cb-cloud/feature/replace-statefulwidget

Refactoring to get rid of `StatefulWidget`.
cb-cloud · Mar 14, 2022 · 5a9a893 · 5a9a893
2 parents 2521fa8 + 444d0e9
commit 5a9a893
Show file tree

Hide file tree

Showing 7 changed files with 196 additions and 125 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,13 @@
+## 1.1.0
+### FEAT
+- Added `InAppNotification.dismiss()` method that hides notification programmatically.
+  - from #14 .
+
+### CHORE
+- Refactored again.
+  - Get rid of `StatefulWidget`, using `findAncestorStateOfType()` method.
+    - This is expensive when using `BuildContext` that obtained from deep hierarchy of Widget tree, so it replaced with `getElementForInheritedWidgetOfExactType()` method on `InheritedWidget`.
+
 ## 1.0.2
 ### FIX
 - Fixed a bug that `curve` option in `InAppNotification.show()` didn't affect.

diff --git a/example/lib/main.dart b/example/lib/main.dart
@@ -87,7 +87,7 @@ class _HomePageState extends State<HomePage> {
             ),
             const SizedBox(height: 32),
             ElevatedButton(
-              onPressed: () => InAppNotification.dismiss(),
+              onPressed: () => InAppNotification.dismiss(context: context),
               child: Text('Dismiss Notification'),
             )
           ],

diff --git a/example/pubspec.lock b/example/pubspec.lock
@@ -66,7 +66,7 @@ packages:
       path: ".."
       relative: true
     source: path
-    version: "1.0.2"
+    version: "1.1.0"
   matcher:
     dependency: transitive
     description: