chore: improve code style

.github/workflows/flutter.yml

@@ -21,8 +21,6 @@ jobs:
         echo keyAlias=\${{secrets.PLAY_KEY_ALIAS}} >> ${{env.PROPERTIES_PATH}}
         echo storeFile=\${{env.STORE_PATH}} >> ${{env.PROPERTIES_PATH}}
         echo "${{ secrets.PLAY_UPLOAD_KEYSTORE }}" | base64 --decode > ${{env.STORE_PATH}}
-        echo "storePath: $STORE_PATH"
-        ls -la ${{env.STORE_PATH}}
     - uses: actions/setup-java@v2
       with:
         distribution: 'zulu'

lib/hoster/service.dart

@@ -158,7 +158,7 @@ class MailHoster {
                 ),
                 Padding(
                   padding: const EdgeInsets.symmetric(horizontal: 8),
-                  child: PlatformText(buttonText),
+                  child: Text(buttonText),
                 ),
               ],
             ),
@@ -240,7 +240,7 @@ class GmailMailHoster extends MailHoster {
               ),
               Padding(
                 padding: const EdgeInsets.only(left: 8, right: 16),
-                child: PlatformText(
+                child: Text(
                   localizations.addAccountOauthSignInGoogle,
                   style: GoogleFonts.roboto(
                     color: googleText,

lib/lock/view.dart

@@ -59,7 +59,7 @@ class _LockScreenState extends State<LockScreen> {
                 child: Text(localizations.lockScreenIntro),
               ),
               PlatformTextButton(
-                child: PlatformText(localizations.lockScreenUnlockAction),
+                child: Text(localizations.lockScreenUnlockAction),
                 onPressed: () => _authenticate(context),
               ),
             ],

lib/models/async_mime_source.dart

@@ -977,20 +977,20 @@ class AsyncSearchMimeSource extends AsyncMimeSource {
   @override
   Future<MoveResult> moveMessages(
       List<MimeMessage> messages, Mailbox targetMailbox) {
-    // TODO: implement moveMessages
+    // TODO(RV): implement moveMessages
     throw UnimplementedError();
   }
 
   @override
   Future<MoveResult> moveMessagesToFlag(
       List<MimeMessage> messages, MailboxFlag targetMailboxFlag) {
-    // TODO: implement moveMessagesToFlag
+    // TODO(RV): implement moveMessagesToFlag
     throw UnimplementedError();
   }
 
   @override
   Future<MoveResult> undoMoveMessages(MoveResult moveResult) {
-    // TODO: implement undoMoveMessages
+    // TODO(RV): implement undoMoveMessages
     throw UnimplementedError();
   }
 

lib/models/hive/hive_mime_storage.dart

@@ -219,7 +219,7 @@ class HiveMailboxMimeStorage extends OfflineMimeStorage {
 
   @override
   Future<void> moveMessages(List<MimeMessage> messages, Mailbox targetMailbox) {
-    // TODO: implement moveMessages
+    // TODO(RV): implement moveMessages
     throw UnimplementedError();
   }
 }

lib/models/mail_operation.dart

@@ -130,7 +130,7 @@ class StoreFlagsOperation extends MailOperation {
 
   @override
   Future<void> execute(MailClient mailClient, OfflineMimeStorage storage) {
-    // TODO: implement execute
+    // TODO(RV): implement execute
     throw UnimplementedError();
   }
 }

lib/models/message_source.dart

@@ -1268,7 +1268,7 @@ class SingleMessageSource extends MessageSource {
 
   @override
   void onMailCacheInvalidated(AsyncMimeSource source) {
-    // TODO: implement onMailCacheInvalidated
+    // TODO(RV): implement onMailCacheInvalidated
   }
 }
 
@@ -1350,7 +1350,7 @@ class ListMessageSource extends MessageSource {
 
   @override
   void onMailCacheInvalidated(AsyncMimeSource source) {
-    // TODO: implement onMailCacheInvalidated
+    // TODO(RV): implement onMailCacheInvalidated
   }
 }
 

lib/models/offline_mime_source.dart

@@ -166,31 +166,31 @@ class OfflineMailboxMimeSource extends PagedCachedMimeSource {
     List<MimeMessage> messages,
     MailboxFlag targetMailboxFlag,
   ) {
-    // TODO: implement moveMessagesToFlag
+    // TODO(RV): implement moveMessagesToFlag
     throw UnimplementedError();
   }
 
   @override
   Future<MoveResult> undoMoveMessages(MoveResult moveResult) {
-    // TODO: implement undoMoveMessages
+    // TODO(RV): implement undoMoveMessages
     throw UnimplementedError();
   }
 
   @override
   Future<List<DeleteResult>> deleteAllMessages({bool expunge = false}) {
-    // TODO: implement deleteAllMessages
+    // TODO(RV): implement deleteAllMessages
     throw UnimplementedError();
   }
 
   @override
   Future<DeleteResult> deleteMessages(List<MimeMessage> messages) {
-    // TODO: implement deleteMessages
+    // TODO(RV): implement deleteMessages
     throw UnimplementedError();
   }
 
   @override
   Future<DeleteResult> undoDeleteMessages(DeleteResult deleteResult) {
-    // TODO: implement undoDeleteMessages
+    // TODO(RV): implement undoDeleteMessages
     throw UnimplementedError();
   }
 
@@ -199,7 +199,7 @@ class OfflineMailboxMimeSource extends PagedCachedMimeSource {
 
   @override
   AsyncMimeSource search(MailSearch search) {
-    // TODO: implement search
+    // TODO(RV): implement search
     throw UnimplementedError();
   }
 
@@ -228,7 +228,7 @@ class OfflineMailboxMimeSource extends PagedCachedMimeSource {
     List<String> flags, {
     StoreAction action = StoreAction.add,
   }) {
-    // TODO: implement storeAll
+    // TODO(RV): implement storeAll
     throw UnimplementedError();
   }
 

lib/screens/account_edit_screen.dart

@@ -93,7 +93,7 @@ class AccountEditScreen extends HookConsumerWidget {
                                   isRetryingToConnectState,
                                 ),
                                 icon: Icon(iconService.retry),
-                                label: PlatformText(
+                                label: Text(
                                   localizations
                                       .editAccountFailureToConnectRetryAction,
                                 ),
@@ -107,7 +107,7 @@ class AccountEditScreen extends HookConsumerWidget {
                                   account,
                                   isRetryingToConnectState,
                                 ),
-                                child: PlatformText(
+                                child: Text(
                                   localizations
                                       .editAccountFailureToConnectChangePasswordAction,
                                 ),

lib/screens/welcome_screen.dart

@@ -103,7 +103,7 @@ class WelcomeScreen extends StatelessWidget {
               child: PlatformFilledButtonIcon(
                 icon: Icon(IconService.instance.email),
                 label: Center(
-                  child: PlatformText(localizations.welcomeActionSignIn),
+                  child: Text(localizations.welcomeActionSignIn),
                 ),
                 onPressed: () {
                   context.goNamed(Routes.accountAdd);

lib/widgets/account_hoster_selector.dart

@@ -22,7 +22,7 @@ class MailHosterSelector extends StatelessWidget {
         if (index == 0) {
           return Center(
             child: PlatformTextButton(
-              child: PlatformText(localizations.accountProviderCustom),
+              child: Text(localizations.accountProviderCustom),
               onPressed: () => onSelected(null),
             ),
           );

lib/widgets/ical_composer.dart

@@ -328,7 +328,7 @@ class _DateTimePicker extends StatelessWidget {
       children: [
         // set date button:
         PlatformTextButton(
-          child: PlatformText(
+          child: Text(
             dt == null
                 ? localizations.composeAppointmentLabelDay
                 : context.formatDate(dt.toLocal(), useLongFormat: true),
@@ -355,7 +355,7 @@ class _DateTimePicker extends StatelessWidget {
         if (!onlyDate)
           // set time button:
           PlatformTextButton(
-            child: PlatformText(
+            child: Text(
               dt == null
                   ? localizations.composeAppointmentLabelTime
                   : context.formatTimeOfDay(

lib/widgets/ical_interactive_media.dart

@@ -107,7 +107,7 @@ class _IcalInteractiveMediaState extends State<IcalInteractiveMedia> {
               Row(
                 children: [
                   PlatformTextButton(
-                    child: PlatformText(localizations.actionAccept),
+                    child: Text(localizations.actionAccept),
                     onPressed: () => _changeParticipantStatus(
                       ParticipantStatus.accepted,
                       localizations,
@@ -122,7 +122,7 @@ class _IcalInteractiveMediaState extends State<IcalInteractiveMedia> {
                     ),
                   ),
                   PlatformTextButton(
-                    child: PlatformText(localizations.actionDecline),
+                    child: Text(localizations.actionDecline),
                     onPressed: () => _changeParticipantStatus(
                       ParticipantStatus.declined,
                       localizations,
@@ -140,7 +140,7 @@ class _IcalInteractiveMediaState extends State<IcalInteractiveMedia> {
                     ),
                   ),
                   PlatformTextButton(
-                    child: PlatformText(
+                    child: Text(
                       localizations.icalendarActionChangeParticipantStatus,
                     ),
                     onPressed: () => _queryParticipantStatus(localizations),
@@ -354,7 +354,7 @@ class _IcalInteractiveMediaState extends State<IcalInteractiveMedia> {
             ),
             if (!isReply)
               PlatformElevatedButton(
-                child: PlatformText(localizations.icalendarExportAction),
+                child: Text(localizations.icalendarExportAction),
                 onPressed: () => _exportToNativeCalendar(_calendar),
               ),
           ],
@@ -436,19 +436,19 @@ class _IcalInteractiveMediaState extends State<IcalInteractiveMedia> {
       localizations.icalendarParticipantStatusChangeText,
       actions: [
         PlatformTextButton(
-          child: PlatformText(localizations.actionAccept),
+          child: Text(localizations.actionAccept),
           onPressed: () => context.pop(ParticipantStatus.accepted),
         ),
         PlatformTextButton(
-          child: PlatformText(localizations.icalendarAcceptTentatively),
+          child: Text(localizations.icalendarAcceptTentatively),
           onPressed: () => context.pop(ParticipantStatus.tentative),
         ),
         PlatformTextButton(
-          child: PlatformText(localizations.actionDecline),
+          child: Text(localizations.actionDecline),
           onPressed: () => context.pop(ParticipantStatus.declined),
         ),
         PlatformTextButton(
-          child: PlatformText(localizations.actionCancel),
+          child: Text(localizations.actionCancel),
           onPressed: () => context.pop(),
         ),
       ],