Merge commit '68064248428713d75125cd758701eaa34b660766' into custom-lg

* commit '68064248428713d75125cd758701eaa34b660766':
  chewie, version 1.10.0. Addresses Issue fluttercommunity#618.
  refactor: fixed formatting and made onTap not null
  refactor: removed unused parameter
  refactor: refactored
  fix: added _buildOptions
  fix: fixed closing when options are opened

Author: lg8294

CHANGELOG.md

@@ -1,3 +1,10 @@
+## [1.10.0]
+* [#871](https://github.com/fluttercommunity/chewie/pull/871): Fixed pop the wrong page when changing the speed. Thanks [akmalova](https://github.com/akmalova).
+* **BREAKING CHANGES**: 
+  * `OptionItem.onTap` now takes in a `BuildContext`.
+  * `OptionItem`'s properties are now marked as `final`. Use `copyWith` to mutate its properties into
+    a new instance.
+
 ## [1.9.2]
 * Fixed broken Table of Contents links in `README.md`, take two.
 

example/lib/app/app.dart

@@ -127,7 +127,7 @@ class _ChewieDemoState extends State<ChewieDemo> {
       additionalOptions: (context) {
         return <OptionItem>[
           OptionItem(
-            onTap: toggleVideo,
+            onTap: (context) => toggleVideo(),
             iconData: Icons.live_tv_sharp,
             title: 'Toggle Video Src',
           ),

lib/src/cupertino/widgets/cupertino_options_dialog.dart

@@ -24,7 +24,7 @@ class _CupertinoOptionsDialogState extends State<CupertinoOptionsDialog> {
         actions: widget.options
             .map(
               (option) => CupertinoActionSheetAction(
-                onPressed: () => option.onTap!(),
+                onPressed: () => option.onTap(context),
                 child: Text(option.title),
               ),
             )

lib/src/material/material_controls.dart

@@ -162,10 +162,10 @@ class _MaterialControlsState extends State<MaterialControls>
     );
   }
 
-  Widget _buildOptionsButton() {
+  List<OptionItem> _buildOptions(BuildContext context) {
     final options = <OptionItem>[
       OptionItem(
-        onTap: () async {
+        onTap: (context) async {
           Navigator.pop(context);
           _onSpeedButtonTap();
         },
@@ -179,7 +179,10 @@ class _MaterialControlsState extends State<MaterialControls>
         chewieController.additionalOptions!(context).isNotEmpty) {
       options.addAll(chewieController.additionalOptions!(context));
     }
+    return options;
+  }
 
+  Widget _buildOptionsButton() {
     return AnimatedOpacity(
       opacity: notifier.hideStuff ? 0.0 : 1.0,
       duration: const Duration(milliseconds: 250),
@@ -188,14 +191,15 @@ class _MaterialControlsState extends State<MaterialControls>
           _hideTimer?.cancel();
 
           if (chewieController.optionsBuilder != null) {
-            await chewieController.optionsBuilder!(context, options);
+            await chewieController.optionsBuilder!(
+                context, _buildOptions(context));
           } else {
             await showModalBottomSheet<OptionItem>(
               context: context,
               isScrollControlled: true,
               useRootNavigator: chewieController.useRootNavigator,
               builder: (context) => OptionsDialog(
-                options: options,
+                options: _buildOptions(context),
                 cancelButtonText:
                     chewieController.optionsTranslation?.cancelButtonText,
               ),

lib/src/material/material_desktop_controls.dart

@@ -183,7 +183,7 @@ class _MaterialDesktopControlsState extends State<MaterialDesktopControls>
   }) {
     final options = <OptionItem>[
       OptionItem(
-        onTap: () async {
+        onTap: (context) async {
           Navigator.pop(context);
           _onSpeedButtonTap();
         },

lib/src/material/widgets/options_dialog.dart

@@ -28,7 +28,7 @@ class _OptionsDialogState extends State<OptionsDialog> {
             itemCount: widget.options.length,
             itemBuilder: (context, i) {
               return ListTile(
-                onTap: widget.options[i].onTap,
+                onTap: () => widget.options[i].onTap(context),
                 leading: Icon(widget.options[i].iconData),
                 title: Text(widget.options[i].title),
                 subtitle: widget.options[i].subtitle != null

lib/src/models/option_item.dart

@@ -8,13 +8,13 @@ class OptionItem {
     this.subtitle,
   });
 
-  Function()? onTap;
-  IconData iconData;
-  String title;
-  String? subtitle;
+  final void Function(BuildContext context) onTap;
+  final IconData iconData;
+  final String title;
+  final String? subtitle;
 
   OptionItem copyWith({
-    Function()? onTap,
+    Function(BuildContext context)? onTap,
     IconData? iconData,
     String? title,
     String? subtitle,

pubspec.yaml

@@ -1,6 +1,6 @@
 name: chewie
 description: A video player for Flutter with Cupertino and Material play controls
-version: 1.9.2
+version: 1.10.0
 homepage: https://github.com/fluttercommunity/chewie
 
 environment: