feat: improve coverage and resolve problems to generate the img

Author: Endika

README.md

@@ -36,7 +36,7 @@ Pictograms are universally easy to identify, making communication simple and int
 
 - **Pictogram keyboard** with categorized icons (descriptive, people, prepositions, determiners, nouns, verbs)
 - **Visual board** to compose messages by selecting pictograms
-- **Share** the board as an image with text description
+- **Share** the board as a 1080×1080 image with text description
 - **Installable PWA** — add to home screen on Android and iOS
 - **Multi-language support**: English, Spanish, Catalan, Basque, French, Galician, Portuguese, Valencian
 - **Accessibility-focused** design with semantic labels for screen readers
@@ -73,18 +73,19 @@ lib/
 ├── main.dart                     # App entry point
 ├── data/
 │   └── pictogram_data.dart       # Pictogram categories and icon data
-├── utils/
-│   └── pictogram_utils.dart      # Icon utility functions
 ├── screens/
 │   └── pictotap_screen.dart      # Main screen with board and state
+├── services/
+│   ├── image_saver.dart          # Platform export selector
+│   ├── image_saver_native.dart   # Native image sharing (Android/iOS)
+│   ├── image_saver_web.dart      # Web image download / Web Share API
+│   └── image_saver_stub.dart     # Stub for unsupported platforms
+├── utils/
+│   └── pictogram_utils.dart      # Icon utility functions
 ├── widgets/
+│   ├── board_empty_hint.dart     # Empty board hint animation
 │   ├── pictogram_icon.dart       # Pictogram icon widget
