fix issue when video autoplay is enabled, it would not browse back to…

… video lists fix #338
lamarios · Nov 30, 2023 · f0b0f22 · f0b0f22
1 parent 3c4b99c
commit f0b0f22
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/lib/videos/states/video.dart b/lib/videos/states/video.dart
@@ -2,6 +2,7 @@ import 'package:copy_with_extension/copy_with_extension.dart';
 import 'package:flutter/cupertino.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:invidious/downloads/models/downloaded_video.dart';
+import 'package:invidious/router.dart';
 import 'package:invidious/videos/models/base_video.dart';
 import 'package:invidious/videos/models/dislike.dart';
 import 'package:logging/logging.dart';
@@ -43,9 +44,6 @@ class VideoCubit extends Cubit<VideoState> {
 
       getDownloadStatus();
 
-      if (settings.state.autoplayVideoOnLoad) {
-        playVideo(false);
-      }
     } catch (err) {
       var state = this.state.copyWith();
       if (err is InvidiousServiceError) {

diff --git a/lib/videos/views/screens/video.dart b/lib/videos/views/screens/video.dart
@@ -80,7 +80,12 @@ class VideoScreen extends StatelessWidget {
             create: (BuildContext context) =>
                 VideoCubit(VideoState(videoId: videoId), downloadManager, player, settings)),
       ],
-      child: BlocBuilder<VideoCubit, VideoState>(
+      child: BlocConsumer<VideoCubit, VideoState>(
+        listenWhen: (previous, current) => settings.state.autoplayVideoOnLoad && previous.video != current.video,
+        listener: (context, state) {
+            AutoRouter.of(context).pop();
+            context.read<VideoCubit>().playVideo(false);
+        },
         builder: (context, _) {
           var cubit = context.read<VideoCubit>();
           var settings = context.read<SettingsCubit>();