TW-1644: allow access to the gallery is not center

linagora · May 29, 2024 · 6d5b67c · 6d5b67c
1 parent 15affc5
commit 6d5b67c
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 13 deletions.
diff --git a/assets/l10n/intl_en.arb b/assets/l10n/intl_en.arb
@@ -2690,7 +2690,7 @@
   "@noImagesFound": {},
   "captionForImagesIsNotSupportYet": "Caption for images is not support yet.",
   "@captionForImagesIsNotSupportYet": {},
-  "tapToAllowAccessToYourGallery": "Tap to allow access to your Gallery",
+  "tapToAllowAccessToYourGallery": "Tap to allow gallery access",
   "@tapToAllowAccessToYourGallery": {},
   "tapToAllowAccessToYourCamera": "You can enable camera access in the Settings app to make video calls in",
   "@tapToAllowAccessToYourCamera": {},

diff --git a/lib/presentation/mixins/media_picker_mixin.dart b/lib/presentation/mixins/media_picker_mixin.dart
@@ -289,18 +289,20 @@ mixin MediaPickerMixin on CommonMediaPickerMixin {
         ),
       ),
       goToSettingsWidget: Column(
+        mainAxisAlignment: MainAxisAlignment.center,
         children: [
           SvgPicture.asset(
             ImagePaths.icPhotosSettingPermission,
-            width: 40,
-            height: 40,
+            width: MediaPickerStyle.photoPermissionIconSize,
+            height: MediaPickerStyle.photoPermissionIconSize,
           ),
           Text(
             L10n.of(context)!.tapToAllowAccessToYourGallery,
-            style: Theme.of(context)
-                .textTheme
-                .titleSmall
-                ?.copyWith(color: LinagoraRefColors.material().neutral),
+            style: Theme.of(context).textTheme.titleSmall?.copyWith(
+                  color: LinagoraRefColors.material().neutral,
+                  fontWeight: MediaPickerStyle.photoPermissionFontWeight,
+                  fontSize: MediaPickerStyle.photoPermissionFontSize,
+                ),
             textAlign: TextAlign.center,
           ),
         ],

diff --git a/lib/presentation/mixins/single_image_picker_mixin.dart b/lib/presentation/mixins/single_image_picker_mixin.dart
@@ -1,3 +1,4 @@
+import 'package:fluffychat/presentation/style/media_picker_style.dart';
 import 'package:fluffychat/resource/image_paths.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_gen/gen_l10n/l10n.dart';
@@ -37,18 +38,20 @@ mixin SingleImagePickerMixin on CommonMediaPickerMixin {
         expandedWidget: const SizedBox(height: 50),
         counterImageBuilder: (_) => const SizedBox.shrink(),
         goToSettingsWidget: Column(
+          mainAxisAlignment: MainAxisAlignment.center,
           children: [
             SvgPicture.asset(
               ImagePaths.icPhotosSettingPermission,
-              width: 40,
-              height: 40,
+              width: MediaPickerStyle.photoPermissionIconSize,
+              height: MediaPickerStyle.photoPermissionIconSize,
             ),
             Text(
               L10n.of(context)!.tapToAllowAccessToYourGallery,
-              style: Theme.of(context)
-                  .textTheme
-                  .titleSmall
-                  ?.copyWith(color: LinagoraRefColors.material().neutral),
+              style: Theme.of(context).textTheme.titleSmall?.copyWith(
+                    color: LinagoraRefColors.material().neutral,
+                    fontWeight: MediaPickerStyle.photoPermissionFontWeight,
+                    fontSize: MediaPickerStyle.photoPermissionFontSize,
+                  ),
               textAlign: TextAlign.center,
             ),
           ],

diff --git a/lib/presentation/style/media_picker_style.dart b/lib/presentation/style/media_picker_style.dart
@@ -42,4 +42,10 @@ class MediaPickerStyle {
   static const double borderSideWidth = 1.5;
 
   static const double minFontSize = 8;
+
+  static double photoPermissionIconSize = 48.0;
+
+  static double photoPermissionFontSize = 18;
+
+  static FontWeight photoPermissionFontWeight = FontWeight.w600;
 }