-│   ├── pictogram_keyboard.dart   # Keyboard widget with categories
-│   └── board_empty_hint.dart     # Empty board hint animation
-├── image_saver.dart              # Platform export selector
-├── image_saver_native.dart       # Native image sharing
-├── image_saver_web.dart          # Web image download
-├── image_saver_stub.dart         # Stub for unsupported platforms
+│   └── pictogram_keyboard.dart   # Keyboard widget with categories
 └── l10n/                         # Localization (ARB files)
 ```
 

lib/data/pictogram_data.dart

@@ -9,7 +9,7 @@ class KeyboardCategory {
 }
 
 const String spaceIcon = '__space__';
-const int defaultBoardMaxIcons = 10;
+const int defaultBoardMaxIcons = 4;
 const int maxRecommendationIcons = 6;
 
 const List<String> descriptiveIcons = [

lib/screens/pictotap_screen.dart

@@ -1,9 +1,11 @@
 import 'dart:async';
+import 'dart:math' as math;
+import 'dart:typed_data';
 import 'dart:ui' as ui;
 import 'package:flutter/material.dart';
 import 'package:flutter/rendering.dart';
 import 'package:pictotap/data/pictogram_data.dart';
-import 'package:pictotap/image_saver.dart';
+import 'package:pictotap/services/image_saver.dart';
 import 'package:pictotap/l10n/app_localizations.dart';
 import 'package:pictotap/utils/pictogram_utils.dart';
 import 'package:pictotap/widgets/board_empty_hint.dart';
@@ -28,7 +30,9 @@ class _PictoTapScreenState extends State<PictoTapScreen> {
 
   static const String _backgroundAssetPath =
       'assets/background/background.webp';
-  static const double _boardIconSize = 80;
+  static const double _boardIconSize = 110;
+  static const int _shareImageSize = 1080;
+  static const Color _backgroundFallback = Color(0xFFF5F0EB);
   static const Duration _animationDuration = Duration(milliseconds: 450);
 
   void _addIcon(String icon) {
@@ -110,11 +114,51 @@ class _PictoTapScreenState extends State<PictoTapScreen> {
           as RenderRepaintBoundary?;
       if (boundary == null) return;
 
-      final image = await boundary.toImage(pixelRatio: 3);
-      final byteData = await image.toByteData(format: ui.ImageByteFormat.png);
-      if (byteData == null) return;
+      const int out = _shareImageSize;
+      final bSize = boundary.size;
+      final pixelRatio = out / math.min(bSize.width, bSize.height);
+      final rawImage = await boundary.toImage(pixelRatio: pixelRatio);
 
-      final pngBytes = byteData.buffer.asUint8List();
+      final srcW = rawImage.width;
+      final srcH = rawImage.height;
+      final rawBytes = await rawImage.toByteData(
+        format: ui.ImageByteFormat.rawStraightRgba,
+      );
+      if (rawBytes == null) return;
+      final srcPixels = rawBytes.buffer.asUint8List();
+
+      final cropSize = math.min(srcW, srcH);
+      final cropX = (srcW - cropSize) ~/ 2;
+      final cropY = (srcH - cropSize) ~/ 2;
+
+      final outPixels = Uint8List(out * out * 4);
+      for (var y = 0; y < out; y++) {
+        final sy = (cropY + y * cropSize ~/ out).clamp(0, srcH - 1);
+        for (var x = 0; x < out; x++) {
+          final sx = (cropX + x * cropSize ~/ out).clamp(0, srcW - 1);
+          final si = (sy * srcW + sx) * 4;
+          final di = (y * out + x) * 4;
+          outPixels[di] = srcPixels[si];
+          outPixels[di + 1] = srcPixels[si + 1];
+          outPixels[di + 2] = srcPixels[si + 2];
+          outPixels[di + 3] = srcPixels[si + 3];
+        }
+      }
+
+      final completer = Completer<ui.Image>();
+      ui.decodeImageFromPixels(
+        outPixels,
+        out,
+        out,
+        ui.PixelFormat.rgba8888,
+        completer.complete,
+      );
+      final outputImage = await completer.future;
+      final pngData =
+          await outputImage.toByteData(format: ui.ImageByteFormat.png);
+      if (pngData == null) return;
+
+      final pngBytes = pngData.buffer.asUint8List();
       final fileName =
           'pictotap-board-${DateTime.now().millisecondsSinceEpoch}.png';
       await saveAndShareImage(pngBytes, fileName, shareText);
@@ -223,30 +267,29 @@ class _PictoTapScreenState extends State<PictoTapScreen> {
         ),
         foregroundColor: Colors.white,
       ),
-      body: Stack(
+      body: Column(
         children: [
-          Positioned.fill(
-            child: Image.asset(
-              _backgroundAssetPath,
-              fit: BoxFit.cover,
-              errorBuilder: (context, error, stackTrace) {
-                return Container(color: const Color(0xFFF5F0EB));
-              },
-            ),
-          ),
-          Positioned.fill(
-            child: Container(color: Colors.white.withAlpha(140)),
-          ),
-          Column(
-            children: [
-              Expanded(
-                child: GestureDetector(
-                  onTap: _toggleKeyboard,
-                  child: RepaintBoundary(
-                    key: _boardBoundaryKey,
-                    child: Container(
+          Expanded(
+            child: GestureDetector(
+              onTap: _toggleKeyboard,
+              child: RepaintBoundary(
+                key: _boardBoundaryKey,
+                child: Stack(
+                  children: [
+                    Positioned.fill(
+                      child: Image.asset(
+                        _backgroundAssetPath,
+                        fit: BoxFit.cover,
+                        errorBuilder: (context, error, stackTrace) {
+                          return Container(color: _backgroundFallback);
+                        },
+                      ),
+                    ),
+                    Positioned.fill(
+                      child: Container(color: Colors.white.withAlpha(140)),
+                    ),
+                    SizedBox(
                       width: double.infinity,
-                      color: Colors.transparent,
                       child: _selectedIcons.isEmpty
                           ? Center(
                               child: BoardEmptyHint(
@@ -267,19 +310,19 @@ class _PictoTapScreenState extends State<PictoTapScreen> {
                               ),
                             ),
                     ),
-                  ),
+                  ],
                 ),
               ),
-              if (_isKeyboardVisible)
-                PictogramKeyboard(
-                  onIconSelected: _addIcon,
-                  onSpace: _addSpace,
-                  onBackspace: _removeLast,
-                  recommendations: buildRecommendations(_selectedIcons),
-                  showLimitBanner: _showLimitReachedBanner,
-                ),
-            ],
+            ),
           ),
+          if (_isKeyboardVisible)
+            PictogramKeyboard(
+              onIconSelected: _addIcon,
+              onSpace: _addSpace,
+              onBackspace: _removeLast,
+              recommendations: buildRecommendations(_selectedIcons),
+              showLimitBanner: _showLimitReachedBanner,
+            ),
         ],
       ),
       floatingActionButton: _isKeyboardVisible

lib/image_saver.dart lib/services/image_saver.dartlib/image_saver.dart renamed to lib/services/image_saver.dart

@@ -0,0 +1,33 @@
+import 'package:flutter_test/flutter_test.dart';
+import 'package:pictotap/data/pictogram_data.dart';
+
+void main() {
+  test('all icon lists use correct prefix and have no duplicates', () {
+    final expected = {
+      'descriptive:': descriptiveIcons,
+      'people:': peopleIcons,
+      'prepositions:': prepositionsIcons,
+      'some:': someIcons,
+      'substantive:': substantiveIcons,
+      'verbs:': verbsIcons,
+    };
+
+    final allIcons = <String>[];
+    for (final entry in expected.entries) {
+      expect(entry.value, isNotEmpty, reason: '${entry.key} list is empty');
+      for (final icon in entry.value) {
+        expect(icon, startsWith(entry.key));
+      }
+      allIcons.addAll(entry.value);
+    }
+
+    expect(allIcons.toSet().length, allIcons.length,
+        reason: 'Found duplicate icons');
+  });
+
+  test('keyboardCategories maps all icon lists', () {
+    expect(keyboardCategories.length, 6);
+    expect(keyboardCategories[0].icons, descriptiveIcons);
+    expect(keyboardCategories[5].icons, verbsIcons);
+  });
+}

lib/image_saver_native.dart lib/services/image_saver_native.dartlib/image_saver_native.dart renamed to lib/services/image_saver_native.dart

@@ -0,0 +1,17 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:pictotap/l10n/app_localizations.dart';
+
+extension PumpApp on WidgetTester {
+  Future<void> pumpApp(Widget widget) async {
+    await pumpWidget(
+      MaterialApp(
+        localizationsDelegates: AppLocalizations.localizationsDelegates,
+        supportedLocales: AppLocalizations.supportedLocales,
+        locale: const Locale('en'),
+        home: Scaffold(body: widget),
+      ),
+    );
+    await pump();
+  }
+}

lib/image_saver_stub.dart lib/services/image_saver_stub.dartlib/image_saver_stub.dart renamed to lib/services/image_saver_stub.dart

@@ -0,0 +1,13 @@
+import 'dart:typed_data';
+
+import 'package:flutter_test/flutter_test.dart';
+import 'package:pictotap/services/image_saver_stub.dart';
+
+void main() {
+  test('saveAndShareImage throws UnsupportedError', () {
+    expect(
+      () => saveAndShareImage(Uint8List(0), 'test.png', 'text'),
+      throwsA(isA<UnsupportedError>()),
+    );
+  });
+}