From 1218cf254a2a03f23640f42b4893dc480e755b41 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Beno=C3=AEt=20Harrault?= <benoit@harrault.fr>
Date: Mon, 6 Nov 2023 21:07:41 +0100
Subject: [PATCH] Improve some layout

---
 android/gradle.properties                       |  4 ++--
 assets/translations/en.json                     |  3 +--
 assets/translations/fr.json                     |  5 ++---
 .../metadata/android/en-US/changelogs/21.txt    |  1 +
 .../metadata/android/fr-FR/changelogs/21.txt    |  1 +
 lib/ui/screens/main_screen.dart                 | 17 +++++++++++------
 .../widgets/main_screen/counts_by_day_card.dart |  4 ++--
 .../main_screen/counts_by_hour_card.dart        |  4 ++--
 .../widgets/main_screen/discoveries_card.dart   |  4 ++--
 lib/ui/widgets/main_screen/statistics_card.dart |  4 ++--
 .../widgets/main_screen/statistics_content.dart | 11 ++---------
 lib/ui/widgets/main_screen/timeline_card.dart   |  4 ++--
 pubspec.yaml                                    |  2 +-
 13 files changed, 31 insertions(+), 33 deletions(-)
 create mode 100644 fastlane/metadata/android/en-US/changelogs/21.txt
 create mode 100644 fastlane/metadata/android/fr-FR/changelogs/21.txt

diff --git a/android/gradle.properties b/android/gradle.properties
index 24add27..eeed3ef 100644
--- a/android/gradle.properties
+++ b/android/gradle.properties
@@ -1,5 +1,5 @@
 org.gradle.jvmargs=-Xmx1536M
 android.useAndroidX=true
 android.enableJetifier=true
-app.versionName=0.0.20
-app.versionCode=20
+app.versionName=0.0.21
+app.versionCode=21
diff --git a/assets/translations/en.json b/assets/translations/en.json
index 18cc8f1..d944c41 100644
--- a/assets/translations/en.json
+++ b/assets/translations/en.json
@@ -5,8 +5,7 @@
   "statistics_total_scrobbles_count": "Total scrobbles count: {count}",
   "statistics_last_scrobble": "Last scrobble: {datetime}",
   "statistics_selected_period": "On last {daysCount} days:",
-  "statistics_recent_scrobbles_count": "Scrobbles: {count}",
-  "statistics_discoveries": "Discoveries: {artistsCount} artists / {tracksCount} tracks",
+  "statistics_recent_scrobbles_count_and_discoveries": "{count} scrobbles and {artistsCount} artists / {tracksCount} tracks discovered.",
 
   "timeline_title": "Recent scrobbles ({daysCount} days)",
   "counts_by_day": "Counts by day ({daysCount} days)",
diff --git a/assets/translations/fr.json b/assets/translations/fr.json
index 5b69d01..a0a5011 100644
--- a/assets/translations/fr.json
+++ b/assets/translations/fr.json
@@ -4,9 +4,8 @@
   "global_statistics": "Statistiques globales d'écoutes",
   "statistics_total_scrobbles_count": "Nombre total d'écoutes : {count}",
   "statistics_last_scrobble": "Dernière écoute : {datetime}",
-  "statistics_selected_period": "Sur les {daysCount} derniers jours:",
-  "statistics_recent_scrobbles_count": "Écoutes : {count}",
-  "statistics_discoveries": "Découvertes : {artistsCount} artistes / {tracksCount} morceaux",
+  "statistics_selected_period": "Sur les {daysCount} derniers jours :",
+  "statistics_recent_scrobbles_count_and_discoveries": "{count} écoutes et {artistsCount} artistes / {tracksCount} morceaux découverts.",
 
   "timeline_title": "Écoutes récentes ({daysCount} jours)",
   "counts_by_day": "Écoutes par jour ({daysCount} jours)",
