feat: use material 3 by default and fix deprecated errors

zeshuaro · Nov 25, 2023 · cfd1298 · cfd1298
1 parent 4b71f97
commit cfd1298
Show file tree

Hide file tree

Showing 7 changed files with 27 additions and 10 deletions.
diff --git a/lib/advanced_theme/cubit/advanced_theme_state.dart b/lib/advanced_theme/cubit/advanced_theme_state.dart
@@ -5,7 +5,10 @@ class AdvancedThemeState extends Equatable {
   final bool isDark;
   final bool useMaterial3;
 
-  const AdvancedThemeState({this.isDark = false, this.useMaterial3 = false});
+  const AdvancedThemeState({
+    this.isDark = false,
+    this.useMaterial3 = true,
+  });
 
   @override
   List<Object> get props => [isDark, useMaterial3];

diff --git a/lib/app.dart b/lib/app.dart
@@ -213,6 +213,8 @@ class _MaterialApp extends StatefulWidget {
 }
 
 class _MaterialAppState extends State<_MaterialApp> {
+  final _seedColor = Colors.blue;
+
   @override
   void initState() {
     super.initState();
@@ -225,8 +227,17 @@ class _MaterialAppState extends State<_MaterialApp> {
       builder: (context, state) {
         return MaterialApp(
           title: 'Appainter',
-          theme: ThemeData(),
-          darkTheme: ThemeData.dark(),
+          theme: ThemeData.from(
+            colorScheme: ColorScheme.fromSeed(
+              seedColor: _seedColor,
+            ),
+          ),
+          darkTheme: ThemeData.from(
+            colorScheme: ColorScheme.fromSeed(
+              seedColor: _seedColor,
+              brightness: Brightness.dark,
+            ),
+          ),
           themeMode: state.themeMode,
           home: state.status == HomeStatus.success
               ? const HomePage()

diff --git a/lib/basic_theme/cubit/basic_theme_state.dart b/lib/basic_theme/cubit/basic_theme_state.dart
@@ -16,7 +16,7 @@ class BasicThemeState extends Equatable {
     Color? seedColor,
     ColorScheme? colorScheme,
     this.isDark = false,
-    this.useMaterial3 = false,
+    this.useMaterial3 = true,
   })  : seedColor = seedColor ?? defaultSeedColor,
         colorScheme = colorScheme ?? _colorSchemeLight;
 
@@ -96,9 +96,12 @@ class BasicThemeState extends Equatable {
 
   ThemeData get theme {
     return ThemeData.localize(
-      ThemeData.from(colorScheme: colorScheme),
+      ThemeData.from(
+        colorScheme: colorScheme,
+        useMaterial3: useMaterial3,
+      ),
       Typography.englishLike2018,
-    ).copyWith(useMaterial3: useMaterial3);
+    );
   }
 
   Brightness get brightness => isDark ? Brightness.dark : Brightness.light;

diff --git a/lib/home/views/home_page.dart b/lib/home/views/home_page.dart
@@ -16,7 +16,7 @@ class HomePage extends StatefulWidget {
 }
 
 class HomePageState extends State<HomePage> {
-  static const _sdkVersion = '3.13.0+';
+  static const _sdkVersion = '3.16.0+';
   static final _backgroundColorDark = Colors.grey[900]!;
   static final _backgroundColorLight = Colors.grey[200]!;
 

diff --git a/lib/home/widgets/material3_switch.dart b/lib/home/widgets/material3_switch.dart
@@ -12,7 +12,7 @@ class Material3Switch extends StatelessWidget {
     return Row(
       children: [
         Text(
-          'Material 3 (beta)',
+          'Material 3',
           style: Theme.of(context).textTheme.titleMedium,
         ),
         const _Switch(),

diff --git a/lib/theme_preview/views/theme_preview.dart b/lib/theme_preview/views/theme_preview.dart
@@ -192,6 +192,7 @@ class ThemePreview extends StatelessWidget {
                     ),
                   ),
                   drawer: _Drawer(),
+                  // ignore: prefer_const_constructors
                   body: TabBarView(
                     children: _pages,
                   ),

diff --git a/test/color_theme/color_theme_editor_test.dart b/test/color_theme/color_theme_editor_test.dart
@@ -285,11 +285,10 @@ void main() {
     });
 
     testWidgets('change color', (tester) async {
-      final opaqueColor = color.withOpacity(0.12);
       await pumpApp(tester);
       await tester.verifyColorPicker(
         key,
-        opaqueColor,
+        color,
         colorThemeCubit.dividerColorChanged,
       );
     });