feat: replace youtube player to iframe_youtube_place (#73)

codingpot · Jun 11, 2021 · f644afb · f644afb
1 parent c62a593
commit f644afb
Show file tree

Hide file tree

Showing 11 changed files with 82 additions and 239 deletions.
diff --git a/client/ios/Podfile.lock b/client/ios/Podfile.lock
@@ -8,10 +8,13 @@ PODS:
     - Flutter
     - OrderedSet (~> 5.0)
   - OrderedSet (5.0.0)
+  - url_launcher (0.0.1):
+    - Flutter
 
 DEPENDENCIES:
   - Flutter (from `Flutter`)
   - flutter_inappwebview (from `.symlinks/plugins/flutter_inappwebview/ios`)
+  - url_launcher (from `.symlinks/plugins/url_launcher/ios`)
 
 SPEC REPOS:
   trunk:
@@ -22,11 +25,14 @@ EXTERNAL SOURCES:
     :path: Flutter
   flutter_inappwebview:
     :path: ".symlinks/plugins/flutter_inappwebview/ios"
+  url_launcher:
+    :path: ".symlinks/plugins/url_launcher/ios"
 
 SPEC CHECKSUMS:
   Flutter: 434fef37c0980e73bb6479ef766c45957d4b510c
   flutter_inappwebview: bfd58618f49dc62f2676de690fc6dcda1d6c3721
   OrderedSet: aaeb196f7fef5a9edf55d89760da9176ad40b93c
+  url_launcher: 6fef411d543ceb26efce54b05a0a40bfd74cbbef
 
 PODFILE CHECKSUM: aafe91acc616949ddb318b77800a7f51bffa2a4c
 

diff --git a/client/lib/test-server.dart b/client/lib/test-server.dart
diff --git a/client/lib/utils/formatter.dart b/client/lib/utils/formatter.dart
diff --git a/client/lib/widgets/detail/header.dart b/client/lib/widgets/detail/header.dart
@@ -1,7 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:intl/intl.dart';
 import 'package:pr12er/protos/pkg/pr12er/messages.pb.dart';
-import 'package:pr12er/utils/formatter.dart';
 
 // ignore: must_be_immutable
 class HeaderWidget extends StatelessWidget {
@@ -43,12 +42,12 @@ class HeaderWidget extends StatelessWidget {
       const SizedBox(width: 8),
       const Icon(Icons.remove_red_eye),
       const SizedBox(width: 8),
-      Text(reformatNumbers(video.numberOfViews.toInt()),
+      Text(NumberFormat.compact().format(video.numberOfViews),
           style: const TextStyle(fontSize: 18)),
       const SizedBox(width: 15),
       getLikeIcon(),
       const SizedBox(width: 8),
-      Text(reformatNumbers(video.numberOfLike.toInt()),
+      Text(NumberFormat.compact().format(video.numberOfLike),
           style: const TextStyle(fontSize: 18)),
     ];
   }

diff --git a/client/lib/widgets/detail/youtube.dart b/client/lib/widgets/detail/youtube.dart
@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
-import 'package:youtube_player_flutter/youtube_player_flutter.dart';
+// import 'package:youtube_player_flutter/youtube_player_flutter.dart';
+import 'package:youtube_player_iframe/youtube_player_iframe.dart';
 
 class YoutubeWidget extends StatefulWidget {
   final String youtubeId;
@@ -22,15 +23,13 @@ class _DetailState extends State<YoutubeWidget> {
 
   @override
   Widget build(BuildContext context) {
-    return YoutubePlayerBuilder(
-      player: YoutubePlayer(
-        controller: _controller,
+    return YoutubePlayerControllerProvider(
+      // Provides controller to all the widget below it.
+      controller: _controller,
+      child: const YoutubePlayerIFrame(
+        // ignore: avoid_redundant_argument_values
+        aspectRatio: 16 / 9,
       ),
-      builder: (context, player) {
-        return Column(
-          children: [player],
-        );
-      },
     );
   }
 }
diff --git a/client/pubspec.lock b/client/pubspec.lock
@@ -233,6 +233,11 @@ packages:
     description: flutter
     source: sdk
     version: "0.0.0"
+  flutter_web_plugins:
+    dependency: transitive
+    description: flutter
+    source: sdk
+    version: "0.0.0"
   frontend_server_client:
     dependency: transitive
     description:
@@ -413,6 +418,13 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "3.0.0"
+  plugin_platform_interface:
+    dependency: transitive
+    description:
+      name: plugin_platform_interface
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "2.0.0"
   pool:
     dependency: transitive
     description:
@@ -593,6 +605,48 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.3.0"
+  url_launcher:
+    dependency: transitive
+    description:
+      name: url_launcher
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "6.0.6"
+  url_launcher_linux:
+    dependency: transitive
+    description:
+      name: url_launcher_linux
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "2.0.0"
+  url_launcher_macos:
+    dependency: transitive
+    description:
+      name: url_launcher_macos
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "2.0.0"
+  url_launcher_platform_interface:
+    dependency: transitive
+    description:
+      name: url_launcher_platform_interface
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "2.0.3"
+  url_launcher_web:
+    dependency: transitive
+    description:
+      name: url_launcher_web
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "2.0.1"
+  url_launcher_windows:
+    dependency: transitive
+    description:
+      name: url_launcher_windows
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "2.0.0"
   vector_math:
     dependency: transitive
     description:
@@ -649,6 +703,13 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "8.0.0"
+  youtube_player_iframe:
+    dependency: "direct main"
+    description:
+      name: youtube_player_iframe
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "2.0.0"
 sdks:
   dart: ">=2.12.0 <3.0.0"
-  flutter: ">=1.22.2"
+  flutter: ">=2.0.0"
diff --git a/client/pubspec.yaml b/client/pubspec.yaml
@@ -30,6 +30,7 @@ dependencies:
   protobuf: ^2.0.0
   provider: ^5.0.0
   youtube_player_flutter: ^8.0.0
+  youtube_player_iframe: ^2.0.0
 
 dev_dependencies:
   build_runner: ^2.0.4