diff --git a/fastlane/metadata/android/en-US/changelogs/21.txt b/fastlane/metadata/android/en-US/changelogs/21.txt
new file mode 100644
index 0000000..cba4984
--- /dev/null
+++ b/fastlane/metadata/android/en-US/changelogs/21.txt
@@ -0,0 +1 @@
+Minor layout improvments.
diff --git a/fastlane/metadata/android/fr-FR/changelogs/21.txt b/fastlane/metadata/android/fr-FR/changelogs/21.txt
new file mode 100644
index 0000000..ede2372
--- /dev/null
+++ b/fastlane/metadata/android/fr-FR/changelogs/21.txt
@@ -0,0 +1 @@
+Améliorations visuelles mineures.
diff --git a/lib/ui/screens/main_screen.dart b/lib/ui/screens/main_screen.dart
index ee6b344..780c644 100644
--- a/lib/ui/screens/main_screen.dart
+++ b/lib/ui/screens/main_screen.dart
@@ -6,26 +6,31 @@ import '../widgets/main_screen/discoveries_card.dart';
 import '../widgets/main_screen/statistics_card.dart';
 import '../widgets/main_screen/timeline_card.dart';
 
-class MainScreen extends StatelessWidget {
+class MainScreen extends StatefulWidget {
   const MainScreen({super.key});
 
+  @override
+  State<MainScreen> createState() => _MainScreenState();
+}
+
+class _MainScreenState extends State<MainScreen> {
   @override
   Widget build(BuildContext context) {
     return Material(
       color: Theme.of(context).colorScheme.background,
       child: ListView(
-        padding: const EdgeInsets.symmetric(horizontal: 16),
+        padding: const EdgeInsets.symmetric(horizontal: 4),
         physics: const BouncingScrollPhysics(),
         children: <Widget>[
           const SizedBox(height: 90),
           const StatisticsCard(),
-          const SizedBox(height: 8),
+          const SizedBox(height: 6),
           const ChartTimelineCard(),
-          const SizedBox(height: 8),
+          const SizedBox(height: 6),
           const ChartDiscoveriesCard(),
-          const SizedBox(height: 8),
+          const SizedBox(height: 6),
           const ChartCountsByDayCard(),
-          const SizedBox(height: 8),
+          const SizedBox(height: 6),
           const ChartCountsByHourCard(),
           const SizedBox(height: 36),
         ],
diff --git a/lib/ui/widgets/main_screen/counts_by_day_card.dart b/lib/ui/widgets/main_screen/counts_by_day_card.dart
index ab39210..7f46132 100644
--- a/lib/ui/widgets/main_screen/counts_by_day_card.dart
+++ b/lib/ui/widgets/main_screen/counts_by_day_card.dart
@@ -28,11 +28,11 @@ class ChartCountsByDayCard extends StatelessWidget {
           color: Theme.of(context).colorScheme.surface,
           shape: const RoundedRectangleBorder(
             borderRadius: BorderRadius.all(
-              Radius.circular(12),
+              Radius.circular(8),
             ),
           ),
           child: Padding(
-            padding: const EdgeInsets.all(12.0),
+            padding: const EdgeInsets.all(8.0),
             child: ChartCountsByDayCardContent(
               daysCount: daysCount,
               chartData: snapshot.hasData
diff --git a/lib/ui/widgets/main_screen/counts_by_hour_card.dart b/lib/ui/widgets/main_screen/counts_by_hour_card.dart
index a5def57..8060aed 100644
--- a/lib/ui/widgets/main_screen/counts_by_hour_card.dart
+++ b/lib/ui/widgets/main_screen/counts_by_hour_card.dart
@@ -29,11 +29,11 @@ class ChartCountsByHourCard extends StatelessWidget {
           color: Theme.of(context).colorScheme.surface,
           shape: const RoundedRectangleBorder(
             borderRadius: BorderRadius.all(
-              Radius.circular(12),
+              Radius.circular(8),
             ),
           ),
           child: Padding(
-            padding: const EdgeInsets.all(12.0),
+            padding: const EdgeInsets.all(8.0),
             child: ChartCountsByHourCardContent(
               daysCount: daysCount,
               chartData: snapshot.hasData
diff --git a/lib/ui/widgets/main_screen/discoveries_card.dart b/lib/ui/widgets/main_screen/discoveries_card.dart
index 98d1065..838d379 100644
--- a/lib/ui/widgets/main_screen/discoveries_card.dart
+++ b/lib/ui/widgets/main_screen/discoveries_card.dart
@@ -28,11 +28,11 @@ class ChartDiscoveriesCard extends StatelessWidget {
           color: Theme.of(context).colorScheme.surface,
           shape: const RoundedRectangleBorder(
             borderRadius: BorderRadius.all(
-              Radius.circular(12),
+              Radius.circular(8),
             ),
           ),
           child: Padding(
-            padding: const EdgeInsets.all(12.0),
+            padding: const EdgeInsets.all(8.0),
             child: ChartDiscoveriesCardContent(
               daysCount: daysCount,
               chartData: snapshot.hasData
diff --git a/lib/ui/widgets/main_screen/statistics_card.dart b/lib/ui/widgets/main_screen/statistics_card.dart
index 329915b..4285410 100644
--- a/lib/ui/widgets/main_screen/statistics_card.dart
+++ b/lib/ui/widgets/main_screen/statistics_card.dart
@@ -28,11 +28,11 @@ class StatisticsCard extends StatelessWidget {
           color: Theme.of(context).colorScheme.primary,
           shape: const RoundedRectangleBorder(
             borderRadius: BorderRadius.all(
-              Radius.circular(12),
+              Radius.circular(8),
             ),
           ),
           child: Padding(
-            padding: const EdgeInsets.all(12.0),
+            padding: const EdgeInsets.all(8.0),
             child: StatisticsContent(
               statistics: snapshot.hasData
                   ? StatisticsData.fromJson(jsonDecode(snapshot.data.toString()))
diff --git a/lib/ui/widgets/main_screen/statistics_content.dart b/lib/ui/widgets/main_screen/statistics_content.dart
index c4659e4..bee5636 100644
--- a/lib/ui/widgets/main_screen/statistics_content.dart
+++ b/lib/ui/widgets/main_screen/statistics_content.dart
@@ -43,7 +43,7 @@ class StatisticsContent extends StatelessWidget {
         ),
         Text(
           'statistics_selected_period',
-          style: textTheme.bodyMedium,
+          style: textTheme.bodyMedium!.apply(fontWeightDelta: 2),
         ).tr(
           namedArgs: {
             'daysCount':
@@ -51,18 +51,11 @@ class StatisticsContent extends StatelessWidget {
           },
         ),
         Text(
-          'statistics_recent_scrobbles_count',
+          'statistics_recent_scrobbles_count_and_discoveries',
           style: textTheme.bodyMedium,
         ).tr(
           namedArgs: {
             'count': this.isLoading ? placeholder : this.statistics.recentCount.toString(),
-          },
-        ),
-        Text(
-          'statistics_discoveries',
-          style: textTheme.bodyMedium,
-        ).tr(
-          namedArgs: {
             'artistsCount': this.isLoading
                 ? placeholder
                 : this.statistics.firstPlayedArtistsCount.toString(),
diff --git a/lib/ui/widgets/main_screen/timeline_card.dart b/lib/ui/widgets/main_screen/timeline_card.dart
index 843b93b..931ea22 100644
--- a/lib/ui/widgets/main_screen/timeline_card.dart
+++ b/lib/ui/widgets/main_screen/timeline_card.dart
@@ -28,11 +28,11 @@ class ChartTimelineCard extends StatelessWidget {
           color: Theme.of(context).colorScheme.surface,
           shape: const RoundedRectangleBorder(
             borderRadius: BorderRadius.all(
-              Radius.circular(12),
+              Radius.circular(8),
             ),
           ),
           child: Padding(
-            padding: const EdgeInsets.all(12.0),
+            padding: const EdgeInsets.all(8.0),
             child: ChartTimelineCardContent(
               daysCount: daysCount,
               chartData: snapshot.hasData
diff --git a/pubspec.yaml b/pubspec.yaml
index 8551972..0e1c58c 100644
--- a/pubspec.yaml
+++ b/pubspec.yaml
@@ -3,7 +3,7 @@ description: Display scrobbles data and charts
 
 publish_to: 'none'
 
-version: 0.0.20+20
+version: 0.0.21+21
 
 environment:
   sdk: '^3.0.0'
-- 
GitLab