Added loading and success icons to login button

lib/features/auth/view/widgets/auth_view_body.dart

@@ -1,5 +1,5 @@
 import 'package:flutter/material.dart';
-import 'package:syncrow_app/features/shared_widgets/default_text_button.dart';
+import 'package:syncrow_app/features/shared_widgets/default_button.dart';
 import 'package:syncrow_app/features/shared_widgets/syncrow_logo.dart';
 
 import '../../../../navigation/routing_constants.dart';
@@ -19,14 +19,14 @@ class AuthViewBody extends StatelessWidget {
           const Expanded(child: SizedBox()),
           const SyncrowLogo(),
           const Expanded(flex: 2, child: SizedBox()),
-          DefaultTextButton(
+          DefaultButton(
             text: 'Login',
             onPressed: () {
               Navigator.pushNamed(context, Routes.authLogin);
             },
           ),
           const SizedBox(height: 15),
-          const DefaultTextButton(
+          const DefaultButton(
             text: 'Sign Up',
             isSecondary: true,
           ),

lib/features/auth/view/widgets/login/login_button.dart

@@ -0,0 +1,50 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_bloc/flutter_bloc.dart';
+import 'package:syncrow_app/features/auth/bloc/auth_cubit.dart';
+import 'package:syncrow_app/features/shared_widgets/default_button.dart';
+
+class LoginButton extends StatelessWidget {
+  const LoginButton({
+    super.key,
+  });
+
+  @override
+  Widget build(BuildContext context) {
+    return BlocBuilder<AuthCubit, AuthState>(
+      builder: (context, state) {
+        return Row(
+          mainAxisAlignment: MainAxisAlignment.center,
+          children: [
+            Expanded(
+              child: DefaultButton(
+                enabled: AuthCubit.get(context).agreeToTerms,
+                text: state is AuthLoading
+                    ? null
+                    : state is AuthSuccess
+                        ? null
+                        : "Login",
+                child: state is AuthSuccess
+                    ? const Icon(
+                        Icons.check_circle_outline,
+                        color: Colors.white,
+                      )
+                    : const SizedBox.square(
+                        dimension: 20,
+                        child: CircularProgressIndicator(
+                          color: Colors.white,
+                        ),
+                      ),
+                onPressed: () {
+                  if (AuthCubit.get(context).formKey.currentState!.validate()) {
+                    AuthCubit.get(context).login();
+                    FocusScope.of(context).unfocus();
+                  }
+                },
+              ),
+            ),
+          ],
+        );
+      },
+    );
+  }
+}

lib/features/auth/view/widgets/login/login_view.dart

@@ -1,9 +1,9 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:syncrow_app/features/auth/bloc/auth_cubit.dart';
+import 'package:syncrow_app/features/auth/view/widgets/login/login_button.dart';
 import 'package:syncrow_app/features/auth/view/widgets/login/login_form.dart';
 import 'package:syncrow_app/features/auth/view/widgets/login/login_user_agreement.dart';
-import 'package:syncrow_app/features/shared_widgets/default_text_button.dart';
 import 'package:syncrow_app/features/shared_widgets/text_widgets/title_medium.dart';
 import 'package:syncrow_app/navigation/routing_constants.dart';
 import 'package:syncrow_app/utils/resource_manager/constants.dart';
@@ -30,46 +30,25 @@ class LoginView extends StatelessWidget {
       builder: (context, state) {
         return Scaffold(
           appBar: AppBar(),
-          body: Padding(
+          body: const Padding(
-            padding: const EdgeInsets.symmetric(
+            padding: EdgeInsets.symmetric(
               horizontal: Constants.defaultPadding,
             ),
             child: Column(
               mainAxisAlignment: MainAxisAlignment.center,
               crossAxisAlignment: CrossAxisAlignment.start,
               children: [
-                const TitleMedium(
+                TitleMedium(
                   text: 'Login',
                 ),
-                const SizedBox(
+                SizedBox(
                   height: 10,
                 ),
-                const LoginForm(),
+                LoginForm(),
-                const SizedBox(height: 10),
+                SizedBox(height: 10),
-                const LoginUserAgreement(),
+                LoginUserAgreement(),
-                const SizedBox(height: 10),
+                SizedBox(height: 10),
-                Row(
+                LoginButton(),
-                  mainAxisAlignment: MainAxisAlignment.center,
-                  children: [
-                    state is AuthLoading
-                        ? const CircularProgressIndicator()
-                        : Expanded(
-                            child: DefaultTextButton(
-                              enabled: AuthCubit.get(context).agreeToTerms,
-                              text: "Login",
-                              onPressed: () {
-                                if (AuthCubit.get(context)
-                                    .formKey
-                                    .currentState!
-                                    .validate()) {
-                                  AuthCubit.get(context).login();
-                                  FocusScope.of(context).unfocus();
-                                }
-                              },
-                            ),
-                          ),
-                  ],
-                ),
               ],
             ),
           ),

lib/features/devices/view/widgets/no_devices_view.dart

@@ -1,5 +1,5 @@
 import 'package:flutter/material.dart';
-import 'package:syncrow_app/features/shared_widgets/default_text_button.dart';
+import 'package:syncrow_app/features/shared_widgets/default_button.dart';
 import 'package:syncrow_app/generated/assets.dart';
 
 class NoDevicesView extends StatelessWidget {
@@ -28,7 +28,7 @@ class NoDevicesView extends StatelessWidget {
             ),
           ),
           const SizedBox(height: 15),
-          const DefaultTextButton(
+          const DefaultButton(
             text: 'Add Device',
           ),
         ],

lib/features/scene/view/widgets/scene_view_no_scenes.dart

@@ -1,5 +1,5 @@
 import 'package:flutter/material.dart';
-import 'package:syncrow_app/features/shared_widgets/default_text_button.dart';
+import 'package:syncrow_app/features/shared_widgets/default_button.dart';
 import 'package:syncrow_app/generated/assets.dart';
 
 class SceneViewNoScenes extends StatelessWidget {
@@ -28,7 +28,7 @@ class SceneViewNoScenes extends StatelessWidget {
               style: TextStyle(color: Colors.grey),
             ),
             const SizedBox(height: 20),
-            const DefaultTextButton(
+            const DefaultButton(
               text: 'Create Scene',
             )
           ],

lib/features/shared_widgets/default_button.dart

@@ -1,24 +1,27 @@
 import 'package:flutter/material.dart';
 import 'package:syncrow_app/utils/resource_manager/color_manager.dart';
 
-class DefaultTextButton extends StatelessWidget {
+class DefaultButton extends StatelessWidget {
-  const DefaultTextButton({
+  const DefaultButton({
     super.key,
     this.enabled = true,
     this.onPressed,
-    required this.text,
+    this.child,
     this.isSecondary = false,
+    this.text,
   });
 
   final void Function()? onPressed;
-  final String text;
+  final Widget? child;
+
+  final String? text;
   final bool isSecondary;
 
   final bool enabled;
 
   @override
   Widget build(BuildContext context) {
-    return TextButton(
+    return ElevatedButton(
       onPressed: enabled ? onPressed : null,
       style: isSecondary
           ? null
@@ -31,15 +34,18 @@ class DefaultTextButton extends StatelessWidget {
                 ),
               ),
             ),
-      child: Text(
+      child: text != null
-        text,
+          ? Text(
-        style: TextStyle(
+              text!,
-            color: isSecondary
+              style: TextStyle(
-                ? Colors.black
+                color: isSecondary
-                : enabled
+                    ? Colors.black
-                    ? Colors.white
+                    : enabled
-                    : Colors.black),
+                        ? Colors.white
-      ),
+                        : Colors.black,
+              ),
+            )
+          : child,
     );
   }
 }

lib/features/splash/view/widgets/user_agreement_dialog.dart

@@ -1,5 +1,5 @@
 import 'package:flutter/material.dart';
-import 'package:syncrow_app/features/shared_widgets/default_text_button.dart';
+import 'package:syncrow_app/features/shared_widgets/default_button.dart';
 
 class UserAgreementDialog extends StatelessWidget {
   const UserAgreementDialog({
@@ -14,13 +14,13 @@ class UserAgreementDialog extends StatelessWidget {
       content:
           const Text('By using this app you agree to the terms and conditions'),
       actions: [
-        DefaultTextButton(
+        DefaultButton(
           text: 'I Agree',
           onPressed: () {
             Navigator.of(context).pop();
           },
         ),
-        DefaultTextButton(
+        DefaultButton(
           text: 'I Disagree',
           onPressed: () => Navigator.of(context).pop(),
           isSecondary: true,