CHANGELOG.md

@@ -1,10 +1,7 @@
 
-## 1.1.1
-
 ## 1.1.0
 
 * refactor to clear domain models
-* follow MVVM
 
 ## 1.0.0
 

README.md

@@ -22,9 +22,8 @@ flutter analyze
 # > run unit tests and widget tests
 flutter test
 # > run integration tests
-# flutter test integration_test/ -d <device_id>
+flutter test integration_test/ -d <device_id>
-# Examples: --device=windows | --device=linux | --device=macos | --device=chrome
+# dart run tool/run_integration_tests.dart --device=linux
-dart run tool/run_integration_tests.dart --device=<device_id>
 
 # dart run tool/gen_view_wireframe_md.dart
 # flutter pub run dead_code_analyzer
@@ -38,7 +37,6 @@ flutter run -d <device_id>
 #### Windows
 
 ```bash
-dart run pdfrx:remove_wasm_modules
 flutter build windows
 # create windows installer
 flutter pub run msix:create
@@ -72,7 +70,6 @@ Access your app at [http://localhost:8080](http://localhost:8080)
 For Linux
 
 ```bash
-dart run pdfrx:remove_wasm_modules
 flutter build linux
 cp -r build/linux/x64/release/bundle/ AppDir
 appimagetool-x86_64.AppImage AppDir

android/app/src/main/AndroidManifest.xml

@@ -2,7 +2,7 @@
     <application
         android:label="pdf_signature"
         android:name="${applicationName}"
-        android:icon="@mipmap/ic_launcher">
+        android:icon="@mipmap/launcher_icon">
         <activity
             android:name=".MainActivity"
             android:exported="true"

android/settings.gradle.kts

@@ -19,7 +19,7 @@ pluginManagement {
 plugins {
     id("dev.flutter.flutter-plugin-loader") version "1.0.0"
     id("com.android.application") version "8.7.0" apply false
-    id("org.jetbrains.kotlin.android") version "2.1.0" apply false
+    id("org.jetbrains.kotlin.android") version "1.8.22" apply false
 }
 
 include(":app")

assets/icon/pdf_signature-icon.svg

@@ -1,44 +1,52 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<!-- Cropped viewBox from 0 0 128 128 to 8 8 112 112 so icon artwork occupies full box -->
+<svg
-<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="128px" height="128px" viewBox="8 8 112 112" version="1.1">
+	xmlns="http://www.w3.org/2000/svg"
-<defs>
+	xmlns:xlink="http://www.w3.org/1999/xlink"
-<filter id="alpha" filterUnits="objectBoundingBox" x="0%" y="0%" width="100%" height="100%">
+	viewBox="0 0 64 64"
-  <feColorMatrix type="matrix" in="SourceGraphic" values="0 0 0 0 1 0 0 0 0 1 0 0 0 0 1 0 0 0 1 0"/>
+	width="64"
-</filter>
+	height="64"
-<mask id="mask0">
+	role="img"
-  <g filter="url(#alpha)">
+	aria-labelledby="title desc"
-<rect x="0" y="0" width="128" height="128" style="fill:rgb(0%,0%,0%);fill-opacity:0.0509804;stroke:none;"/>
+>
-  </g>
+	<title id="title">PDF Signature</title>
-</mask>
+	<desc id="desc">An app icon showing a PDF page with a folded corner and a handwritten signature.</desc>
-<clipPath id="clip1">
+
-  <rect x="0" y="0" width="128" height="128"/>
+	<!-- Background tile -->
-</clipPath>
+	<rect x="4" y="4" width="56" height="56" rx="12" fill="#2563EB" />
-<g id="surface5" clip-path="url(#clip1)">
+
-<path style=" stroke:none;fill-rule:nonzero;fill:rgb(0%,0%,0%);fill-opacity:1;" d="M 40 32 L 76 32 L 96 52 L 96 54 L 88 54 C 79.199219 54 72 46.800781 72 38 L 72 32 L 40 32 C 37.800781 32 36 33.800781 36 36 C 36 33.800781 37.800781 32 40 32 Z M 40 32 "/>
+	<!-- Paper with folded corner -->
-</g>
+	<g>
-<mask id="mask1">
+		<path
-  <g filter="url(#alpha)">
+			d="M20 16h18l10 10v22c0 2.2-1.8 4-4 4H20c-2.2 0-4-1.8-4-4V20c0-2.2 1.8-4 4-4z"
-<rect x="0" y="0" width="128" height="128" style="fill:rgb(0%,0%,0%);fill-opacity:0.898039;stroke:none;"/>
+			fill="#FFFFFF"
-  </g>
+		/>
-</mask>
+		<path
-<clipPath id="clip2">
+			d="M38 16v8c0 3.3 2.7 6 6 6h8l-14-14z"
-  <rect x="0" y="0" width="128" height="128"/>
+			fill="#F3F4F6"
-</clipPath>
+		/>
-<g id="surface8" clip-path="url(#clip2)">
+	</g>
-<path style=" stroke:none;fill-rule:nonzero;fill:rgb(89.803922%,22.352941%,20.784314%);fill-opacity:1;" d="M 50 56 C 50 54.894531 49.105469 54 48 54 C 46.894531 54 46 54.894531 46 56 C 46 57.105469 46.894531 58 48 58 C 49.105469 58 50 57.105469 50 56 Z M 50 56 "/>
+
-<path style=" stroke:none;fill-rule:nonzero;fill:rgb(89.803922%,22.352941%,20.784314%);fill-opacity:1;" d="M 58 56 C 58 54.894531 57.105469 54 56 54 C 54.894531 54 54 54.894531 54 56 C 54 57.105469 54.894531 58 56 58 C 57.105469 58 58 57.105469 58 56 Z M 58 56 "/>
+	<!-- Signature stroke -->
-<path style=" stroke:none;fill-rule:nonzero;fill:rgb(89.803922%,22.352941%,20.784314%);fill-opacity:1;" d="M 66 56 C 66 54.894531 65.105469 54 64 54 C 62.894531 54 62 54.894531 62 56 C 62 57.105469 62.894531 58 64 58 C 65.105469 58 66 57.105469 66 56 Z M 66 56 "/>
+	<path
-</g>
+		d="M18 42c3-2 6-2.2 8.5 0 2.5 2.2 4.8 1.8 8.2-1.2 3.4-3 6.9-5.3 9.4-2.8 1.2 1.2 0.5 3.2-1.2 3.6-3.5 0.9 3.3-6.8 6.4-4.6 2 1.4-1.5 6.7-4.8 7.8-4.6 1.6-10.9-0.6-13.8-0.6-4.4 0-7.5 2.4-12 2.8"
-</defs>
+		fill="none"
-<g id="surface1">
+		stroke="#1F2937"
-<path style=" stroke:none;fill-rule:nonzero;fill:rgb(14.509805%,38.82353%,92.156863%);fill-opacity:1;" d="M 32 8 L 96 8 C 109.253906 8 120 18.746094 120 32 L 120 96 C 120 109.253906 109.253906 120 96 120 L 32 120 C 18.746094 120 8 109.253906 8 96 L 8 32 C 8 18.746094 18.746094 8 32 8 Z M 32 8 "/>
+		stroke-width="2.5"
-<!-- Slight enlargement (1.05x) of document artwork -->
+		stroke-linecap="round"
-<g id="doc" transform="matrix(1.05 0 0 1.05 -4.8 -3.4)">
+		stroke-linejoin="round"
-  <path style=" stroke:none;fill-rule:nonzero;fill:rgb(100%,100%,100%);fill-opacity:1;" d="M 40 32 L 76 32 L 96 52 L 96 96 C 96 100.398438 92.398438 104 88 104 L 40 104 C 35.601562 104 32 100.398438 32 96 L 32 40 C 32 35.601562 35.601562 32 40 32 Z M 40 32 "/>
+	/>
-  <path style=" stroke:none;fill-rule:nonzero;fill:rgb(95.294118%,95.686275%,96.470588%);fill-opacity:1;" d="M 76 32 L 76 48 C 76 54.601562 81.398438 60 88 60 L 104 60 Z M 76 32 "/>
+
-  <path style="fill:none;stroke-width:2.5;stroke-linecap:round;stroke-linejoin:round;stroke:rgb(12.156863%,16.078432%,21.568628%);stroke-opacity:1;stroke-miterlimit:4;" d="M 18 42 C 21 40 24 39.800781 26.5 42 C 29 44.199219 31.300781 43.800781 34.699219 40.800781 C 38.099609 37.800781 41.599609 35.5 44.099609 38 C 45.300781 39.199219 44.599609 41.199219 42.900391 41.599609 C 39.400391 42.5 46.199219 34.800781 49.300781 37 C 51.300781 38.400391 47.800781 43.699219 44.5 44.800781 C 39.900391 46.400391 33.599609 44.199219 30.699219 44.199219 C 26.300781 44.199219 23.199219 46.599609 18.699219 47 " transform="matrix(2,0,0,2,0,0)"/>
+	<!-- Subtle page shadow for depth (kept minimal for clarity) -->
-  <use xlink:href="#surface5" mask="url(#mask0)"/>
+	<path
-  <use xlink:href="#surface8" mask="url(#mask1)"/>
+		d="M20 16h18l10 10v1H44c-4.4 0-8-3.6-8-8v-3H20c-1.1 0-2 .9-2 2v0c0-1.1.9-2 2-2z"
-</g>
+		fill="#000"
-</g>
+		opacity=".05"
+	/>
+
+	<!-- Optional PDF label dots (very subtle) -->
+	<g fill="#E53935" opacity=".9">
+		<circle cx="24" cy="28" r="1" />
+		<circle cx="28" cy="28" r="1" />
+		<circle cx="32" cy="28" r="1" />
+	</g>
 </svg>

docs/meta-arch.md

@@ -94,5 +94,3 @@ Some rule of thumb:
   * whole app use its image object as image representation.
   * aware that minimize, encode/decode usage, because its has poor performance on web
   * `ColorFilterGenerator` can not replace `adjustColor` due to custom background removal algorithm need at last stage. It is GPU based, and offscreen-render then readback is not ideal.
-* [responsive_framework]
-  * RWD support

docs/wireframe.md

@@ -78,23 +78,6 @@ Illustration:
 
 ---
 
-### Mobile PDF screen (phone)
-
-Purpose: compact PDF viewing and signature placement on small screens.
-Route: root --> opened (mobile)
-
-Design notes:
-- Top app bar with: menu (page thumbnails), title with current page and total pages (prev/next).
-- Center viewport supports scroll and pinch-zoom.
-- Bottom sheet button to show Signatures drawers.
-  - bottom drawer to add/drag a signature onto the page.
-
-Illustration:
-
-![](wireframe.assets/mobile_pdf_phone.excalidraw)
-
----
-
 ## How to view and export
 
 We keep links in this file pointing to `.excalidraw`. To preview the SVGs and generate `docs/.wireframe.md` with `.svg` links, run from repo root:

integration_test/export_flow_test.dart

@@ -1,35 +1,70 @@
-import 'dart:io';
+import 'dart:typed_data';
-
-import 'package:cross_file/cross_file.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:integration_test/integration_test.dart';
-import 'package:pdf_signature/data/repositories/document_repository.dart';
+import 'dart:io';
-import 'package:pdf_signature/data/repositories/preferences_repository.dart';
+import 'package:file_selector/file_selector.dart' as fs;
+
 import 'package:pdf_signature/data/services/export_service.dart';
-import 'package:pdf_signature/l10n/app_localizations.dart';
+
-import 'package:pdf_signature/ui/features/pdf/view_model/pdf_export_view_model.dart';
+import 'package:pdf_signature/data/repositories/signature_asset_repository.dart';
+import 'package:pdf_signature/data/repositories/signature_card_repository.dart';
+import 'package:pdf_signature/data/repositories/document_repository.dart';
+import 'package:pdf_signature/domain/models/model.dart';
 import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
+import 'package:pdf_signature/ui/features/pdf/view_model/pdf_export_view_model.dart';
 import 'package:pdf_signature/ui/features/pdf/widgets/pages_sidebar.dart';
 import 'package:pdf_signature/ui/features/pdf/widgets/pdf_screen.dart';
-import 'package:pdf_signature/ui/features/pdf/widgets/pdf_viewer_widget.dart';
 import 'package:shared_preferences/shared_preferences.dart';
+import 'package:image/image.dart' as img;
+import 'package:pdf_signature/data/repositories/preferences_repository.dart';
+import 'package:pdf_signature/l10n/app_localizations.dart';
 
-// Note: We use the real ExportService via the repository; no mocks here.
+class RecordingExporter extends ExportService {
+  bool called = false;
+  @override
+  Future<bool> saveBytesToFile({required bytes, required outputPath}) async {
+    called = true;
+    return true;
+  }
+}
+
+// Lightweight fake exporter to avoid invoking heavy rasterization during tests
+class LightweightExporter extends ExportService {
+  @override
+  Future<Uint8List?> exportSignedPdfFromBytes({
+    required Uint8List srcBytes,
+    required Size uiPageSize,
+    required Uint8List? signatureImageBytes,
+    Map<int, List<SignaturePlacement>>? placementsByPage,
+    Map<String, img.Image>? libraryImages,
+    double targetDpi = 144.0,
+  }) async {
+    // Return minimal non-empty bytes; content isn't used further in tests
+    return Uint8List.fromList([1, 2, 3]);
+  }
+
+  @override
+  Future<bool> saveBytesToFile({
+    required Uint8List bytes,
+    required String outputPath,
+  }) async {
+    return true;
+  }
+}
 
 void main() {
-  final binding = IntegrationTestWidgetsFlutterBinding.ensureInitialized();
+  IntegrationTestWidgetsFlutterBinding.ensureInitialized();
-  binding.framePolicy = LiveTestWidgetsFlutterBindingFramePolicy.fullyLive;
 
   testWidgets('Save uses file selector (via provider) and injected exporter', (
     tester,
   ) async {
+    final fake = RecordingExporter();
     SharedPreferences.setMockInitialValues({});
     final prefs = await SharedPreferences.getInstance();
-    final pdfBytes =
-        await File('integration_test/data/sample-local-pdf.pdf').readAsBytes();
 
+    // For this test, we don't need the PDF bytes since it's not loaded
     await tester.pumpWidget(
       ProviderScope(
         overrides: [
@@ -37,21 +72,15 @@ void main() {
             (ref) => PreferencesStateNotifier(prefs),
           ),
           documentRepositoryProvider.overrideWith(
-            (ref) => DocumentStateNotifier(service: ExportService())
+            (ref) => DocumentStateNotifier()..openPicked(pageCount: 3),
-              ..openDocument(
-                bytes: pdfBytes,
-                pageCount: 3,
-                knownPageCount: true,
-              ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
           ),
-          // Disable overlays to avoid long-lived overlay animations in CI
-          viewerOverlaysEnabledProvider.overrideWith((ref) => false),
           pdfExportViewModelProvider.overrideWith(
             (ref) => PdfExportViewModel(
               ref,
+              exporter: fake,
               savePathPicker: () async {
                 final dir = Directory.systemTemp.createTempSync('pdfsig_');
                 return '${dir.path}/output.pdf';
@@ -62,11 +91,11 @@ void main() {
         child: MaterialApp(
           localizationsDelegates: AppLocalizations.localizationsDelegates,
           supportedLocales: AppLocalizations.supportedLocales,
-          locale: const Locale('en'),
+          locale: Locale('en'),
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -81,14 +110,30 @@ void main() {
     expect(find.textContaining('Saved:'), findsOneWidget);
   });
 
-  testWidgets('Export completes successfully (FOSS path)', (tester) async {
+  // Helper to build a simple in-memory PNG as a signature image
-    // Verify the exporter completes and shows SnackBar using the single
+  Uint8List _makeSig() {
-    // FOSS path (pdfrx render + pdf compose) on all platforms.
+    final canvas = img.Image(width: 80, height: 40);
+    img.fill(canvas, color: img.ColorUint8.rgb(255, 255, 255));
+    img.drawLine(
+      canvas,
+      x1: 6,
+      y1: 20,
+      x2: 74,
+      y2: 20,
+      color: img.ColorUint8.rgb(0, 0, 0),
+    );
+    return Uint8List.fromList(img.encodePng(canvas));
+  }
+
+  testWidgets('E2E (integration): place and confirm keeps size', (
+    tester,
+  ) async {
+    final sigBytes = _makeSig();
     final pdfBytes =
         await File('integration_test/data/sample-local-pdf.pdf').readAsBytes();
+
     SharedPreferences.setMockInitialValues({});
     final prefs = await SharedPreferences.getInstance();
-
     await tester.pumpWidget(
       ProviderScope(
         overrides: [
@@ -96,60 +141,81 @@ void main() {
             (ref) => PreferencesStateNotifier(prefs),
           ),
           documentRepositoryProvider.overrideWith(
-            (ref) => DocumentStateNotifier(service: ExportService())
+            (ref) =>
-              ..openDocument(
+                DocumentStateNotifier()
-                bytes: pdfBytes,
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-                pageCount: 3,
-                knownPageCount: true,
-              ),
           ),
+          signatureAssetRepositoryProvider.overrideWith((ref) {
+            final c = SignatureAssetRepository();
+            c.addImage(img.decodeImage(sigBytes)!, name: 'image');
+            return c;
+          }),
+          signatureCardRepositoryProvider.overrideWith((ref) {
+            final cardRepo = SignatureCardStateNotifier();
+            final asset = SignatureAsset(
+              sigImage: img.decodeImage(sigBytes)!,
+              name: 'image',
+            );
+            cardRepo.addWithAsset(asset, 0.0);
+            return cardRepo;
+          }),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
           ),
-          pdfExportViewModelProvider.overrideWith(
-            (ref) => PdfExportViewModel(
-              ref,
-              savePathPicker: () async {
-                final dir = Directory.systemTemp.createTempSync(
-                  'pdfsig_linux_',
-                );
-                return '${dir.path}/out.pdf';
-              },
-            ),
-          ),
         ],
         child: MaterialApp(
           localizationsDelegates: AppLocalizations.localizationsDelegates,
           supportedLocales: AppLocalizations.supportedLocales,
-          locale: const Locale('en'),
+          locale: Locale('en'),
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
     );
-    await tester.pump();
-
-    await tester.tap(find.byKey(const Key('btn_save_pdf')));
     await tester.pumpAndSettle();
 
-    expect(find.textContaining('Saved:'), findsOneWidget);
+    final card = find.byKey(const Key('gd_signature_card_area')).first;
-  });
+    await tester.tap(card);
+    await tester.pump();
 
-  testWidgets('E2E (integration): place and confirm keeps size', (
+    final active = find.byKey(const Key('signature_overlay'));
-    tester,
+    expect(active, findsOneWidget);
-  ) async {
+    final sizeBefore = tester.getSize(active);
-    // Skip in integration environment: overlay interaction was refactored
-    // and this check is covered by widget tests.
-  }, skip: true);
 
-  testWidgets('E2E (integration): programmatic placement size matches', (
+    await tester.ensureVisible(active);
-    tester,
+    await tester.pumpAndSettle();
-  ) async {
+    // Programmatically simulate confirm: add placement with current rect and bound image, then clear active overlay.
-    // Skip in integration run; covered by lower-level widget tests.
+    final ctx = tester.element(find.byType(PdfSignatureHomePage));
-    return;
+    final container = ProviderScope.containerOf(ctx);
+    final r = container.read(pdfViewModelProvider).activeRect!;
+    final lib = container.read(signatureAssetRepositoryProvider);
+    final asset = lib.isNotEmpty ? lib.first : null;
+    final currentPage = container.read(pdfViewModelProvider).currentPage;
+    container
+        .read(documentRepositoryProvider.notifier)
+        .addPlacement(page: currentPage, rect: r, asset: asset);
+    // Clear active overlay by hiding signatures temporarily
+    // Note: signatureVisibilityProvider was removed in migration
+    // container.read(signatureVisibilityProvider.notifier).state = false;
+    await tester.pump();
+    // container.read(signatureVisibilityProvider.notifier).state = true;
+    await tester.pumpAndSettle();
+
+    final placed = find.byKey(const Key('placed_signature_0'));
+    expect(placed, findsOneWidget);
+    final sizeAfter = tester.getSize(placed);
+
+    expect(
+      (sizeAfter.width - sizeBefore.width).abs() < sizeBefore.width * 0.15,
+      isTrue,
+    );
+    expect(
+      (sizeAfter.height - sizeBefore.height).abs() < sizeBefore.height * 0.15,
+      isTrue,
+    );
   });
 
   // ---- PDF view interaction tests (merged from pdf_view_test.dart) ----
@@ -168,13 +234,9 @@ void main() {
             (ref) => PreferencesStateNotifier(prefs),
           ),
           documentRepositoryProvider.overrideWith(
-            (ref) => DocumentStateNotifier(
+            (ref) =>
-              service: ExportService(enableRaster: false),
+                DocumentStateNotifier()
-            )..openDocument(
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-              bytes: pdfBytes,
-              pageCount: 3,
-              knownPageCount: true,
-            ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
@@ -183,11 +245,11 @@ void main() {
         child: MaterialApp(
           localizationsDelegates: AppLocalizations.localizationsDelegates,
           supportedLocales: AppLocalizations.supportedLocales,
-          locale: const Locale('en'),
+          locale: Locale('en'),
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -218,13 +280,9 @@ void main() {
             (ref) => PreferencesStateNotifier(prefs),
           ),
           documentRepositoryProvider.overrideWith(
-            (ref) => DocumentStateNotifier(
+            (ref) =>
-              service: ExportService(enableRaster: false),
+                DocumentStateNotifier()
-            )..openDocument(
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-              bytes: pdfBytes,
-              pageCount: 3,
-              knownPageCount: true,
-            ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
@@ -233,11 +291,11 @@ void main() {
         child: MaterialApp(
           localizationsDelegates: AppLocalizations.localizationsDelegates,
           supportedLocales: AppLocalizations.supportedLocales,
-          locale: const Locale('en'),
+          locale: Locale('en'),
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -271,13 +329,9 @@ void main() {
             (ref) => PreferencesStateNotifier(prefs),
           ),
           documentRepositoryProvider.overrideWith(
-            (ref) => DocumentStateNotifier(
+            (ref) =>
-              service: ExportService(enableRaster: false),
+                DocumentStateNotifier()
-            )..openDocument(
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-              bytes: pdfBytes,
-              pageCount: 3,
-              knownPageCount: true,
-            ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
@@ -286,11 +340,11 @@ void main() {
         child: MaterialApp(
           localizationsDelegates: AppLocalizations.localizationsDelegates,
           supportedLocales: AppLocalizations.supportedLocales,
-          locale: const Locale('en'),
+          locale: Locale('en'),
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -327,12 +381,9 @@ void main() {
             (ref) => PreferencesStateNotifier(prefs),
           ),
           documentRepositoryProvider.overrideWith(
-            (ref) => DocumentStateNotifier(service: ExportService())
+            (ref) =>
-              ..openDocument(
+                DocumentStateNotifier()
-                bytes: pdfBytes,
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-                pageCount: 3,
-                knownPageCount: true,
-              ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
@@ -341,11 +392,11 @@ void main() {
         child: MaterialApp(
           localizationsDelegates: AppLocalizations.localizationsDelegates,
           supportedLocales: AppLocalizations.supportedLocales,
-          locale: const Locale('en'),
+          locale: Locale('en'),
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -365,101 +416,66 @@ void main() {
     expect(container.read(pdfViewModelProvider).currentPage, 2);
   });
 
-  testWidgets(
+  testWidgets('PDF View: tap viewer after export does not crash', (
-    'PDF View: tap viewer after export does not crash',
+    tester,
-    (tester) async {
+  ) async {
-      final pdfBytes =
+    final pdfBytes =
-          await File(
+        await File('integration_test/data/sample-local-pdf.pdf').readAsBytes();
-            'integration_test/data/sample-local-pdf.pdf',
+    SharedPreferences.setMockInitialValues({});
-          ).readAsBytes();
+    final prefs = await SharedPreferences.getInstance();
-      SharedPreferences.setMockInitialValues({});
-      final prefs = await SharedPreferences.getInstance();
 
-      await tester.pumpWidget(
+    await tester.pumpWidget(
-        ProviderScope(
+      ProviderScope(
-          overrides: [
+        overrides: [
-            preferencesRepositoryProvider.overrideWith(
+          preferencesRepositoryProvider.overrideWith(
-              (ref) => PreferencesStateNotifier(prefs),
+            (ref) => PreferencesStateNotifier(prefs),
-            ),
+          ),
-            documentRepositoryProvider.overrideWith(
+          documentRepositoryProvider.overrideWith(
-              (ref) => DocumentStateNotifier(service: ExportService())
+            (ref) =>
-                ..openDocument(
+                DocumentStateNotifier()
-                  bytes: pdfBytes,
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-                  pageCount: 3,
+          ),
-                  knownPageCount: true,
+          pdfViewModelProvider.overrideWith(
-                ),
+            (ref) => PdfViewModel(ref, useMockViewer: false),
-            ),
+          ),
-            pdfViewModelProvider.overrideWith(
+          pdfExportViewModelProvider.overrideWith(
-              (ref) => PdfViewModel(ref, useMockViewer: false),
+            (ref) => PdfExportViewModel(
-            ),
+              ref,
-            // Disable overlays to reduce post-export timers/animations.
+              exporter: LightweightExporter(),
-            viewerOverlaysEnabledProvider.overrideWith((ref) => false),
+              savePathPicker: () async {
-            // Override only save path picker to avoid native dialogs; use real exporter
+                final dir = Directory.systemTemp.createTempSync(
-            pdfExportViewModelProvider.overrideWith(
+                  'pdfsig_after_',
-              (ref) => PdfExportViewModel(
+                );
-                ref,
+                return '${dir.path}/output-after-export.pdf';
-                savePathPicker: () async {
+              },
-                  final dir = Directory.systemTemp.createTempSync(
-                    'pdfsig_after_',
-                  );
-                  return '${dir.path}/output-after-export.pdf';
-                },
-              ),
-            ),
-          ],
-          child: MaterialApp(
-            localizationsDelegates: AppLocalizations.localizationsDelegates,
-            supportedLocales: AppLocalizations.supportedLocales,
-            locale: const Locale('en'),
-            home: PdfSignatureHomePage(
-              onPickPdf: () async {},
-              onClosePdf: () {},
-              currentFile: XFile('test.pdf'),
             ),
           ),
+        ],
+        child: MaterialApp(
+          localizationsDelegates: AppLocalizations.localizationsDelegates,
+          supportedLocales: AppLocalizations.supportedLocales,
+          locale: const Locale('en'),
+          home: PdfSignatureHomePage(
+            onPickPdf: () async {},
+            onClosePdf: () {},
+            currentFile: fs.XFile('test.pdf'),
+          ),
         ),
-      );
+      ),
-      await tester.pumpAndSettle();
+    );
+    await tester.pumpAndSettle();
 
-      // Trigger export
+    // Trigger export
-      debugPrint('[AFTER_EXPORT] Tap save to start export');
+    await tester.tap(find.byKey(const Key('btn_save_pdf')));
-      await tester.tap(find.byKey(const Key('btn_save_pdf')));
+    await tester.pumpAndSettle();
-      // Wait for export to complete using a real async wait so the test harness
+
-      // doesn't expect frame settling.
+    // Tap on the page area; should not crash
-      await tester.runAsync(() async {
+    final pageArea = find.byKey(const ValueKey('pdf_page_area'));
-        final deadline = DateTime.now().add(const Duration(seconds: 6));
+    expect(pageArea, findsOneWidget);
-        while (DateTime.now().isBefore(deadline)) {
+    await tester.tap(pageArea);
-          try {
+    await tester.pumpAndSettle();
-            final container = ProviderScope.containerOf(
+
-              tester.element(find.byType(PdfSignatureHomePage)),
+    // Still present and responsive
-            );
+    expect(pageArea, findsOneWidget);
-            final exporting =
+  });
-                container.read(pdfExportViewModelProvider).exporting;
-            if (!exporting) break;
-          } catch (_) {
-            // If widget unmounted, just stop waiting.
-            break;
-          }
-          await Future<void>.delayed(const Duration(milliseconds: 100));
-        }
-      });
-      // Tap the viewer after export finished to ensure no crash
-      final viewer = find.byKey(const ValueKey('pdf_page_area'));
-      expect(viewer, findsOneWidget);
-      await tester.tap(viewer);
-      await tester.pump(const Duration(milliseconds: 150));
-      // Hard-unmount the app to stop any viewer timers/animations
-      await tester.pumpWidget(const SizedBox.shrink());
-      await tester.pump(const Duration(milliseconds: 250));
-      await tester.pump(const Duration(milliseconds: 250));
-      // Give async zone a brief chance to flush background timers
-      await tester.runAsync(() async {
-        await Future<void>.delayed(const Duration(milliseconds: 250));
-      });
-      debugPrint('[AFTER_EXPORT] Test end reached (no crash)');
-      // Ensure the test registers a completed assertion.
-      expect(true, isTrue);
-    },
-    timeout: const Timeout(Duration(minutes: 2)),
-  );
 }

integration_test/pdf_view_test.dart

@@ -3,7 +3,7 @@ import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:integration_test/integration_test.dart';
 import 'dart:io';
-import 'package:cross_file/cross_file.dart';
+import 'package:file_selector/file_selector.dart' as fs;
 
 import 'package:pdf_signature/ui/features/pdf/widgets/pdf_screen.dart';
 import 'package:pdf_signature/ui/features/pdf/widgets/pages_sidebar.dart';
@@ -33,11 +33,8 @@ void main() {
           ),
           documentRepositoryProvider.overrideWith(
             (ref) =>
-                DocumentStateNotifier()..openDocument(
+                DocumentStateNotifier()
-                  bytes: pdfBytes,
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-                  pageCount: 3,
-                  knownPageCount: true,
-                ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
@@ -50,7 +47,7 @@ void main() {
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -65,31 +62,14 @@ void main() {
     final vm = container.read(pdfViewModelProvider);
     expect(vm.currentPage, 1);
 
-    final controller = container.read(pdfViewModelProvider).controller;
+    container.read(pdfViewModelProvider.notifier).jumpToPage(2);
-    // Wait until the underlying viewer controller reports ready.
+    await tester.pumpAndSettle();
-    final readyStart = DateTime.now();
+    await tester.pump(const Duration(milliseconds: 120));
-    while (!controller.isReady) {
-      await tester.pump(const Duration(milliseconds: 40));
-      if (DateTime.now().difference(readyStart) > const Duration(seconds: 5)) {
-        fail('PdfViewerController never became ready');
-      }
-    }
-    Future<void> goAndAwait(int target) async {
-      controller.goToPage(pageNumber: target);
-      final start = DateTime.now();
-      while (container.read(pdfViewModelProvider).currentPage != target) {
-        await tester.pump(const Duration(milliseconds: 40));
-        if (DateTime.now().difference(start) > const Duration(seconds: 3)) {
-          fail(
-            'Timeout waiting to reach page $target (current=${container.read(pdfViewModelProvider).currentPage})',
-          );
-        }
-      }
-    }
-
-    await goAndAwait(2);
     expect(container.read(pdfViewModelProvider).currentPage, 2);
-    await goAndAwait(3);
+
+    container.read(pdfViewModelProvider.notifier).jumpToPage(3);
+    await tester.pumpAndSettle();
+    await tester.pump(const Duration(milliseconds: 120));
     expect(container.read(pdfViewModelProvider).currentPage, 3);
   });
 
@@ -107,11 +87,8 @@ void main() {
           ),
           documentRepositoryProvider.overrideWith(
             (ref) =>
-                DocumentStateNotifier()..openDocument(
+                DocumentStateNotifier()
-                  bytes: pdfBytes,
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-                  pageCount: 3,
-                  knownPageCount: true,
-                ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
@@ -124,7 +101,7 @@ void main() {
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -164,11 +141,8 @@ void main() {
           ),
           documentRepositoryProvider.overrideWith(
             (ref) =>
-                DocumentStateNotifier()..openDocument(
+                DocumentStateNotifier()
-                  bytes: pdfBytes,
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-                  pageCount: 3,
-                  knownPageCount: true,
-                ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
@@ -181,7 +155,7 @@ void main() {
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -254,11 +228,8 @@ void main() {
           ),
           documentRepositoryProvider.overrideWith(
             (ref) =>
-                DocumentStateNotifier()..openDocument(
+                DocumentStateNotifier()
-                  bytes: pdfBytes,
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-                  pageCount: 3,
-                  knownPageCount: true,
-                ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
@@ -271,7 +242,7 @@ void main() {
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -323,11 +294,8 @@ void main() {
           ),
           documentRepositoryProvider.overrideWith(
             (ref) =>
-                DocumentStateNotifier()..openDocument(
+                DocumentStateNotifier()
-                  bytes: pdfBytes,
+                  ..openPicked(pageCount: 3, bytes: pdfBytes),
-                  pageCount: 3,
-                  knownPageCount: true,
-                ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: false),
@@ -340,7 +308,7 @@ void main() {
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile('test.pdf'),
+            currentFile: fs.XFile('test.pdf'),
           ),
         ),
       ),
@@ -371,116 +339,4 @@ void main() {
     await tester.pumpAndSettle();
     expect(container.read(pdfViewModelProvider).currentPage, 3);
   });
-
-  testWidgets('PDF View: reopen another PDF via toolbar picker updates viewer', (
-    tester,
-  ) async {
-    final initialBytes =
-        await File('integration_test/data/sample-local-pdf.pdf').readAsBytes();
-    // 3 pages
-    final newBytes =
-        await File(
-          'integration_test/data/PPFZ-Local-Purchase-Form.pdf',
-        ).readAsBytes();
-    // 10 pages
-    SharedPreferences.setMockInitialValues({});
-    final prefs = await SharedPreferences.getInstance();
-
-    // We'll override onPickPdf to simulate opening a new file with a different page count
-    // TODO: Replace PPFZ-Local-Purchase-Form.pdf with a 10-page PDF to test page count change
-    late ProviderContainer container; // capture to use inside callback
-    Future<void> simulatePick() async {
-      container
-          .read(documentRepositoryProvider.notifier)
-          .openDocument(bytes: newBytes);
-      // Reset the current page explicitly to 1 as openPicked establishes new doc
-      container.read(pdfViewModelProvider.notifier).jumpToPage(1);
-    }
-
-    int? lastDocPageCount; // capture page count from callback
-    await tester.pumpWidget(
-      ProviderScope(
-        overrides: [
-          preferencesRepositoryProvider.overrideWith(
-            (ref) => PreferencesStateNotifier(prefs),
-          ),
-          documentRepositoryProvider.overrideWith(
-            (ref) =>
-                DocumentStateNotifier()..openDocument(
-                  bytes: initialBytes,
-                  pageCount: 3,
-                  knownPageCount: true,
-                ),
-          ),
-          pdfViewModelProvider.overrideWith(
-            (ref) => PdfViewModel(ref, useMockViewer: false),
-          ),
-        ],
-        child: Builder(
-          builder: (context) {
-            container = ProviderScope.containerOf(context);
-            return MaterialApp(
-              localizationsDelegates: AppLocalizations.localizationsDelegates,
-              supportedLocales: AppLocalizations.supportedLocales,
-              locale: const Locale('en'),
-              home: PdfSignatureHomePage(
-                onPickPdf: simulatePick,
-                onClosePdf: () {},
-                currentFile: XFile('initial.pdf'),
-                // The only reliable way to detect the new document load correctly
-                onDocumentChanged: (doc) {
-                  if (doc != null) {
-                    lastDocPageCount = doc.pages.length;
-                  }
-                },
-              ),
-            );
-          },
-        ),
-      ),
-    );
-
-    await tester.pumpAndSettle();
-    // Verify initial state Page 1/3
-    expect(find.byKey(const Key('lbl_page_info')), findsOneWidget);
-    final initialLabel =
-        tester.widget<Text>(find.byKey(const Key('lbl_page_info'))).data;
-    expect(initialLabel, contains('/3'));
-
-    // Tap open picker button to simulate opening new PDF
-    await tester.tap(find.byKey(const Key('btn_open_pdf_picker')));
-    // Allow frame to process state changes from simulatePick
-    await tester.pump();
-    await tester.pump(const Duration(milliseconds: 50));
-    await tester.pumpAndSettle();
-
-    // Wait for async page count detection to complete in repository
-    await tester.runAsync(() async {
-      final start = DateTime.now();
-      while (container.read(documentRepositoryProvider).pageCount != 10) {
-        await Future<void>.delayed(const Duration(milliseconds: 50));
-        await tester.pump();
-
-        if (DateTime.now().difference(start) > const Duration(seconds: 8)) {
-          final pageCount =
-              container.read(documentRepositoryProvider).pageCount;
-          fail(
-            'Timeout waiting for repository page count to update to 10 (current=$pageCount)',
-          );
-        }
-      }
-
-      // Wait for restoration mechanism to complete
-      await Future<void>.delayed(const Duration(milliseconds: 500));
-      await tester.pump();
-    });
-
-    final updatedLabel =
-        tester.widget<Text>(find.byKey(const Key('lbl_page_info'))).data;
-    expect(updatedLabel, contains('/10'));
-    // Verify that repository correctly analyzed PDF bytes and updated page count
-    expect(container.read(documentRepositoryProvider).pageCount, 10);
-    expect(lastDocPageCount, 10);
-    expect(container.read(pdfViewModelProvider).currentPage, 1);
-  });
 }

lib/app.dart

@@ -5,7 +5,6 @@ import 'package:pdf_signature/l10n/app_localizations.dart';
 import 'package:pdf_signature/routing/router.dart';
 import 'package:pdf_signature/ui/features/preferences/widgets/settings_screen.dart';
 import 'data/repositories/preferences_repository.dart';
-import 'package:responsive_framework/responsive_framework.dart';
 
 class MyApp extends StatelessWidget {
   const MyApp({super.key});
@@ -57,7 +56,7 @@ class MyApp extends StatelessWidget {
             routerConfig: ref.watch(routerProvider),
             builder: (context, child) {
               final router = ref.watch(routerProvider);
-              final content = Scaffold(
+              return Scaffold(
                 appBar: AppBar(
                   title: Text(AppLocalizations.of(context).appTitle),
                   actions: [
@@ -79,17 +78,6 @@ class MyApp extends StatelessWidget {
                 ),
                 body: child,
               );
-
-              // Apply Responsive Framework globally for layout and scrolling.
-              return ResponsiveBreakpoints.builder(
-                child: ClampingScrollWrapper.builder(context, content),
-                breakpoints: const [
-                  Breakpoint(start: 0, end: 450, name: MOBILE),
-                  Breakpoint(start: 451, end: 800, name: TABLET),
-                  Breakpoint(start: 801, end: 1920, name: DESKTOP),
-                  Breakpoint(start: 1921, end: double.infinity, name: '4K'),
-                ],
-              );
             },
           );
         },

lib/data/repositories/document_repository.dart

@@ -1,19 +1,15 @@
-import 'dart:isolate';
+import 'dart:typed_data';
-import 'package:flutter/foundation.dart';
 import 'package:image/image.dart' as img;
 import 'package:flutter/material.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:pdf_signature/data/services/export_service.dart';
-import 'package:pdfrx/pdfrx.dart';
 
 import '../../domain/models/model.dart';
 
 class DocumentStateNotifier extends StateNotifier<Document> {
-  DocumentStateNotifier({ExportService? service})
+  DocumentStateNotifier() : super(Document.initial());
-    : _service = service ?? ExportService(),
-      super(Document.initial());
 
-  final ExportService _service;
+  final ExportService _service = ExportService();
 
   @visibleForTesting
   void openSample() {
@@ -25,69 +21,7 @@ class DocumentStateNotifier extends StateNotifier<Document> {
     );
   }
 
-  /// Unified open API replacing multiple legacy variants.
+  void openPicked({required int pageCount, Uint8List? bytes}) {
-  ///
-  /// Usage patterns:
-  ///  openDocument(bytes: data) -> derive page count asynchronously.
-  ///  openDocument(bytes: data, pageCount: 203, knownPageCount: true) -> fast path.
-  ///  openDocument(pageCount: 5) -> open empty placeholder document (tests).
-  void openDocument({
-    Uint8List? bytes,
-    int? pageCount,
-    bool knownPageCount = false,
-  }) {
-    debugPrint(
-      '[DocumentRepository] openDocument called (bytes=${bytes?.length} pageCount=$pageCount known=$knownPageCount)',
-    );
-    if (bytes == null) {
-      // No bytes: treat as synthetic document (tests) using provided pageCount or default 1
-      final pc = pageCount ?? 1;
-      state = state.copyWith(
-        loaded: true,
-        pageCount: pc,
-        pickedPdfBytes: null,
-        placementsByPage: <int, List<SignaturePlacement>>{},
-      );
-      return;
-    }
-    // Bytes provided
-    if ((knownPageCount || pageCount != null) && pageCount != null) {
-      // Fast path: caller already determined count
-      state = state.copyWith(
-        loaded: true,
-        pageCount: pageCount.clamp(1, 9999),
-        pickedPdfBytes: bytes,
-        placementsByPage: <int, List<SignaturePlacement>>{},
-      );
-      return;
-    }
-    // Derive asynchronously
-    _openPickedAsync(bytes);
-  }
-
-  // --- Deprecated wrappers for backward compatibility (can be removed later) ---
-  @Deprecated('Use openDocument(bytes: ...) instead')
-  void openPicked({Uint8List? bytes}) => openDocument(bytes: bytes);
-
-  @Deprecated(
-    'Use openDocument(bytes: ..., pageCount: x, knownPageCount: true) instead',
-  )
-  void openPickedKnown({required int pageCount, required Uint8List bytes}) =>
-      openDocument(bytes: bytes, pageCount: pageCount, knownPageCount: true);
-
-  Future<void> _openPickedAsync(Uint8List bytes) async {
-    int pageCount = 1; // default fallback
-
-    try {
-      // Determine actual page count from PDF bytes
-      final doc = await PdfDocument.openData(bytes);
-      pageCount = doc.pages.length;
-      debugPrint('[DocumentRepository] PDF has $pageCount pages');
-    } catch (e) {
-      debugPrint('[DocumentRepository] Failed to read PDF page count: $e');
-      // Keep default pageCount = 1 on error
-    }
-
     state = state.copyWith(
       loaded: true,
       pageCount: pageCount,
@@ -96,23 +30,12 @@ class DocumentStateNotifier extends StateNotifier<Document> {
     );
   }
 
-  // For tests that need to specify page count explicitly
-  @visibleForTesting
-  @Deprecated(
-    'Use openDocument(pageCount: x) for synthetic docs or with bytes+knownPageCount',
-  )
-  void openPickedWithPageCount({required int pageCount, Uint8List? bytes}) =>
-      openDocument(bytes: bytes, pageCount: pageCount, knownPageCount: true);
-
   void close() {
     state = Document.initial();
   }
 
   void setPageCount(int count) {
     if (!state.loaded) return;
-    debugPrint(
-      '[DocumentRepository] setPageCount called: $count (current: ${state.pageCount})',
-    );
     state = state.copyWith(pageCount: count.clamp(1, 9999));
   }
 
@@ -149,12 +72,21 @@ class DocumentStateNotifier extends StateNotifier<Document> {
   // signature bytes were provided.
   static final img.Image _singleTransparentPng = img.Image(width: 1, height: 1);
 
-  @Deprecated('Use modifyPlacement')
   void updatePlacementRotation({
     required int page,
     required int index,
     required double rotationDeg,
-  }) => modifyPlacement(page: page, index: index, rotationDeg: rotationDeg);
+  }) {
+    if (!state.loaded) return;
+    final p = page.clamp(1, state.pageCount);
+    final map = Map<int, List<SignaturePlacement>>.from(state.placementsByPage);
+    final list = List<SignaturePlacement>.from(map[p] ?? const []);
+    if (index >= 0 && index < list.length) {
+      list[index] = list[index].copyWith(rotationDeg: rotationDeg);
+      map[p] = list;
+      state = state.copyWith(placementsByPage: map);
+    }
+  }
 
   void removePlacement({required int page, required int index}) {
     if (!state.loaded) return;
@@ -173,36 +105,21 @@ class DocumentStateNotifier extends StateNotifier<Document> {
   }
 
   // Update the rect of an existing placement on a page.
-  @Deprecated('Use modifyPlacement')
   void updatePlacementRect({
     required int page,
     required int index,
     required Rect rect,
-  }) => modifyPlacement(page: page, index: index, rect: rect);
-
-  /// Generic partial update for a placement. Any non-null field is applied.
-  void modifyPlacement({
-    required int page,
-    required int index,
-    Rect? rect,
-    double? rotationDeg,
-    SignatureAsset? asset,
-    GraphicAdjust? graphicAdjust,
   }) {
     if (!state.loaded) return;
     final p = page.clamp(1, state.pageCount);
     final map = Map<int, List<SignaturePlacement>>.from(state.placementsByPage);
     final list = List<SignaturePlacement>.from(map[p] ?? const []);
-    if (index < 0 || index >= list.length) return;
+    if (index >= 0 && index < list.length) {
-    final current = list[index];
+      final existing = list[index];
-    list[index] = current.copyWith(
+      list[index] = existing.copyWith(rect: rect);
-      rect: rect ?? current.rect,
+      map[p] = list;
-      rotationDeg: rotationDeg ?? current.rotationDeg,
+      state = state.copyWith(placementsByPage: map);
-      asset: asset ?? current.asset,
+    }
-      graphicAdjust: graphicAdjust ?? current.graphicAdjust,
-    );
-    map[p] = list;
-    state = state.copyWith(placementsByPage: map);
   }
 
   List<SignaturePlacement> placementsOn(int page) {
@@ -218,62 +135,21 @@ class DocumentStateNotifier extends StateNotifier<Document> {
     return list[index].asset;
   }
 
-  Future<bool> exportDocument({
+  Future<void> exportDocument({
     required String outputPath,
     required Size uiPageSize,
     required Uint8List? signatureImageBytes,
-    double targetDpi = 144.0,
   }) async {
-    final bytes = await exportDocumentToBytes(
+    if (!state.loaded || state.pickedPdfBytes == null) return;
-      uiPageSize: uiPageSize,
+    final bytes = await _service.exportSignedPdfFromBytes(
-      signatureImageBytes: signatureImageBytes,
-      targetDpi: targetDpi,
-    );
-
-    Future<void> _ = Future<void>.delayed(Duration.zero);
-
-    if (bytes == null) return false;
-    final ok = await _service.saveBytesToFile(
-      bytes: bytes,
-      outputPath: outputPath,
-    );
-    return ok;
-  }
-
-  Future<Uint8List?> exportDocumentToBytes({
-    required Size uiPageSize,
-    required Uint8List? signatureImageBytes,
-    double targetDpi = 144.0,
-  }) async {
-    if (!state.loaded || state.pickedPdfBytes == null) return null;
-    // Experimental: run export in a background isolate using `compute`.
-    // We serialize placements and signature assets to isolate-safe data.
-    try {
-      final args = _buildIsolateArgs(
-        srcBytes: state.pickedPdfBytes!,
-        uiPageSize: uiPageSize,
-        signatureImageBytes: signatureImageBytes,
-        placementsByPage: state.placementsByPage,
-        targetDpi: targetDpi,
-      );
-      final result = await compute<_ExportIsolateArgs, Uint8List?>(
-        _exportInIsolate,
-        args,
-      );
-      if (result != null) return result;
-    } catch (_) {
-      debugPrint('Warning: export in isolate failed');
-      // Fall back to main-isolate export if isolate fails (e.g., engine limitations).
-    }
-
-    // Fallback on main isolate
-    return await _service.exportSignedPdfFromBytes(
       srcBytes: state.pickedPdfBytes!,
       uiPageSize: uiPageSize,
       signatureImageBytes: signatureImageBytes,
       placementsByPage: state.placementsByPage,
-      targetDpi: targetDpi,
     );
+    if (bytes == null) return;
+    _service.saveBytesToFile(bytes: bytes, outputPath: outputPath);
+    // await
   }
 }
 
@@ -281,129 +157,3 @@ final documentRepositoryProvider =
     StateNotifierProvider<DocumentStateNotifier, Document>(
       (ref) => DocumentStateNotifier(),
     );
-
-/// --- Isolate helpers of DocumentRepository ---
-/// Following are helpers to transfer data to/from an isolate for export.
-
-class _ExportIsolateArgs {
-  final TransferableTypedData src;
-  final double pageW;
-  final double pageH;
-  final double targetDpi;
-  final List<_IsoPagePlacements> pages;
-  final TransferableTypedData? signatureImageBytes; // not used currently
-  _ExportIsolateArgs({
-    required this.src,
-    required this.pageW,
-    required this.pageH,
-    required this.targetDpi,
-    required this.pages,
-    required this.signatureImageBytes,
-  });
-}
-
-class _IsoPagePlacements {
-  final int page;
-  final List<_IsoPlacement> items;
-  _IsoPagePlacements(this.page, this.items);
-}
-
-class _IsoPlacement {
-  final double l, t, w, h;
-  final double rot;
-  final double contrast, brightness;
-  final bool bgRemoval;
-  final TransferableTypedData assetPng;
-  _IsoPlacement({
-    required this.l,
-    required this.t,
-    required this.w,
-    required this.h,
-    required this.rot,
-    required this.contrast,
-    required this.brightness,
-    required this.bgRemoval,
-    required this.assetPng,
-  });
-}
-
-_ExportIsolateArgs _buildIsolateArgs({
-  required Uint8List srcBytes,
-  required Size uiPageSize,
-  required Uint8List? signatureImageBytes,
-  required Map<int, List<SignaturePlacement>> placementsByPage,
-  required double targetDpi,
-}) {
-  final pages = <_IsoPagePlacements>[];
-  placementsByPage.forEach((page, items) {
-    final isoItems = <_IsoPlacement>[];
-    for (final p in items) {
-      // Encode the asset image to PNG for transfer; small count expected.
-      final png = Uint8List.fromList(img.encodePng(p.asset.sigImage, level: 3));
-      isoItems.add(
-        _IsoPlacement(
-          l: p.rect.left,
-          t: p.rect.top,
-          w: p.rect.width,
-          h: p.rect.height,
-          rot: p.rotationDeg,
-          contrast: p.graphicAdjust.contrast,
-          brightness: p.graphicAdjust.brightness,
-          bgRemoval: p.graphicAdjust.bgRemoval,
-          assetPng: TransferableTypedData.fromList([png]),
-        ),
-      );
-    }
-    pages.add(_IsoPagePlacements(page, isoItems));
-  });
-  return _ExportIsolateArgs(
-    src: TransferableTypedData.fromList([srcBytes]),
-    pageW: uiPageSize.width,
-    pageH: uiPageSize.height,
-    targetDpi: targetDpi,
-    pages: pages,
-    signatureImageBytes:
-        signatureImageBytes == null
-            ? null
-            : TransferableTypedData.fromList([signatureImageBytes]),
-  );
-}
-
-Future<Uint8List?> _exportInIsolate(_ExportIsolateArgs args) async {
-  // Rebuild placements
-  final placementsByPage = <int, List<SignaturePlacement>>{};
-  for (final page in args.pages) {
-    final list = <SignaturePlacement>[];
-    for (final it in page.items) {
-      final bytes = it.assetPng.materialize().asUint8List();
-      final decoded = img.decodePng(bytes);
-      if (decoded == null) continue;
-      final asset = SignatureAsset(sigImage: decoded);
-      list.add(
-        SignaturePlacement(
-          rect: Rect.fromLTWH(it.l, it.t, it.w, it.h),
-          asset: asset,
-          rotationDeg: it.rot,
-          graphicAdjust: GraphicAdjust(
-            contrast: it.contrast,
-            brightness: it.brightness,
-            bgRemoval: it.bgRemoval,
-          ),
-        ),
-      );
-    }
-    if (list.isNotEmpty) {
-      placementsByPage[page.page] = list;
-    }
-  }
-
-  final src = args.src.materialize().asUint8List();
-  final service = ExportService();
-  return await service.exportSignedPdfFromBytes(
-    srcBytes: src,
-    uiPageSize: Size(args.pageW, args.pageH),
-    signatureImageBytes: args.signatureImageBytes?.materialize().asUint8List(),
-    placementsByPage: placementsByPage,
-    targetDpi: args.targetDpi,
-  );
-}

lib/data/services/export_service.dart

@@ -1,13 +1,13 @@
 import 'dart:io';
-import 'dart:async';
 import 'dart:typed_data';
+import 'package:flutter/rendering.dart';
 import 'package:flutter/widgets.dart';
-import 'package:image/image.dart' as img;
 import 'package:pdf/widgets.dart' as pw;
 import 'package:pdf/pdf.dart' as pdf;
-import 'package:pdfrx_engine/pdfrx_engine.dart' as engine;
+import 'package:printing/printing.dart' as printing;
-import 'package:pdfrx/pdfrx.dart' show pdfrxFlutterInitialize;
+import 'package:image/image.dart' as img;
 import '../../domain/models/model.dart';
+// math moved to utils in rot
 import '../../utils/rotation_utils.dart' as rot;
 import '../../utils/background_removal.dart' as br;
 
@@ -18,32 +18,32 @@ import '../../utils/background_removal.dart' as br;
 //   cannot import/modify existing PDF pages. If/when a suitable FOSS library exists, wire it here.
 
 class ExportService {
-  ExportService({this.enableRaster = true});
+  /// Compose a new PDF from source PDF bytes; returns the resulting PDF bytes.
-  // Deprecated: retained for API compatibility. Raster is no longer used.
-  final bool enableRaster;
-
-  /// Compose a new PDF by rendering source pages to images (FOSS path via pdfrx)
-  /// and overlaying signature images at normalized rects. Returns resulting bytes.
   Future<Uint8List?> exportSignedPdfFromBytes({
     required Uint8List srcBytes,
-    required Size uiPageSize, // not used in this implementation
+    required Size uiPageSize,
-    required Uint8List?
+    required Uint8List? signatureImageBytes,
-    signatureImageBytes, // not used; placements carry images
     Map<int, List<SignaturePlacement>>? placementsByPage,
     Map<String, img.Image>? libraryImages,
     double targetDpi = 144.0,
   }) async {
-    // Caches per call
+    // Per-call caches to avoid redundant decode/encode and image embedding work
     final Map<String, img.Image> _baseImageCache = <String, img.Image>{};
     final Map<String, img.Image> _processedImageCache = <String, img.Image>{};
     final Map<String, Uint8List> _encodedPngCache = <String, Uint8List>{};
+    final Map<String, pw.MemoryImage> _memoryImageCache =
+        <String, pw.MemoryImage>{};
     final Map<String, double> _aspectRatioCache = <String, double>{};
 
+    // Returns a stable-ish cache key for bytes within this process (not content-hash, but good enough per-call)
     String _baseKeyForImage(img.Image im) =>
         'im:${identityHashCode(im)}:${im.width}x${im.height}';
     String _adjustKey(GraphicAdjust adj) =>
         'c=${adj.contrast}|b=${adj.brightness}|bg=${adj.bgRemoval}';
 
+    // Removed: PNG signature helper is no longer needed; we always encode to PNG explicitly.
+
+    // Resolve base (unprocessed) image for a placement, considering library override.
     img.Image _getBaseImage(SignaturePlacement placement) {
       final libKey = placement.asset.name;
       if (libKey != null && libraryImages != null) {
@@ -58,6 +58,7 @@ class ExportService {
       return placement.asset.sigImage;
     }
 
+    // Get processed image for a placement, with caching.
     img.Image _getProcessedImage(SignaturePlacement placement) {
       final base = _getBaseImage(placement);
       final key =
@@ -73,15 +74,14 @@ class ExportService {
           brightness: adj.brightness,
         );
       }
-      Future<void> _ = Future<void>.delayed(Duration.zero);
       if (adj.bgRemoval) {
         processed = br.removeNearWhiteBackground(processed, threshold: 240);
       }
-      Future<void> _ = Future<void>.delayed(Duration.zero);
       _processedImageCache[key] = processed;
       return processed;
     }
 
+    // Get PNG bytes for the processed image, caching the encoding.
     Uint8List _getProcessedPng(SignaturePlacement placement) {
       final base = _getBaseImage(placement);
       final key =
@@ -94,6 +94,20 @@ class ExportService {
       return png;
     }
 
+    // Wrap bytes in a pw.MemoryImage with caching.
+    pw.MemoryImage? _getMemoryImage(Uint8List bytes, String key) {
+      final cached = _memoryImageCache[key];
+      if (cached != null) return cached;
+      try {
+        final imgObj = pw.MemoryImage(bytes);
+        _memoryImageCache[key] = imgObj;
+        return imgObj;
+      } catch (_) {
+        return null;
+      }
+    }
+
+    // Compute and cache aspect ratio (width/height) for given image
     double? _getAspectRatioFromImage(img.Image image) {
       final key = _baseKeyForImage(image);
       final c = _aspectRatioCache[key];
@@ -104,56 +118,119 @@ class ExportService {
       return ar;
     }
 
-    // Initialize engine (safe to call multiple times)
-    pdfrxFlutterInitialize();
-
-    // Open source document from memory; if not supported, write temp file
-    engine.PdfDocument? doc;
-    try {
-      doc = await engine.PdfDocument.openData(srcBytes);
-    } catch (_) {
-      debugPrint('Warning: pdfrx openData failed');
-      final tmp = File(
-        '${Directory.systemTemp.path}/pdfrx_src_${DateTime.now().millisecondsSinceEpoch}.pdf',
-      );
-      await tmp.writeAsBytes(srcBytes, flush: true);
-      doc = await engine.PdfDocument.openFile(tmp.path);
-      try {
-        tmp.deleteSync();
-      } catch (_) {
-        debugPrint('Warning: temp file delete failed');
-      }
-    }
-    // doc is guaranteed to be assigned by either openData or openFile above
-
     final out = pw.Document(version: pdf.PdfVersion.pdf_1_4, compress: false);
-    final pages = doc.pages;
+    int pageIndex = 0;
-    final scale = targetDpi / 72.0;
+    bool anyPage = false;
-    for (int i = 0; i < pages.length; i++) {
+    try {
-      // Cooperative yield between pages so the UI can animate the spinner.
+      await for (final raster in printing.Printing.raster(
-      await Future<void>.delayed(Duration.zero);
+        srcBytes,
-      final page = pages[i];
+        dpi: targetDpi,
-      final pageIndex = i + 1;
+      )) {
-      final widthPts = page.width;
+        anyPage = true;
-      final heightPts = page.height;
+        pageIndex++;
+        final widthPx = raster.width;
+        final heightPx = raster.height;
+        final widthPts = widthPx * 72.0 / targetDpi;
+        final heightPts = heightPx * 72.0 / targetDpi;
 
-      // Render background image via engine
+        final bgPng = await raster.toPng();
-      final imgPage = await page.render(
+        final bgImg = pw.MemoryImage(bgPng);
-        fullWidth: widthPts * scale,
-        fullHeight: heightPts * scale,
-      );
-      if (imgPage == null) continue;
-      final bgImage = imgPage.createImageNF();
-      imgPage.dispose();
-      // Lower compression for background snapshot too.
-      final bgPng = Uint8List.fromList(img.encodePng(bgImage, level: 1));
-      final _ = Future<void>.delayed(Duration.zero);
-      final bgMem = pw.MemoryImage(bgPng);
 
+        final hasMulti =
+            (placementsByPage != null && placementsByPage.isNotEmpty);
+        final pagePlacements =
+            hasMulti
+                ? (placementsByPage[pageIndex] ?? const <SignaturePlacement>[])
+                : const <SignaturePlacement>[];
+
+        out.addPage(
+          pw.Page(
+            pageTheme: pw.PageTheme(
+              margin: pw.EdgeInsets.zero,
+              pageFormat: pdf.PdfPageFormat(widthPts, heightPts),
+            ),
+            build: (ctx) {
+              final children = <pw.Widget>[
+                pw.Positioned(
+                  left: 0,
+                  top: 0,
+                  child: pw.Image(
+                    bgImg,
+                    width: widthPts,
+                    height: heightPts,
+                    fit: pw.BoxFit.fill,
+                  ),
+                ),
+              ];
+              // Multi-placement stamping: per-placement image from libraryBytes
+              if (hasMulti && pagePlacements.isNotEmpty) {
+                for (var i = 0; i < pagePlacements.length; i++) {
+                  final placement = pagePlacements[i];
+                  final r = placement.rect;
+                  // rect is stored in normalized units (0..1) relative to page
+                  final left = r.left * widthPts;
+                  final top = r.top * heightPts;
+                  final w = r.width * widthPts;
+                  final h = r.height * heightPts;
+
+                  // Get processed image and embed as MemoryImage (cached)
+                  final processedPng = _getProcessedPng(placement);
+                  final baseImage = _getBaseImage(placement);
+                  final memKey =
+                      '${_baseKeyForImage(baseImage)}|${_adjustKey(placement.graphicAdjust)}';
+                  if (processedPng.isNotEmpty) {
+                    final imgObj = _getMemoryImage(processedPng, memKey);
+                    if (imgObj != null) {
+                      // Align with RotatedSignatureImage: counterclockwise positive
+                      final angle = rot.radians(placement.rotationDeg);
+                      // Use AR from base image
+                      final ar = _getAspectRatioFromImage(baseImage);
+                      final scaleToFit = rot.scaleToFitForAngle(angle, ar: ar);
+
+                      children.add(
+                        pw.Positioned(
+                          left: left,
+                          top: top,
+                          child: pw.SizedBox(
+                            width: w,
+                            height: h,
+                            child: pw.FittedBox(
+                              fit: pw.BoxFit.contain,
+                              child: pw.Transform.scale(
+                                scale: scaleToFit,
+                                child: pw.Transform.rotate(
+                                  angle: angle,
+                                  child: pw.Image(imgObj),
+                                ),
+                              ),
+                            ),
+                          ),
+                        ),
+                      );
+                    }
+                  }
+                }
+              }
+              return pw.Stack(children: children);
+            },
+          ),
+        );
+      }
+    } catch (e) {
+      anyPage = false;
+    }
+
+    if (!anyPage) {
+      // Fallback as A4 blank page with optional signature
+      final widthPts = pdf.PdfPageFormat.a4.width;
+      final heightPts = pdf.PdfPageFormat.a4.height;
+
+      final hasMulti =
+          (placementsByPage != null && placementsByPage.isNotEmpty);
       final pagePlacements =
-          (placementsByPage ??
+          hasMulti
-              const <int, List<SignaturePlacement>>{})[pageIndex] ??
+              ? (placementsByPage[1] ?? const <SignaturePlacement>[])
-          const <SignaturePlacement>[];
+              : const <SignaturePlacement>[];
 
       out.addPage(
         pw.Page(
@@ -163,70 +240,72 @@ class ExportService {
           ),
           build: (ctx) {
             final children = <pw.Widget>[
-              pw.Positioned(
+              pw.Container(
-                left: 0,
+                width: widthPts,
-                top: 0,
+                height: heightPts,
-                child: pw.Image(
+                color: pdf.PdfColors.white,
-                  bgMem,
-                  width: widthPts,
-                  height: heightPts,
-                  fit: pw.BoxFit.fill,
-                ),
               ),
             ];
 
-            for (final placement in pagePlacements) {
+            if (hasMulti && pagePlacements.isNotEmpty) {
-              final r = placement.rect;
+              for (var i = 0; i < pagePlacements.length; i++) {
-              final left = r.left * widthPts;
+                final placement = pagePlacements[i];
-              final top = r.top * heightPts;
+                final r = placement.rect;
-              final w = r.width * widthPts;
+                // rect is stored in normalized units (0..1) relative to page
-              final h = r.height * heightPts;
+                final left = r.left * widthPts;
+                final top = r.top * heightPts;
+                final w = r.width * widthPts;
+                final h = r.height * heightPts;
 
-              final processedPng = _getProcessedPng(placement);
+                final processedPng = _getProcessedPng(placement);
-              if (processedPng.isEmpty) continue;
+                final baseImage = _getBaseImage(placement);
-              final memImg = pw.MemoryImage(processedPng);
+                final memKey =
-              final angle = rot.radians(placement.rotationDeg);
+                    '${_baseKeyForImage(baseImage)}|${_adjustKey(placement.graphicAdjust)}';
-              final baseImage = _getBaseImage(placement);
+                if (processedPng.isNotEmpty) {
-              final ar = _getAspectRatioFromImage(baseImage);
+                  final imgObj = _getMemoryImage(processedPng, memKey);
-              final scaleToFit = rot.scaleToFitForAngle(angle, ar: ar);
+                  if (imgObj != null) {
+                    final angle = rot.radians(placement.rotationDeg);
+                    final ar = _getAspectRatioFromImage(baseImage);
+                    final scaleToFit = rot.scaleToFitForAngle(angle, ar: ar);
 
-              children.add(
+                    children.add(
-                pw.Positioned(
+                      pw.Positioned(
-                  left: left,
+                        left: left,
-                  top: top,
+                        top: top,
-                  child: pw.SizedBox(
+                        child: pw.SizedBox(
-                    width: w,
+                          width: w,
-                    height: h,
+                          height: h,
-                    child: pw.FittedBox(
+                          child: pw.FittedBox(
-                      fit: pw.BoxFit.contain,
+                            fit: pw.BoxFit.contain,
-                      child: pw.Transform.scale(
+                            child: pw.Transform.scale(
-                        scale: scaleToFit,
+                              scale: scaleToFit,
-                        child: pw.Transform.rotate(
+                              child: pw.Transform.rotate(
-                          angle: angle,
+                                angle: angle,
-                          child: pw.Image(memImg),
+                                child: pw.Image(imgObj),
+                              ),
+                            ),
+                          ),
                         ),
                       ),
-                    ),
+                    );
-                  ),
+                  }
-                ),
+                }
-              );
+              }
-              // Yield occasionally within large placement lists to keep UI responsive.
-              // ignore: unused_local_variable
-              final _ = Future<void>.delayed(Duration.zero);
             }
             return pw.Stack(children: children);
           },
         ),
       );
-      final _ = Future<void>.delayed(Duration.zero);
     }
 
-    final bytes = await out.save();
+    try {
-    doc.dispose();
+      return await out.save();
-    debugPrint('exportSignedPdfFromBytes succeeded');
+    } catch (_) {
-    return bytes;
+      return null;
+    }
   }
 
+  /// Helper: write bytes returned from [exportSignedPdfFromBytes] to a file path.
   Future<bool> saveBytesToFile({
     required Uint8List bytes,
     required String outputPath,
@@ -236,8 +315,9 @@ class ExportService {
       await file.writeAsBytes(bytes, flush: true);
       return true;
     } catch (_) {
-      debugPrint('Error: saveBytesToFile failed');
       return false;
     }
   }
+
+  // Background removal implemented in utils/background_removal.dart
 }

lib/main.dart

@@ -1,22 +1,13 @@
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
-import 'package:pdfrx/pdfrx.dart';
 import 'package:pdf_signature/app.dart';
-import 'package:pdf_signature/utils/pdfrx_cache_init/pdfrx_cache_init.dart';
 export 'package:pdf_signature/app.dart';
 
-Future<void> main() async {
+void main() {
+  // Ensure Flutter bindings are initialized before platform channel usage
   WidgetsFlutterBinding.ensureInitialized();
-  // Initialize pdfrx core (safe to call multiple times) and set up cache directory.
-  pdfrxFlutterInitialize();
-  await initPdfrxCache();
   // Disable right-click context menu on web using Flutter API
-  if (kReleaseMode) {
-    debugPrint = (String? message, {int? wrapWidth}) {
-      // Empty implementation in release mode, effectively disabling debugPrint
-    };
-  }
   if (kIsWeb) {
     BrowserContextMenu.disableContextMenu();
   }

lib/routing/router.dart

@@ -33,7 +33,7 @@ final routerProvider = Provider<GoRouter>((ref) {
                 ({String? path, Uint8List? bytes, String? fileName}) =>
                     sessionVm.openPdf(
                       path: path,
-                      bytes: bytes ?? Uint8List(0),
+                      bytes: bytes,
                       fileName: fileName,
                     ),
           );

lib/ui/features/pdf/view_model/document_version.dart

@@ -1,31 +0,0 @@
-import 'dart:typed_data';
-import 'package:freezed_annotation/freezed_annotation.dart';
-
-part 'document_version.freezed.dart';
-
-/// Internal data model for tracking document versions in the UI layer.
-/// This is separate from the domain Document model to avoid coupling UI concerns with business logic.
-@freezed
-abstract class DocumentVersion with _$DocumentVersion {
-  const factory DocumentVersion({
-    @Default(0) int version,
-    Uint8List? lastBytes,
-  }) = _DocumentVersion;
-
-  factory DocumentVersion.initial() => const DocumentVersion();
-}
-
-extension DocumentVersionMethods on DocumentVersion {
-  /// Generate the source name for PdfDocumentRefData based on version
-  String get sourceName => 'document_v$version.pdf';
-
-  /// Check if bytes have changed and need version increment
-  bool shouldIncrementVersion(Uint8List? newBytes) {
-    return !identical(lastBytes, newBytes);
-  }
-
-  /// Increment version and update bytes
-  DocumentVersion incrementVersion(Uint8List? newBytes) {
-    return copyWith(version: version + 1, lastBytes: newBytes);
-  }
-}

lib/ui/features/pdf/view_model/pdf_export_view_model.dart

@@ -1,10 +1,7 @@
-import 'dart:io' show Platform;
+import 'package:file_selector/file_selector.dart' as fs;
-import 'package:file_picker/file_picker.dart' as fp;
-import 'package:path_provider/path_provider.dart' as pp;
 import 'package:flutter/foundation.dart';
-import 'package:flutter/material.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
-import 'package:pdf_signature/data/repositories/document_repository.dart';
+import 'package:pdf_signature/data/services/export_service.dart';
 
 /// ViewModel for export-related UI state and helpers.
 class PdfExportViewModel extends ChangeNotifier {
@@ -12,6 +9,7 @@ class PdfExportViewModel extends ChangeNotifier {
   bool _exporting = false;
 
   // Dependencies (injectable via constructor for tests)
+  final ExportService _exporter;
   // Zero-arg picker retained for backward compatibility with tests.
   final Future<String?> Function() _savePathPicker;
   // Preferred picker that accepts a suggested filename.
@@ -20,10 +18,12 @@ class PdfExportViewModel extends ChangeNotifier {
 
   PdfExportViewModel(
     this.ref, {
+    ExportService? exporter,
     Future<String?> Function()? savePathPicker,
     Future<String?> Function(String suggestedName)?
     savePathPickerWithSuggestedName,
-  }) : _savePathPicker = savePathPicker ?? _defaultSavePathPicker,
+  }) : _exporter = exporter ?? ExportService(),
+       _savePathPicker = savePathPicker ?? _defaultSavePathPicker,
        // Prefer provided suggested-name picker; otherwise, if only zero-arg
        // picker is given (tests), wrap it; else use default that honors name.
        _savePathPickerWithSuggestedName =
@@ -40,22 +40,8 @@ class PdfExportViewModel extends ChangeNotifier {
     notifyListeners();
   }
 
-  /// Perform export via document repository. Returns true on success.
+  /// Get the export service (overridable in tests via constructor).
-  Future<bool> exportToPath({
+  ExportService get exporter => _exporter;
-    required String outputPath,
-    required Size uiPageSize,
-    required Uint8List? signatureImageBytes,
-    double targetDpi = 144.0,
-  }) async {
-    return await ref
-        .read(documentRepositoryProvider.notifier)
-        .exportDocument(
-          outputPath: outputPath,
-          uiPageSize: uiPageSize,
-          signatureImageBytes: signatureImageBytes,
-          targetDpi: targetDpi,
-        );
-  }
 
   /// Show save dialog and return the chosen path (null if canceled).
   Future<String?> pickSavePath() async {
@@ -74,51 +60,13 @@ class PdfExportViewModel extends ChangeNotifier {
   static Future<String?> _defaultSavePathPickerWithSuggestedName(
     String suggestedName,
   ) async {
-    // Prefer native save dialog via file_picker on all non-web platforms.
+    final group = fs.XTypeGroup(label: 'PDF', extensions: ['pdf']);
-    // If the user cancels (null) simply bubble up null. If an exception occurs
+    final location = await fs.getSaveLocation(
-    // (unsupported platform or plugin issue), fall back to an app documents path.
+      acceptedTypeGroups: [group],
-    try {
+      suggestedName: suggestedName,
-      final result = await fp.FilePicker.platform.saveFile(
+      confirmButtonText: 'Save',
-        dialogTitle: 'Please select an output file:',
-        fileName: suggestedName,
-        type: fp.FileType.custom,
-        allowedExtensions: const ['pdf'],
-        // lockParentWindow is ignored on mobile; useful on desktop.
-        lockParentWindow: true,
-      );
-      return result; // null if canceled
-    } catch (_) {
-      // Fall through to app documents fallback below.
-    }
-
-    debugPrint(
-      'Fallback: select a folder and build path with suggested name (mobile platform)',
     );
-
+    return location?.path; // null if user cancels
-    // On some mobile providers, saveFile may not present a picker or returns null.
-    // Offer a folder picker and compose the final path.
-    if (Platform.isAndroid || Platform.isIOS) {
-      final dir = await fp.FilePicker.platform.getDirectoryPath(
-        dialogTitle: 'Select folder to save',
-        lockParentWindow: true,
-      );
-      if (dir != null && dir.trim().isNotEmpty) {
-        final d = dir.trim();
-        final needsSep = !(d.endsWith('/') || d.endsWith('\\'));
-        return (needsSep ? (d + '/') : d) + suggestedName;
-      }
-      // User canceled directory selection; bubble up null.
-      return null;
-    }
-
-    debugPrint('Fallback: build a default path (web platform)');
-    try {
-      final dir = await pp.getApplicationDocumentsDirectory();
-      return '${dir.path}/$suggestedName';
-    } catch (_) {
-      // Last resort: let the caller handle a null path
-      return null;
-    }
   }
 }
 

lib/ui/features/pdf/view_model/pdf_view_model.dart

@@ -1,17 +1,13 @@
-// ignore_for_file: unnecessary_import
 import 'dart:typed_data';
-import 'package:cross_file/cross_file.dart';
+import 'package:file_selector/file_selector.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:pdf_signature/data/repositories/document_repository.dart';
 import 'package:pdf_signature/data/repositories/signature_card_repository.dart';
 import 'package:pdf_signature/domain/models/model.dart';
-import 'package:pdf_signature/ui/features/pdf/view_model/document_version.dart';
 import 'package:pdfrx/pdfrx.dart';
-import 'package:file_picker/file_picker.dart' as fp;
+import 'package:file_selector/file_selector.dart' as fs;
 import 'package:go_router/go_router.dart';
-import 'package:path_provider/path_provider.dart';
-import 'package:flutter/foundation.dart';
 
 class PdfViewModel extends ChangeNotifier {
   final Ref ref;
@@ -35,26 +31,6 @@ class PdfViewModel extends ChangeNotifier {
   final Set<String> _lockedPlacements = {};
   Set<String> get lockedPlacements => Set.unmodifiable(_lockedPlacements);
 
-  // Document version tracking for UI consistency
-  DocumentVersion _documentVersion = DocumentVersion.initial();
-
-  // Get current document source name for PdfDocumentRefData
-  String get documentSourceName {
-    // Ensure document version is up to date, but only update if really needed
-    _updateDocumentVersionIfNeeded();
-    return _documentVersion.sourceName;
-  }
-
-  void _updateDocumentVersionIfNeeded() {
-    final document = ref.read(documentRepositoryProvider);
-    if (!identical(_documentVersion.lastBytes, document.pickedPdfBytes)) {
-      _documentVersion = DocumentVersion(
-        version: _documentVersion.version + 1,
-        lastBytes: document.pickedPdfBytes,
-      );
-    }
-  }
-
   // const bool.fromEnvironment('FLUTTER_TEST', defaultValue: false);
   PdfViewModel(this.ref, {bool? useMockViewer})
     : _useMockViewer =
@@ -67,8 +43,6 @@ class PdfViewModel extends ChangeNotifier {
 
   set currentPage(int value) {
     _currentPage = value.clamp(1, document.pageCount);
-    // ignore: avoid_print
-    debugPrint('PdfViewModel.currentPage set to $_currentPage');
     if (!_isDisposed) {
       notifyListeners();
     }
@@ -80,8 +54,6 @@ class PdfViewModel extends ChangeNotifier {
   Document get document => ref.read(documentRepositoryProvider);
 
   void jumpToPage(int page) {
-    // ignore: avoid_print
-    debugPrint('PdfViewModel.jumpToPage ' + page.toString());
     currentPage = page;
   }
 
@@ -138,7 +110,11 @@ class PdfViewModel extends ChangeNotifier {
   }) {
     ref
         .read(documentRepositoryProvider.notifier)
-        .modifyPlacement(page: page, index: index, rotationDeg: rotationDeg);
+        .updatePlacementRotation(
+          page: page,
+          index: index,
+          rotationDeg: rotationDeg,
+        );
   }
 
   void removePlacement({required int page, required int index}) {
@@ -159,7 +135,7 @@ class PdfViewModel extends ChangeNotifier {
   }) {
     ref
         .read(documentRepositoryProvider.notifier)
-        .modifyPlacement(page: page, index: index, rect: rect);
+        .updatePlacementRect(page: page, index: index, rect: rect);
   }
 
   List<SignaturePlacement> placementsOn(int page) {
@@ -267,7 +243,7 @@ final pdfViewModelProvider = ChangeNotifierProvider<PdfViewModel>((ref) {
 class PdfSessionViewModel extends ChangeNotifier {
   final Ref ref;
   final GoRouter router;
-  XFile _currentFile = XFile('');
+  fs.XFile _currentFile = fs.XFile('');
   // Keep a human display name in addition to XFile, because on Linux via
   // xdg-desktop-portal the path can look like /run/user/.../doc/<UUID>, and
   // XFile.name derives from that basename, yielding a random UUID instead of
@@ -277,103 +253,52 @@ class PdfSessionViewModel extends ChangeNotifier {
 
   PdfSessionViewModel({required this.ref, required this.router});
 
-  XFile get currentFile => _currentFile;
+  fs.XFile get currentFile => _currentFile;
   String get displayFileName => _displayFileName;
 
   Future<void> pickAndOpenPdf() async {
-    final result = await fp.FilePicker.platform.pickFiles(
+    final typeGroup = const fs.XTypeGroup(label: 'PDF', extensions: ['pdf']);
-      type: fp.FileType.custom,
+    final XFile? file = await fs.openFile(acceptedTypeGroups: [typeGroup]);
-      allowedExtensions: const ['pdf'],
+    if (file != null) {
-      withData: true,
+      Uint8List? bytes;
-    );
-    if (result == null || result.files.isEmpty) return;
-    final picked = result.files.single;
-    final String name = picked.name;
-    final String? path = picked.path;
-    final Uint8List? bytes = picked.bytes;
-    Uint8List? effectiveBytes = bytes;
-    if (effectiveBytes == null && path != null && path.isNotEmpty) {
       try {
-        effectiveBytes = await XFile(path).readAsBytes();
+        bytes = await file.readAsBytes();
-      } catch (e, st) {
+      } catch (_) {
-        effectiveBytes = null;
+        bytes = null;
-        debugPrint(
-          '[PdfSessionViewModel] Failed to read PDF data from path=$path error=$e',
-        );
-        debugPrint(st.toString());
       }
-    }
+      await openPdf(path: file.path, bytes: bytes, fileName: file.name);
-    if (effectiveBytes != null) {
-      await openPdf(path: path, bytes: effectiveBytes, fileName: name);
-    } else {
-      debugPrint(
-        '[PdfSessionViewModel] No PDF data available to open (path=$path, name=$name)',
-      );
     }
   }
 
   Future<void> openPdf({
     String? path,
-    required Uint8List bytes,
+    Uint8List? bytes,
     String? fileName,
   }) async {
     int pageCount = 1; // default
-    try {
+    if (bytes != null) {
-      // Defensive: ensure Pdfrx cache directory set (in case main init skipped in tests)
+      try {
-      if (Pdfrx.getCacheDirectory == null && !kIsWeb) {
+        final doc = await PdfDocument.openData(bytes);
-        debugPrint('[PdfSessionViewModel] Setting Pdfrx cache directory (io)');
+        pageCount = doc.pages.length;
-        try {
+      } catch (_) {
-          final temp = await getTemporaryDirectory();
+        // ignore invalid bytes
-          Pdfrx.getCacheDirectory = () async => temp.path;
-        } catch (e, st) {
-          debugPrint(
-            '[PdfSessionViewModel] Failed to set fallback cache dir error=$e',
-          );
-          debugPrint(st.toString());
-        }
       }
-      // Ensure engine initialized (safe multiple calls)
-      pdfrxFlutterInitialize();
-      final doc = await PdfDocument.openData(bytes);
-      pageCount = doc.pages.length;
-      debugPrint(
-        '[PdfSessionViewModel] Opened PDF bytes length=${bytes.length} pages=$pageCount',
-      );
-      // Use fast path to populate repository BEFORE navigation so the first
-      // build of PdfViewerWidget sees a loaded document and avoids showing
-      // transient "No PDF loaded".
-      ref
-          .read(documentRepositoryProvider.notifier)
-          .openDocument(
-            bytes: bytes,
-            pageCount: pageCount,
-            knownPageCount: true,
-          );
-    } catch (e, st) {
-      debugPrint(
-        '[PdfSessionViewModel] Failed to read PDF data from bytes error=$e',
-      );
-      debugPrint(st.toString());
     }
     if (path != null && path.isNotEmpty) {
-      _currentFile = XFile(path);
+      _currentFile = fs.XFile(path);
-    } else if ((fileName != null && fileName.isNotEmpty)) {
+    } else if (bytes != null && (fileName != null && fileName.isNotEmpty)) {
       // Keep in-memory XFile so .name is available for suggestion
       try {
-        _currentFile = XFile.fromData(
+        _currentFile = fs.XFile.fromData(
           bytes,
           name: fileName,
           mimeType: 'application/pdf',
         );
-      } catch (e, st) {
+      } catch (_) {
-        _currentFile = XFile(fileName);
+        _currentFile = fs.XFile(fileName);
-        debugPrint(
-          '[PdfSessionViewModel] Failed to create XFile.fromData name=$fileName error=$e',
-        );
-        debugPrint(st.toString());
       }
     } else {
-      _currentFile = XFile('');
+      _currentFile = fs.XFile('');
     }
 
     // Update display name: prefer explicit fileName (from picker/drop),
@@ -385,26 +310,18 @@ class PdfSessionViewModel extends ChangeNotifier {
     } else {
       _displayFileName = '';
     }
-    // If fast path failed to set repository (e.g., exception earlier), fallback to async derive.
+    ref
-    if (ref.read(documentRepositoryProvider).pickedPdfBytes != bytes) {
+        .read(documentRepositoryProvider.notifier)
-      debugPrint(
+        .openPicked(pageCount: pageCount, bytes: bytes);
-        '[PdfSessionViewModel] Fallback deriving page count via openDocument',
+    ref.read(signatureCardRepositoryProvider.notifier).clearAll();
-      );
-      ref.read(documentRepositoryProvider.notifier).openDocument(bytes: bytes);
-    }
-    // Keep existing signature cards when opening a new document.
-    // The feature "Open a different document will reset signature placements but keep signature cards"
-    // relies on this behavior. Placements are reset by openPicked() above.
-    debugPrint('[PdfSessionViewModel] Navigating to /pdf');
     router.go('/pdf');
-    debugPrint('[PdfSessionViewModel] Notifying listeners after open');
     notifyListeners();
   }
 
   void closePdf() {
     ref.read(documentRepositoryProvider.notifier).close();
     ref.read(signatureCardRepositoryProvider.notifier).clearAll();
-    _currentFile = XFile('');
+    _currentFile = fs.XFile('');
     _displayFileName = '';
     router.go('/');
     notifyListeners();

lib/ui/features/pdf/widgets/pages_sidebar.dart

@@ -40,29 +40,11 @@ class ThumbnailsView extends ConsumerWidget {
                   // to update provider when the page is actually reached.
                   // For mock/unready: update provider immediately to drive scroll.
                   final isRealViewer = !viewModel.useMockViewer;
-                  // Debug trace for navigation taps
-                  // ignore: avoid_print
-                  debugPrint(
-                    'PagesSidebar.onTap page=$pageNumber isRealViewer=$isRealViewer controllerReady=${controller.isReady}',
-                  );
                   if (isRealViewer && controller.isReady) {
-                    try {
+                    controller.goToPage(
-                      controller.goToPage(
+                      pageNumber: pageNumber,
-                        pageNumber: pageNumber,
+                      anchor: PdfPageAnchor.top,
-                        anchor: PdfPageAnchor.top,
+                    );
-                      );
-                      // ignore: avoid_print
-                      debugPrint(
-                        'controller.goToPage invoked for page=$pageNumber',
-                      );
-                    } catch (e, st) {
-                      // ignore: avoid_print
-                      debugPrint(
-                        '[ERR] controller.goToPage exception: ' + e.toString(),
-                      );
-                      // ignore: avoid_print
-                      debugPrint(st.toString());
-                    }
                     // Do not set provider here; let onPageChanged handle it
                   } else {
                     // In tests or when controller isn't ready, drive state directly
@@ -70,8 +52,6 @@ class ThumbnailsView extends ConsumerWidget {
                       ref
                           .read(pdfViewModelProvider.notifier)
                           .jumpToPage(pageNumber);
-                      // ignore: avoid_print
-                      debugPrint('jumpToPage set directly to $pageNumber');
                     } catch (_) {}
                   }
                 },
@@ -93,18 +73,14 @@ class ThumbnailsView extends ConsumerWidget {
                     padding: const EdgeInsets.all(6),
                     child: Column(
                       children: [
-                        ConstrainedBox(
+                        SizedBox(
-                          constraints: const BoxConstraints(maxHeight: 180),
+                          height: 180,
-                          child: AspectRatio(
+                          child: ClipRRect(
-                            // A4 portrait aspect: width:height ≈ 1:1.4142
+                            borderRadius: BorderRadius.circular(4),
-                            aspectRatio: 1 / 1.4142,
+                            child: PdfPageView(
-                            child: ClipRRect(
+                              document: document,
-                              borderRadius: BorderRadius.circular(4),
+                              pageNumber: pageNumber,
-                              child: PdfPageView(
+                              alignment: Alignment.center,
-                                document: document,
-                                pageNumber: pageNumber,
-                                alignment: Alignment.center,
-                              ),
                             ),
                           ),
                         ),

lib/ui/features/pdf/widgets/pdf_page_area.dart

@@ -6,19 +6,16 @@ import 'package:pdf_signature/l10n/app_localizations.dart';
 import 'pdf_viewer_widget.dart';
 import 'package:pdfrx/pdfrx.dart';
 import '../view_model/pdf_view_model.dart';
-import '../view_model/pdf_export_view_model.dart';
 
 class PdfPageArea extends ConsumerStatefulWidget {
   const PdfPageArea({
     super.key,
     required this.pageSize,
     required this.controller,
-    this.onDocumentChanged,
   });
 
   final Size pageSize;
   final PdfViewerController controller;
-  final void Function(PdfDocument?)? onDocumentChanged;
   @override
   ConsumerState<PdfPageArea> createState() => _PdfPageAreaState();
 }
@@ -41,8 +38,10 @@ class _PdfPageAreaState extends ConsumerState<PdfPageArea> {
     super.initState();
     // If app starts in continuous mode with a loaded PDF, ensure the viewer
     // is instructed to align to the provider's current page once ready.
-    // Do not schedule mock scroll sync in real viewer mode.
+    WidgetsBinding.instance.addPostFrameCallback((_) {
-    // In mock mode, scrolling is driven on demand when currentPage changes.
+      if (!mounted) return;
+      // initial scroll not needed; controller handles positioning
+    });
   }
 
   // No dispose required for PdfViewerController (managed by owner if any)
@@ -55,9 +54,6 @@ class _PdfPageAreaState extends ConsumerState<PdfPageArea> {
   void _scrollToPage(int page) {
     WidgetsBinding.instance.addPostFrameCallback((_) {
       if (!mounted) return;
-      // Only valid in mock viewer mode; skip otherwise
-      final useMock = ref.read(pdfViewModelProvider).useMockViewer;
-      if (!useMock) return;
       _programmaticTargetPage = page;
       // Mock continuous: try ensureVisible on the page container
       // Mock continuous: try ensureVisible on the page container
@@ -118,13 +114,6 @@ class _PdfPageAreaState extends ConsumerState<PdfPageArea> {
     // React to PdfViewModel currentPage changes. With ChangeNotifierProvider,
     // prev/next are the same instance, so compare to a local cache.
     ref.listen(pdfViewModelProvider, (prev, next) {
-      // Only perform manual scrolling in mock viewer mode. In real viewer mode,
-      // PdfViewerController + onPageChanged keep things in sync, and attempting
-      // to scroll here (without mock page keys) creates repeated frame
-      // callbacks that never find targets, leading to hangs.
-      if (!next.useMockViewer) {
-        return;
-      }
       if (_suppressProviderListen) return;
       final target = next.currentPage;
       if (_lastListenedPage == target) return;
@@ -154,20 +143,11 @@ class _PdfPageAreaState extends ConsumerState<PdfPageArea> {
 
     // Use real PDF viewer
     if (isContinuous) {
-      // While exporting, fully detach the viewer to avoid background activity
-      // and ensure a clean re-initialization afterward.
-      final exporting = ref.watch(pdfExportViewModelProvider).exporting;
-      if (exporting) {
-        return const SizedBox.expand(key: Key('exporting_viewer_placeholder'));
-      }
       return PdfViewerWidget(
         pageSize: widget.pageSize,
         pageKeyBuilder: _pageKey,
         scrollToPage: _scrollToPage,
         controller: widget.controller,
-        // Remove fixed innerViewerKey to allow PdfViewerWidget to generate dynamic keys
-        // innerViewerKey: const ValueKey('viewer_idle'),
-        onDocumentChanged: widget.onDocumentChanged,
       );
     }
     return const SizedBox.shrink();

lib/ui/features/pdf/widgets/pdf_page_overlays.dart

@@ -7,7 +7,6 @@ import '../../../../domain/models/model.dart';
 import 'signature_overlay.dart';
 import '../../signature/widgets/signature_drag_data.dart';
 import '../../signature/view_model/dragging_signature_view_model.dart';
-import 'pdf_viewer_widget.dart' show viewerOverlaysEnabledProvider;
 
 /// Builds all overlays for a given page: placed signatures and the active one.
 class PdfPageOverlays extends ConsumerWidget {
@@ -32,10 +31,6 @@ class PdfPageOverlays extends ConsumerWidget {
 
   @override
   Widget build(BuildContext context, WidgetRef ref) {
-    final overlaysEnabled = ref.watch(viewerOverlaysEnabledProvider);
-    if (!overlaysEnabled) {
-      return const SizedBox.shrink();
-    }
     final pdfViewModel = ref.watch(pdfViewModelProvider);
     // Subscribe to document changes to rebuild overlays
     final pdf = ref.watch(documentRepositoryProvider);

lib/ui/features/pdf/widgets/pdf_screen.dart

@@ -1,5 +1,4 @@
-import 'package:cross_file/cross_file.dart';
+import 'package:file_selector/file_selector.dart' as fs;
-import 'package:file_picker/file_picker.dart' as fp;
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
@@ -17,20 +16,16 @@ import '../view_model/pdf_export_view_model.dart';
 import 'package:pdf_signature/utils/download.dart';
 import '../view_model/pdf_view_model.dart';
 import 'package:image/image.dart' as img;
-import 'package:pdf_signature/data/repositories/document_repository.dart';
-import 'package:responsive_framework/responsive_framework.dart';
 
 class PdfSignatureHomePage extends ConsumerStatefulWidget {
   final Future<void> Function() onPickPdf;
   final VoidCallback onClosePdf;
-  final XFile currentFile;
+  final fs.XFile currentFile;
   // Optional display name for the currently opened file. On Linux
   // xdg-desktop-portal, XFile.name/path can be a UUID-like value. When
   // available, this name preserves the user-selected filename so we can
   // suggest a proper "signed_*.pdf" on save.
   final String? currentFileName;
-  // Optional listener for underlying Pdfrx document change events.
-  final void Function(PdfDocument?)? onDocumentChanged;
 
   const PdfSignatureHomePage({
     super.key,
@@ -38,7 +33,6 @@ class PdfSignatureHomePage extends ConsumerStatefulWidget {
     required this.onClosePdf,
     required this.currentFile,
     this.currentFileName,
-    this.onDocumentChanged,
   });
 
   @override
@@ -63,7 +57,6 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
   final double _signaturesMin = 140;
   final double _signaturesMax = 250;
   late PdfViewModel _viewModel;
-  bool? _lastCanShowPagesSidebar;
 
   // Exposed for tests to trigger the invalid-file SnackBar without UI.
   @visibleForTesting
@@ -117,18 +110,15 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
   }
 
   Future<img.Image?> _loadSignatureFromFile() async {
-    final result = await fp.FilePicker.platform.pickFiles(
+    final typeGroup = fs.XTypeGroup(
-      type: fp.FileType.custom,
+      label:
-      allowedExtensions: const ['png', 'jpg', 'jpeg', 'webp'],
+          Localizations.of<AppLocalizations>(context, AppLocalizations)?.image,
-      withData: true,
+      extensions: ['png', 'jpg', 'jpeg', 'webp'],
     );
-    if (result == null || result.files.isEmpty) return null;
+    final file = await fs.openFile(acceptedTypeGroups: [typeGroup]);
-    final picked = result.files.single;
+    if (file == null) return null;
-    final Uint8List? bytes =
+    final bytes = await file.readAsBytes();
-        picked.bytes ??
-        (picked.path != null ? await XFile(picked.path!).readAsBytes() : null);
     try {
-      if (bytes == null) return null;
       var sigImage = img.decodeImage(bytes);
       return _toStdSignatureImage(sigImage);
     } catch (_) {
@@ -154,114 +144,105 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
   }
 
   Future<void> _saveSignedPdf() async {
-    // Show exporting overlay and then run the heavy work asynchronously so
-    // the UI thread remains responsive to gestures like page navigation.
     ref.read(pdfExportViewModelProvider.notifier).setExporting(true);
-    // ignore: avoid_print
+    try {
-    debugPrint('_saveSignedPdf: exporting flag set true');
+      final pdf = _viewModel.document;
-    final weakContext = context;
+      final messenger = ScaffoldMessenger.of(context);
-    Future<void>(() async {
+      if (!pdf.loaded) {
-      try {
+        messenger.showSnackBar(
-        // ignore: avoid_print
+          SnackBar(
-        debugPrint('_saveSignedPdf: async export task started');
+            content: Text(AppLocalizations.of(context).nothingToSaveYet),
-        final pdf = _viewModel.document;
+          ),
-        final messenger = ScaffoldMessenger.of(weakContext);
+        );
-        if (!pdf.loaded) {
+        return;
-          // ignore: avoid_print
+      }
-          debugPrint('_saveSignedPdf: document not loaded');
+      final exporter = ref.read(pdfExportViewModelProvider).exporter;
-          messenger.showSnackBar(
-            SnackBar(
-              content: Text(AppLocalizations.of(weakContext).nothingToSaveYet),
-            ),
-          );
-          return;
-        }
-        // get DPI from preferences
-        final targetDpi = ref.read(preferencesRepositoryProvider).exportDpi;
-        bool ok = false;
-        String? savedPath;
 
-        // Derive a suggested filename based on the opened file.
+      // get DPI from preferences
-        final display = widget.currentFileName;
+      final targetDpi = ref.read(preferencesRepositoryProvider).exportDpi;
-        final originalName =
+      bool ok = false;
-            (display != null && display.trim().isNotEmpty)
+      String? savedPath;
-                ? display.trim()
-                : widget.currentFile.name.isNotEmpty
-                ? widget.currentFile.name
-                : widget.currentFile.path.isNotEmpty
-                ? widget.currentFile.path.split('/').last.split('\\').last
-                : 'document.pdf';
-        final suggested = _suggestSignedName(originalName);
 
-        if (!kIsWeb) {
+      // Derive a suggested filename based on the opened file. Prefer the
-          final path = await ref
+      // provided display name if available (see Linux portal note above).
-              .read(pdfExportViewModelProvider)
+      final display = widget.currentFileName;
-              .pickSavePathWithSuggestedName(suggested);
+      final originalName =
-          if (path == null || path.trim().isEmpty) return;
+          (display != null && display.trim().isNotEmpty)
-          final fullPath = _ensurePdfExtension(path.trim());
+              ? display.trim()
-          savedPath = fullPath;
+              : widget.currentFile.name.isNotEmpty
-          // ignore: avoid_print
+              ? widget.currentFile.name
-          debugPrint('_saveSignedPdf: picked save path ' + fullPath);
+              : widget.currentFile.path.isNotEmpty
-          ok = await ref
+              ? widget.currentFile.path.split('/').last.split('\\').last
-              .read(pdfExportViewModelProvider)
+              : 'document.pdf';
-              .exportToPath(
+      final suggested = _suggestSignedName(originalName);
-                outputPath: fullPath,
+
-                uiPageSize: _pageSize,
+      if (!kIsWeb) {
-                signatureImageBytes: null,
+        final path = await ref
-                targetDpi: targetDpi,
+            .read(pdfExportViewModelProvider)
-              );
+            .pickSavePathWithSuggestedName(suggested);
-          // ignore: avoid_print
+        if (path == null || path.trim().isEmpty) return;
-          debugPrint('_saveSignedPdf: saveBytesToFile ok=' + ok.toString());
+        final fullPath = _ensurePdfExtension(path.trim());
-        } else {
+        savedPath = fullPath;
-          // Web: export and trigger browser download
+        final src = pdf.pickedPdfBytes ?? Uint8List(0);
-          final out = await ref
+        final out = await exporter.exportSignedPdfFromBytes(
-              .read(documentRepositoryProvider.notifier)
+          srcBytes: src,
-              .exportDocumentToBytes(
+          uiPageSize: _pageSize,
-                uiPageSize: _pageSize,
+          signatureImageBytes: null,
-                signatureImageBytes: null,
+          placementsByPage: pdf.placementsByPage,
-                targetDpi: targetDpi,
+          targetDpi: targetDpi,
-              );
+        );
-          if (out != null) {
+        if (out != null) {
-            ok = await downloadBytes(out, filename: suggested);
+          ok = await exporter.saveBytesToFile(bytes: out, outputPath: fullPath);
-            savedPath = suggested;
-          } else {
-            debugPrint('_saveSignedPdf: export to bytes failed');
-          }
         }
-        if (!kIsWeb) {
+      } else {
-          messenger.showSnackBar(
+        // Web: export and trigger browser download
-            SnackBar(
+        final src = pdf.pickedPdfBytes ?? Uint8List(0);
-              content: Text(
+        final out = await exporter.exportSignedPdfFromBytes(
-                ok
+          srcBytes: src,
-                    ? AppLocalizations.of(
+          uiPageSize: _pageSize,
-                      weakContext,
+          signatureImageBytes: null,
-                    ).savedWithPath(savedPath ?? '')
+          placementsByPage: pdf.placementsByPage,
-                    : AppLocalizations.of(weakContext).failedToSavePdf,
+          targetDpi: targetDpi,
-              ),
+        );
-            ),
+        if (out != null) {
-          );
+          // Use suggested filename for browser download
-          debugPrint('_saveSignedPdf: SnackBar shown ok=' + ok.toString());
+          ok = await downloadBytes(out, filename: suggested);
-        } else {
+          savedPath = suggested;
-          messenger.showSnackBar(
-            SnackBar(
-              content: Text(
-                ok
-                    ? AppLocalizations.of(
-                      weakContext,
-                    ).savedWithPath(savedPath ?? 'signed.pdf')
-                    : AppLocalizations.of(weakContext).failedToSavePdf,
-              ),
-            ),
-          );
-        }
-      } finally {
-        if (mounted) {
-          ref.read(pdfExportViewModelProvider.notifier).setExporting(false);
-          // ignore: avoid_print
-          debugPrint('_saveSignedPdf: exporting flag set false');
         }
       }
-    });
+      if (!kIsWeb) {
+        if (ok) {
+          messenger.showSnackBar(
+            SnackBar(
+              content: Text(
+                AppLocalizations.of(context).savedWithPath(savedPath ?? ''),
+              ),
+            ),
+          );
+        } else {
+          messenger.showSnackBar(
+            SnackBar(
+              content: Text(AppLocalizations.of(context).failedToSavePdf),
+            ),
+          );
+        }
+      } else {
+        // Web: show a toast-like confirmation
+        messenger.showSnackBar(
+          SnackBar(
+            content: Text(
+              ok
+                  ? AppLocalizations.of(
+                    context,
+                  ).savedWithPath(savedPath ?? 'signed.pdf')
+                  : AppLocalizations.of(context).failedToSavePdf,
+            ),
+          ),
+        );
+      }
+    } finally {
+      ref.read(pdfExportViewModelProvider.notifier).setExporting(false);
+    }
   }
 
   String _ensurePdfExtension(String name) {
@@ -316,17 +297,7 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
         max: _pagesMax,
         builder:
             (context, area) => Offstage(
-              offstage: () {
+              offstage: !_showPagesSidebar,
-                try {
-                  return !(ResponsiveBreakpoints.of(
-                        context,
-                      ).largerThan(MOBILE) &&
-                      _showPagesSidebar);
-                } catch (_) {
-                  // In test environments without ResponsiveBreakpoints, default to showing
-                  return !_showPagesSidebar;
-                }
-              }(),
               child: Consumer(
                 builder: (context, ref, child) {
                   final pdfViewModel = ref.watch(pdfViewModelProvider);
@@ -336,7 +307,7 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
                       pdf.loaded && pdf.pickedPdfBytes != null
                           ? PdfDocumentRefData(
                             pdf.pickedPdfBytes!,
-                            sourceName: pdfViewModel.documentSourceName,
+                            sourceName: 'document.pdf',
                           )
                           : null;
 
@@ -357,7 +328,6 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
                 controller: _viewModel.controller,
                 key: const ValueKey('pdf_page_area'),
                 pageSize: _pageSize,
-                onDocumentChanged: widget.onDocumentChanged,
               ),
             ),
       ),
@@ -381,24 +351,6 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
     _applySidebarVisibility();
   }
 
-  @override
-  void didChangeDependencies() {
-    super.didChangeDependencies();
-    // Detect breakpoint changes from Responsive Framework and update areas once.
-    bool canShowPagesSidebar = true;
-    try {
-      canShowPagesSidebar = ResponsiveBreakpoints.of(
-        context,
-      ).largerThan(MOBILE);
-    } catch (_) {
-      canShowPagesSidebar = true;
-    }
-    if (_lastCanShowPagesSidebar != canShowPagesSidebar) {
-      _lastCanShowPagesSidebar = canShowPagesSidebar;
-      _applySidebarVisibility();
-    }
-  }
-
   @override
   void dispose() {
     _viewModel.controller.removeListener(_onControllerChanged);
@@ -407,65 +359,29 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
   }
 
   void _applySidebarVisibility() {
-    // Respect responsive layout: disable Pages sidebar on MOBILE.
-    bool canShowPagesSidebar = true;
-    try {
-      canShowPagesSidebar = ResponsiveBreakpoints.of(
-        context,
-      ).largerThan(MOBILE);
-    } catch (_) {
-      // If ResponsiveBreakpoints isn't available yet (e.g., during early init),
-      // fall back to allowing sidebars to avoid crashes; builders also guard.
-      canShowPagesSidebar = true;
-    }
-
     // Left pages sidebar
     final left = _splitController.areas[0];
-    final wantPagesVisible = _showPagesSidebar && canShowPagesSidebar;
+    if (_showPagesSidebar) {
-    final isPagesHidden =
+      left.max = _pagesMax;
-        (left.max == 1 && left.min == 0 && (left.size ?? 1) == 1);
+      left.min = _pagesMin;
-    if (wantPagesVisible) {
+      left.size = _lastPagesWidth.clamp(_pagesMin, _pagesMax);
-      // Only expand if currently hidden; otherwise keep user's size.
-      if (isPagesHidden) {
-        left.max = _pagesMax;
-        left.min = _pagesMin;
-        left.size = _lastPagesWidth.clamp(_pagesMin, _pagesMax);
-      } else {
-        left.max = _pagesMax;
-        left.min = _pagesMin;
-        // Preserve current size (user may have adjusted it).
-        _lastPagesWidth = left.size ?? _lastPagesWidth;
-      }
     } else {
-      // Only collapse if currently visible; remember current size for restore.
+      _lastPagesWidth = left.size ?? _lastPagesWidth;
-      if (!isPagesHidden) {
+      left.min = 0;
-        _lastPagesWidth = left.size ?? _lastPagesWidth;
+      left.max = 1;
-        left.min = 0;
+      left.size = 1; // effectively hidden
-        left.max = 1;
-        left.size = 1; // effectively hidden
-      }
     }
     // Right signatures sidebar
     final right = _splitController.areas[2];
-    final isSignaturesHidden =
-        (right.max == 1 && right.min == 0 && (right.size ?? 1) == 1);
     if (_showSignaturesSidebar) {
-      if (isSignaturesHidden) {
+      right.max = _signaturesMax;
-        right.max = _signaturesMax;
+      right.min = _signaturesMin;
-        right.min = _signaturesMin;
+      right.size = _lastSignaturesWidth.clamp(_signaturesMin, _signaturesMax);
-        right.size = _lastSignaturesWidth.clamp(_signaturesMin, _signaturesMax);
-      } else {
-        right.max = _signaturesMax;
-        right.min = _signaturesMin;
-        _lastSignaturesWidth = right.size ?? _lastSignaturesWidth;
-      }
     } else {
-      if (!isSignaturesHidden) {
+      _lastSignaturesWidth = right.size ?? _lastSignaturesWidth;
-        _lastSignaturesWidth = right.size ?? _lastSignaturesWidth;
+      right.min = 0;
-        right.min = 0;
+      right.max = 1;
-        right.max = 1;
+      right.size = 1;
-        right.size = 1;
-      }
     }
   }
 
@@ -477,13 +393,6 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
   Widget _buildScaffold(BuildContext context) {
     final isExporting = ref.watch(pdfExportViewModelProvider).exporting;
     final l = AppLocalizations.of(context);
-    // Defensive flag for tests not wrapped in ResponsiveBreakpoints
-    bool largerThanMobile;
-    try {
-      largerThanMobile = ResponsiveBreakpoints.of(context).largerThan(MOBILE);
-    } catch (_) {
-      largerThanMobile = true;
-    }
     return Scaffold(
       body: Padding(
         padding: const EdgeInsets.all(12),
@@ -544,28 +453,6 @@ class _PdfSignatureHomePageState extends ConsumerState<PdfSignatureHomePage> {
                         _applySidebarVisibility();
                       }),
                 ),
-                // Optional quick toggle for pages sidebar on larger screens
-                if (largerThanMobile)
-                  Align(
-                    alignment: Alignment.centerLeft,
-                    child: SizedBox(
-                      height: 0,
-                      width: 0,
-                      child: Offstage(
-                        offstage: true,
-                        child: IconButton(
-                          key: const Key('btn_toggle_pages_sidebar_hidden'),
-                          onPressed: () {
-                            setState(() {
-                              _showPagesSidebar = !_showPagesSidebar;
-                              _applySidebarVisibility();
-                            });
-                          },
-                          icon: const Icon(Icons.view_sidebar),
-                        ),
-                      ),
-                    ),
-                  ),
                 // Expose a compact signature drawer trigger area for tests when sidebar hidden
                 if (!_showSignaturesSidebar)
                   Align(

lib/ui/features/pdf/widgets/pdf_toolbar.dart

@@ -2,10 +2,8 @@ import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:pdf_signature/l10n/app_localizations.dart';
-import 'package:responsive_framework/responsive_framework.dart';
 
 import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
-import 'package:pdf_signature/data/repositories/document_repository.dart';
 
 class PdfToolbar extends ConsumerStatefulWidget {
   const PdfToolbar({
@@ -60,9 +58,7 @@ class _PdfToolbarState extends ConsumerState<PdfToolbar> {
   @override
   Widget build(BuildContext context) {
     final pdfViewModel = ref.watch(pdfViewModelProvider);
-    final pdf = ref.watch(
+    final pdf = pdfViewModel.document;
-      documentRepositoryProvider,
-    ); // Watch document directly for updates
     final currentPage = pdfViewModel.currentPage;
     final l = AppLocalizations.of(context);
     final pageInfo = l.pageInfo(currentPage, pdf.pageCount);
@@ -71,21 +67,6 @@ class _PdfToolbarState extends ConsumerState<PdfToolbar> {
       builder: (context, constraints) {
         final bool compact = constraints.maxWidth < 260;
         final double gotoWidth = 50;
-        // Be defensive in tests that don't provide ResponsiveBreakpoints
-        final bool isLargerThanMobile = () {
-          try {
-            return ResponsiveBreakpoints.of(context).largerThan(MOBILE);
-          } catch (_) {
-            return true; // default to full toolbar on tests/minimal hosts
-          }
-        }();
-        final String fileDisplay = () {
-          final path = widget.filePath;
-          if (path == null || path.isEmpty) return 'No file selected';
-          if (isLargerThanMobile) return path;
-          // Extract file name for mobile (supports both / and \ separators)
-          return path.split('/').last.split('\\').last;
-        }();
 
         // Center content of the toolbar
         final center = Wrap(
@@ -101,15 +82,14 @@ class _PdfToolbarState extends ConsumerState<PdfToolbar> {
                 children: [
                   const Icon(Icons.insert_drive_file, size: 18),
                   const SizedBox(width: 6),
-                  Flexible(
+                  ConstrainedBox(
-                    child: ConstrainedBox(
+                    constraints: const BoxConstraints(maxWidth: 220),
-                      constraints: const BoxConstraints(maxWidth: 220),
+                    child: Text(
-                      child: Text(
+                      // if filePath not null
-                        fileDisplay,
+                      widget.filePath != null
-                        maxLines: 1,
+                          ? widget.filePath!
-                        softWrap: false,
+                          : 'No file selected',
-                        overflow: TextOverflow.ellipsis,
+                      overflow: TextOverflow.ellipsis,
-                      ),
                     ),
                   ),
                 ],
@@ -150,68 +130,62 @@ class _PdfToolbarState extends ConsumerState<PdfToolbar> {
                       ),
                     ],
                   ),
-                  if (isLargerThanMobile)
+                  Wrap(
-                    Wrap(
+                    spacing: 6,
-                      spacing: 6,
+                    runSpacing: 4,
-                      runSpacing: 4,
+                    crossAxisAlignment: WrapCrossAlignment.center,
-                      crossAxisAlignment: WrapCrossAlignment.center,
+                    children: [
-                      children: [
+                      Text(l.goTo),
-                        Text(l.goTo),
+                      SizedBox(
-                        SizedBox(
+                        width: gotoWidth,
-                          width: gotoWidth,
+                        child: TextField(
-                          child: TextField(
+                          key: const Key('txt_goto'),
-                            key: const Key('txt_goto'),
+                          controller: _goToController,
-                            controller: _goToController,
+                          keyboardType: TextInputType.number,
-                            keyboardType: TextInputType.number,
+                          inputFormatters: [
-                            inputFormatters: [
+                            FilteringTextInputFormatter.digitsOnly,
-                              FilteringTextInputFormatter.digitsOnly,
+                          ],
-                            ],
+                          enabled: !widget.disabled,
-                            enabled: !widget.disabled,
+                          decoration: InputDecoration(
-                            decoration: InputDecoration(
+                            isDense: true,
-                              isDense: true,
+                            hintText: '1..${pdf.pageCount}',
-                              hintText: '1..${pdf.pageCount}',
-                            ),
-                            onSubmitted: (_) => _submitGoTo(),
                           ),
+                          onSubmitted: (_) => _submitGoTo(),
                         ),
-                        if (!compact)
+                      ),
-                          IconButton(
+                      if (!compact)
-                            key: const Key('btn_goto_apply'),
-                            tooltip: l.goTo,
-                            icon: const Icon(Icons.arrow_forward),
-                            onPressed: widget.disabled ? null : _submitGoTo,
-                          ),
-                      ],
-                    ),
-
-                  if (isLargerThanMobile) ...[
-                    const SizedBox(width: 8),
-                    Wrap(
-                      crossAxisAlignment: WrapCrossAlignment.center,
-                      children: [
                         IconButton(
-                          key: const Key('btn_zoom_out'),
+                          key: const Key('btn_goto_apply'),
-                          tooltip: 'Zoom out',
+                          tooltip: l.goTo,
-                          onPressed: widget.disabled ? null : widget.onZoomOut,
+                          icon: const Icon(Icons.arrow_forward),
-                          icon: const Icon(Icons.zoom_out),
+                          onPressed: widget.disabled ? null : _submitGoTo,
                         ),
-                        Text(
+                    ],
-                          //if not null
+                  ),
-                          widget.zoomLevel != null
+                  const SizedBox(width: 8),
-                              ? '${widget.zoomLevel}%'
+                  Wrap(
-                              : '',
+                    crossAxisAlignment: WrapCrossAlignment.center,
-                          style: const TextStyle(fontSize: 12),
+                    children: [
-                        ),
+                      IconButton(
-                        IconButton(
+                        key: const Key('btn_zoom_out'),
-                          key: const Key('btn_zoom_in'),
+                        tooltip: 'Zoom out',
-                          tooltip: 'Zoom in',
+                        onPressed: widget.disabled ? null : widget.onZoomOut,
-                          onPressed: widget.disabled ? null : widget.onZoomIn,
+                        icon: const Icon(Icons.zoom_out),
-                          icon: const Icon(Icons.zoom_in),
+                      ),
-                        ),
+                      Text(
-                      ],
+                        //if not null
-                    ),
+                        widget.zoomLevel != null ? '${widget.zoomLevel}%' : '',
-                    SizedBox(width: 6),
+                        style: const TextStyle(fontSize: 12),
-                  ],
+                      ),
+                      IconButton(
+                        key: const Key('btn_zoom_in'),
+                        tooltip: 'Zoom in',
+                        onPressed: widget.disabled ? null : widget.onZoomIn,
+                        icon: const Icon(Icons.zoom_in),
+                      ),
+                    ],
+                  ),
+                  SizedBox(width: 6),
                 ],
               ),
             ],
@@ -220,21 +194,19 @@ class _PdfToolbarState extends ConsumerState<PdfToolbar> {
 
         return Row(
           children: [
-            if (isLargerThanMobile) ...[
+            IconButton(
-              IconButton(
+              key: const Key('btn_toggle_pages_sidebar'),
-                key: const Key('btn_toggle_pages_sidebar'),
+              tooltip: 'Toggle pages overview',
-                tooltip: 'Toggle pages overview',
+              onPressed: widget.disabled ? null : widget.onTogglePagesSidebar,
-                onPressed: widget.disabled ? null : widget.onTogglePagesSidebar,
+              icon: Icon(
-                icon: Icon(
+                Icons.view_sidebar,
-                  Icons.view_sidebar,
+                color:
-                  color:
+                    widget.showPagesSidebar
-                      widget.showPagesSidebar
+                        ? Theme.of(context).colorScheme.primary
-                          ? Theme.of(context).colorScheme.primary
+                        : null,
-                          : null,
-                ),
               ),
-              const SizedBox(width: 8),
+            ),
-            ],
+            const SizedBox(width: 8),
             Expanded(child: center),
             const SizedBox(width: 8),
             IconButton(

lib/ui/features/pdf/widgets/pdf_viewer_widget.dart

@@ -5,13 +5,6 @@ import 'package:pdf_signature/l10n/app_localizations.dart';
 import 'pdf_page_overlays.dart';
 import './pdf_mock_continuous_list.dart';
 import '../view_model/pdf_view_model.dart';
-import 'package:pdf_signature/domain/models/document.dart';
-import 'package:pdf_signature/data/repositories/document_repository.dart';
-import 'dart:typed_data';
-
-// Provider to control whether viewer overlays (like scroll thumbs) are enabled.
-// Integration tests can override this to false to avoid long-running animations.
-final viewerOverlaysEnabledProvider = Provider<bool>((ref) => true);
 
 class PdfViewerWidget extends ConsumerStatefulWidget {
   const PdfViewerWidget({
@@ -20,53 +13,19 @@ class PdfViewerWidget extends ConsumerStatefulWidget {
     this.pageKeyBuilder,
     this.scrollToPage,
     required this.controller,
-    this.innerViewerKey,
-    this.onDocumentChanged,
   });
 
   final Size pageSize;
   final GlobalKey Function(int page)? pageKeyBuilder;
   final void Function(int page)? scrollToPage;
   final PdfViewerController controller;
-  // Optional key applied to the inner Pdfrx PdfViewer to force disposal/rebuild
-  final Key? innerViewerKey;
-  // External hook to observe document changes (forwarded from Pdfrx onDocumentChanged)
-  final void Function(PdfDocument?)? onDocumentChanged;
 
   @override
   ConsumerState<PdfViewerWidget> createState() => _PdfViewerWidgetState();
 }
 
 class _PdfViewerWidgetState extends ConsumerState<PdfViewerWidget> {
-  final ValueNotifier<PdfDocumentRef?> _docRefNotifier = ValueNotifier(null);
+  PdfDocumentRef? _documentRef;
-  Uint8List? _lastBytes;
-  void _updateDocRef(Document doc) {
-    if (!doc.loaded || doc.pickedPdfBytes == null) {
-      if (_docRefNotifier.value != null) {
-        debugPrint('[PdfViewerWidget] Clearing docRef (no document loaded)');
-        _docRefNotifier.value = null;
-      }
-      return;
-    }
-    final bytes = doc.pickedPdfBytes!;
-    if (!identical(bytes, _lastBytes)) {
-      _lastBytes = bytes;
-      final viewModel = ref.read(pdfViewModelProvider);
-      debugPrint(
-        '[PdfViewerWidget] New PDF bytes detected -> ${viewModel.documentSourceName}',
-      );
-      // Force a full detach by setting null first so PdfViewer unmounts even if the
-      // framework would otherwise optimize rebuilds with same key ordering.
-      if (_docRefNotifier.value != null) {
-        _docRefNotifier.value = null;
-      }
-      final newRef = PdfDocumentRefData(
-        bytes,
-        sourceName: viewModel.documentSourceName,
-      );
-      _docRefNotifier.value = newRef;
-    }
-  }
 
   // Public getter for testing the actual viewer page
   int? get viewerCurrentPage => widget.controller.pageNumber;
@@ -85,9 +44,31 @@ class _PdfViewerWidgetState extends ConsumerState<PdfViewerWidget> {
   @override
   Widget build(BuildContext context) {
     final pdfViewModel = ref.watch(pdfViewModelProvider);
-    final document = ref.watch(documentRepositoryProvider);
+    final document = pdfViewModel.document;
     final useMock = pdfViewModel.useMockViewer;
-    _updateDocRef(document);
+    // trigger rebuild when active rect changes
+
+    // Update document ref when document changes
+    if (document.loaded && document.pickedPdfBytes != null) {
+      if (_documentRef == null) {
+        _documentRef = PdfDocumentRefData(
+          document.pickedPdfBytes!,
+          sourceName: 'document.pdf',
+        );
+      }
+    } else {
+      _documentRef = null;
+    }
+
+    if (_documentRef == null && !useMock) {
+      String text;
+      try {
+        text = AppLocalizations.of(context).noPdfLoaded;
+      } catch (_) {
+        text = 'No PDF loaded';
+      }
+      return Center(child: Text(text));
+    }
 
     if (useMock) {
       return PdfMockContinuousList(
@@ -100,120 +81,77 @@ class _PdfViewerWidgetState extends ConsumerState<PdfViewerWidget> {
       );
     }
 
-    final overlaysEnabled = ref.watch(viewerOverlaysEnabledProvider);
+    return PdfViewer(
-    return ValueListenableBuilder<PdfDocumentRef?>(
+      _documentRef!,
-      valueListenable: _docRefNotifier,
+      key: const Key(
-      builder: (context, docRef, _) {
+        'pdf_continuous_mock_list',
-        if (docRef == null) {
+      ), // Keep the same key for test compatibility
-          String text;
+      controller: widget.controller,
-          try {
+      params: PdfViewerParams(
-            text = AppLocalizations.of(context).noPdfLoaded;
+        onViewerReady: (document, controller) {
-          } catch (_) {
+          // Update page count in repository
-            text = 'No PDF loaded';
+          ref
+              .read(pdfViewModelProvider.notifier)
+              .setPageCount(document.pages.length);
+        },
+        onPageChanged: (page) {
+          if (page != null) {
+            // Also update the view model to keep them in sync
+            ref.read(pdfViewModelProvider.notifier).jumpToPage(page);
           }
-          return Center(child: Text(text));
+        },
-        }
+        viewerOverlayBuilder: (context, size, handle) {
-        final pdfViewModel = ref.read(pdfViewModelProvider);
+          return [
-        final viewerKey =
+            // Vertical scroll thumb on the right
-            widget.innerViewerKey ??
+            PdfViewerScrollThumb(
-            Key('pdf_viewer_${pdfViewModel.documentSourceName}');
+              controller: widget.controller,
-
+              orientation: ScrollbarOrientation.right,
-        return PdfViewer(
+              thumbSize: const Size(40, 25),
-          docRef,
+              thumbBuilder:
-          key: viewerKey,
+                  (context, thumbSize, pageNumber, controller) => Container(
-          controller: widget.controller,
+                    color: Colors.black.withValues(alpha: 0.7),
-          params: PdfViewerParams(
+                    child: Center(
-            onViewerReady: (document, controller) {
+                      child: Text(
-              // Update page count in repository
+                        'Pg $pageNumber',
-              ref
+                        style: const TextStyle(
-                  .read(pdfViewModelProvider.notifier)
+                          color: Colors.white,
-                  .setPageCount(document.pages.length);
+                          fontSize: 12,
-            },
-            onPageChanged: (page) {
-              if (page != null) {
-                // Also update the view model to keep them in sync
-                ref.read(pdfViewModelProvider.notifier).jumpToPage(page);
-              }
-            },
-            onDocumentChanged: (doc) async {
-              final pc = doc?.pages.length;
-              debugPrint(
-                '[PdfViewerWidget] onDocumentChanged called (pages=$pc)',
-              );
-              if (doc != null) {
-                // Update internal page count state
-                ref
-                    .read(pdfViewModelProvider.notifier)
-                    .setPageCount(doc.pages.length);
-              }
-              // Invoke external listener after internal handling
-              try {
-                widget.onDocumentChanged?.call(doc);
-              } catch (e, st) {
-                debugPrint(
-                  '[PdfViewerWidget] external onDocumentChanged threw: $e\n$st',
-                );
-              }
-            },
-            viewerOverlayBuilder:
-                overlaysEnabled
-                    ? (context, size, handle) {
-                      return [
-                        // Vertical scroll thumb on the right
-                        PdfViewerScrollThumb(
-                          controller: widget.controller,
-                          orientation: ScrollbarOrientation.right,
-                          thumbSize: const Size(40, 25),
-                          thumbBuilder:
-                              (context, thumbSize, pageNumber, controller) =>
-                                  Container(
-                                    color: Colors.black.withValues(alpha: 0.7),
-                                    child: Center(
-                                      child: Text(
-                                        'Pg $pageNumber',
-                                        style: const TextStyle(
-                                          color: Colors.white,
-                                          fontSize: 12,
-                                        ),
-                                      ),
-                                    ),
-                                  ),
                         ),
-                        // Horizontal scroll thumb on the bottom
+                      ),
-                        PdfViewerScrollThumb(
+                    ),
-                          controller: widget.controller,
+                  ),
-                          orientation: ScrollbarOrientation.bottom,
+            ),
-                          thumbSize: const Size(40, 25),
+            // Horizontal scroll thumb on the bottom
-                          thumbBuilder:
+            PdfViewerScrollThumb(
-                              (context, thumbSize, pageNumber, controller) =>
+              controller: widget.controller,
-                                  Container(
+              orientation: ScrollbarOrientation.bottom,
-                                    color: Colors.black.withValues(alpha: 0.7),
+              thumbSize: const Size(40, 25),
-                                    child: Center(
+              thumbBuilder:
-                                      child: Text(
+                  (context, thumbSize, pageNumber, controller) => Container(
-                                        'Pg $pageNumber',
+                    color: Colors.black.withValues(alpha: 0.7),
-                                        style: const TextStyle(
+                    child: Center(
-                                          color: Colors.white,
+                      child: Text(
-                                          fontSize: 12,
+                        'Pg $pageNumber',
-                                        ),
+                        style: const TextStyle(
-                                      ),
+                          color: Colors.white,
-                                    ),
+                          fontSize: 12,
-                                  ),
                         ),
-                      ];
+                      ),
-                    }
+                    ),
-                    : (context, size, handle) => const <Widget>[],
+                  ),
-            // Per-page overlays to enable page-specific drag targets and placed signatures
+            ),
-            pageOverlaysBuilder: (context, pageRect, page) {
+          ];
-              return [
+        },
-                PdfPageOverlays(
+        // Per-page overlays to enable page-specific drag targets and placed signatures
-                  pageSize: Size(pageRect.width, pageRect.height),
+        pageOverlaysBuilder: (context, pageRect, page) {
-                  pageNumber: page.pageNumber,
+          return [
-                ),
+            PdfPageOverlays(
-              ];
+              pageSize: Size(pageRect.width, pageRect.height),
-            },
+              pageNumber: page.pageNumber,
-          ),
+            ),
-        );
+          ];
-      },
+        },
+      ),
     );
   }
 }

lib/ui/features/welcome/view_model/welcome_view_model.dart

@@ -1,4 +1,4 @@
-import 'package:flutter/foundation.dart';
+import 'dart:typed_data';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
 import 'package:go_router/go_router.dart';
@@ -11,14 +11,6 @@ class WelcomeViewModel {
   WelcomeViewModel(this.ref, this.router);
 
   Future<void> openPdf({required String path, Uint8List? bytes}) async {
-    // Return early if no bytes provided - can't open PDF without data
-    if (bytes == null) {
-      debugPrint(
-        '[WelcomeViewModel] Cannot open PDF: no bytes provided for $path',
-      );
-      return;
-    }
-
     // Use PdfSessionViewModel to open and navigate.
     final session = ref.read(pdfSessionViewModelProvider(router));
     await session.openPdf(path: path, bytes: bytes);

lib/utils/download.dart

@@ -1,17 +1,11 @@
-import 'package:flutter/foundation.dart';
+import 'dart:typed_data';
 
-// On modern Flutter Web (Wasm GC, e.g., Chromium), dart:html is not available.
+import 'download_stub.dart' if (dart.library.html) 'download_web.dart' as impl;
-// Use js_interop capability to select the web implementation that relies on
-// package:web instead of dart:html.
-import 'download_stub.dart'
-    if (dart.library.js_interop) 'download_web.dart'
-    as impl;
 
 /// Initiates a platform-appropriate download/save operation.
 ///
 /// On Web: triggers a browser download with the provided filename.
 /// On non-Web: returns false (no-op). Use your existing IO save flow instead.
 Future<bool> downloadBytes(Uint8List bytes, {required String filename}) {
-  debugPrint('downloadBytes: initiating download');
   return impl.downloadBytes(bytes, filename: filename);
 }

lib/utils/download_stub.dart

@@ -1,9 +1,6 @@
 import 'dart:typed_data';
 
-import 'package:flutter/widgets.dart';
-
 Future<bool> downloadBytes(Uint8List bytes, {required String filename}) async {
   // Not supported on non-web. Return false so caller can fallback to file save.
-  debugPrint('downloadBytes: not supported on this platform');
   return false;
 }

lib/utils/download_web.dart

@@ -1,28 +1,23 @@
-// Implementation for Web using package:web to support Wasm GC (Chromium)
+// ignore_for_file: deprecated_member_use
-// without importing dart:html directly.
+// ignore: avoid_web_libraries_in_flutter
-import 'dart:convert';
+import 'dart:html' as html;
-import 'package:flutter/foundation.dart';
+import 'dart:typed_data';
-import 'package:web/web.dart' as web;
 
 Future<bool> downloadBytes(Uint8List bytes, {required String filename}) async {
   try {
-    // Use a data URL to avoid Blob/typed array interop issues under Wasm GC.
+    final blob = html.Blob([bytes], 'application/pdf');
-    final url = 'data:application/pdf;base64,${base64Encode(bytes)}';
+    final url = html.Url.createObjectUrlFromBlob(blob);
-
-    // Create an anchor element and trigger a click to download
     final anchor =
-        web.HTMLAnchorElement()
+        html.document.createElement('a') as html.AnchorElement
           ..href = url
           ..download = filename
           ..style.display = 'none';
-
+    html.document.body?.children.add(anchor);
-    web.document.body?.append(anchor);
     anchor.click();
     anchor.remove();
-
+    html.Url.revokeObjectUrl(url);
     return true;
-  } catch (e, st) {
+  } catch (_) {
-    debugPrint('Error: downloadBytes failed: $e\n$st');
     return false;
   }
 }

lib/utils/pdfrx_cache_init/pdfrx_cache_init.dart

@@ -1,3 +0,0 @@
-// Conditional export: use IO implementation except when compiling for web (html).
-export 'pdfrx_cache_init_io.dart'
-    if (dart.library.html) 'pdfrx_cache_init_web.dart';

lib/utils/pdfrx_cache_init/pdfrx_cache_init_io.dart

@@ -1,26 +0,0 @@
-import 'dart:io';
-import 'package:flutter/foundation.dart';
-import 'package:path_provider/path_provider.dart';
-import 'package:pdfrx/pdfrx.dart';
-
-/// Initialize Pdfrx cache directory for IO platforms (mobile/desktop). No-op on web.
-Future<void> initPdfrxCache() async {
-  try {
-    if (kIsWeb) return; // Guard (should not be used on web, but extra safety)
-    if (Pdfrx.getCacheDirectory != null) return; // Already set
-    final dir = await getTemporaryDirectory();
-    final cacheDir = Directory('${dir.path}/pdfrx_cache');
-    if (!await cacheDir.exists()) {
-      await cacheDir.create(recursive: true);
-    }
-    Pdfrx.getCacheDirectory = () async => cacheDir.path;
-    debugPrint(
-      '[pdfrx_cache_init_io] Pdfrx cache directory set to ${cacheDir.path}',
-    );
-  } catch (e, st) {
-    debugPrint(
-      '[pdfrx_cache_init_io] Failed to initialize Pdfrx cache directory: $e',
-    );
-    debugPrint(st.toString());
-  }
-}

lib/utils/pdfrx_cache_init/pdfrx_cache_init_web.dart

@@ -1,17 +0,0 @@
-import 'package:flutter/foundation.dart';
-import 'package:pdfrx/pdfrx.dart';
-
-/// Web stub: pdfrx can operate without a filesystem cache; leave getCacheDirectory null.
-Future<void> initPdfrxCache() async {
-  // Intentionally no-op. If desired, could set an in-memory indicator.
-  debugPrint(
-    '[pdfrx_cache_init_web] Skipping Pdfrx cache directory setup on web',
-  );
-  // Ensure any previous (hot-reload) IO assignment isn't kept when switching target.
-  if (kIsWeb && Pdfrx.getCacheDirectory != null) {
-    // Leave as-is; clearing could break existing references. Merely log.
-    debugPrint(
-      '[pdfrx_cache_init_web] Existing getCacheDirectory left unchanged',
-    );
-  }
-}

pubspec.yaml

@@ -37,32 +37,30 @@ dependencies:
   flutter_riverpod: ^2.6.1
   shared_preferences: ^2.5.3
   flutter_dotenv: ^6.0.0
+  file_selector: ^1.0.3
   path_provider: ^2.1.5
   pdfrx: ^2.1.9
   pdf: ^3.10.8
-  # printing: ^5.14.2 # extension of pdf pkg
   hand_signature: ^3.1.0+2
   image: ^4.2.0
+  printing: ^5.14.2
   result_dart: ^2.1.1
   go_router: ^16.2.0
   flutter_localizations:
     sdk: flutter
   intl: any
   flutter_localized_locales: ^2.0.5
-  desktop_drop: ^0.6.1
+  desktop_drop: ^0.5.0
   multi_split_view: ^3.6.1
   freezed_annotation: ^3.1.0
   json_annotation: ^4.9.0
-  share_plus: ^12.0.0
+  share_plus: ^11.1.0
   logging: ^1.3.0
   riverpod_annotation: ^2.6.1
   colorfilter_generator: ^0.0.8
   flutter_box_transform: ^0.4.7
-  file_picker: ^10.3.3
-  responsive_framework: ^1.5.1
   # disable_web_context_menu: ^1.1.0
   # ml_linalg: ^13.12.6
-  web: ^1.1.1
 
 dev_dependencies:
   flutter_test:
@@ -135,11 +133,10 @@ flutter:
 
 
 flutter_launcher_icons:
-  image_path: "assets/icon/pdf_signature-icon.png"
+  android: "launcher_icon"
-  android: true
   ios: true
+  image_path: "assets/icon/pdf_signature-icon.png"
   min_sdk_android: 21 # android min sdk min:16, default 21
-  remove_alpha_ios: true
   web:
     generate: true
     image_path: "assets/icon/pdf_signature-icon.png"

test/features/_test_helper.dart

@@ -40,6 +40,7 @@ Future<ProviderContainer> pumpApp(
 }) async {
   SharedPreferences.setMockInitialValues(initialPrefs);
   final prefs = await SharedPreferences.getInstance();
+  final fakeExport = FakeExportService();
   final container = ProviderContainer(
     overrides: [
       preferencesRepositoryProvider.overrideWith(
@@ -52,7 +53,11 @@ Future<ProviderContainer> pumpApp(
         (ref) => PdfViewModel(ref, useMockViewer: true),
       ),
       pdfExportViewModelProvider.overrideWith(
-        (ref) => PdfExportViewModel(ref, savePathPicker: () async => 'out.pdf'),
+        (ref) => PdfExportViewModel(
+          ref,
+          exporter: fakeExport,
+          savePathPicker: () async => 'out.pdf',
+        ),
       ),
     ],
   );

test/features/pdf_browser.feature

@@ -9,6 +9,13 @@ Feature: document browser
     And the user can move to the next or previous page
     And the page label shows "Page {1} of {5}"
 
+  Scenario: Jump to a specific page by typing Enter
+    Given the document is open
+    When the user types {3} into the Go to input and presses Enter
+    Then page {3} is displayed
+    And the page label shows "Page {3} of {5}"
+    And the left pages overview highlights page {3}
+
   Scenario: Jump to a specific page using the Apply button
     Given the document is open
     When the user types {4} into the Go to input
@@ -22,6 +29,15 @@ Feature: document browser
     Then page {2} is displayed
     And the page label shows "Page {2} of {5}"
 
+  Scenario: Continuous mode scrolls target page into view on jump
+    Given the document is open
+    And the Page view mode is set to Continuous
+    When the user jumps to page {5}
+    Then page {5} becomes visible in the scroll area
+    And the left pages overview highlights page {5}
+
+
+
   Scenario: Go to clamps out-of-range inputs to valid bounds
     Given the document is open
     When the user enters {0} into the Go to input and applies it
@@ -34,14 +50,3 @@ Feature: document browser
   Scenario: Go to is disabled when no document is loaded
     Given no document is open
     Then the Go to input cannot be used
-
-  Scenario: Open a different document will reset signature placements but keep signature cards
-    Given the document is open
-    When the user opens a different document with {3} pages
-    And {1} signature placements exist on page {1}
-    And {1} signature placements exist on page {2}
-    And {2} signature cards exist
-    Then the first page of the new document is displayed
-    And the page label shows "Page {1} of {3}"
-    And number of signature placements is {0}
-    And {2} signature cards exist

test/features/step/_world.dart

@@ -36,9 +36,6 @@ class TestWorld {
   // Generic flags/values
   static int? selectedPage;
   static int? pendingGoTo; // for simulating typed Go To value across steps
-  static int?
-  nextDocPageCount; // for BDD: desired page count for the next opened document
-  static Map<int, int>? prevPlacementsCount; // snapshot before an action
 
   // Preferences & settings
   static Map<String, String> prefs = {};
@@ -64,8 +61,6 @@ class TestWorld {
     nothingToSaveAttempt = false;
     selectedPage = null;
     pendingGoTo = null;
-    nextDocPageCount = null;
-    prevPlacementsCount = null;
 
     // Preferences
     prefs = {};

test/features/step/a_document_is_open_and_contains_at_least_one_signature_placement.dart

@@ -12,9 +12,7 @@ Future<void> aDocumentIsOpenAndContainsAtLeastOneSignaturePlacement(
 ) async {
   final container = TestWorld.container ?? ProviderContainer();
   TestWorld.container = container;
-  container
+  container.read(documentRepositoryProvider.notifier).openPicked(pageCount: 5);
-      .read(documentRepositoryProvider.notifier)
-      .openDocument(pageCount: 5);
   container
       .read(documentRepositoryProvider.notifier)
       .addPlacement(

test/features/step/a_document_is_open_and_contains_multiple_placed_signature_placements_across_pages.dart

@@ -13,9 +13,7 @@ aDocumentIsOpenAndContainsMultiplePlacedSignaturePlacementsAcrossPages(
 ) async {
   final container = TestWorld.container ?? ProviderContainer();
   TestWorld.container = container;
-  container
+  container.read(documentRepositoryProvider.notifier).openPicked(pageCount: 5);
-      .read(documentRepositoryProvider.notifier)
-      .openDocument(pageCount: 5);
   container
       .read(documentRepositoryProvider.notifier)
       .addPlacement(

test/features/step/a_document_is_open_with_no_signature_placements_placed.dart

@@ -11,6 +11,6 @@ Future<void> aDocumentIsOpenWithNoSignaturePlacementsPlaced(
   TestWorld.container = container;
   container
       .read(documentRepositoryProvider.notifier)
-      .openDocument(pageCount: 5);
+      .openPicked(pageCount: 5);
   // No placements added
 }

test/features/step/a_document_page_is_selected_for_signing.dart

@@ -11,7 +11,7 @@ Future<void> aDocumentPageIsSelectedForSigning(WidgetTester tester) async {
   // Ensure a document is open
   final repo = container.read(documentRepositoryProvider.notifier);
   if (!container.read(documentRepositoryProvider).loaded) {
-    repo.openDocument(pageCount: 5);
+    repo.openPicked(pageCount: 5);
   }
   // Ensure current page is 1 for consistent subsequent steps
   try {

test/features/step/a_multipage_document_is_open.dart

@@ -18,9 +18,7 @@ Future<void> aMultipageDocumentIsOpen(WidgetTester tester) async {
   container.read(signatureCardRepositoryProvider.notifier).state = [
     SignatureCard.initial(),
   ];
-  container
+  container.read(documentRepositoryProvider.notifier).openPicked(pageCount: 5);
-      .read(documentRepositoryProvider.notifier)
-      .openDocument(pageCount: 5);
   // Reset page state providers
   try {
     container.read(pdfViewModelProvider.notifier).jumpToPage(1);

test/features/step/a_sample_multipage_document5_pages_is_available.dart

@@ -11,5 +11,5 @@ Future<void> aSampleMultipageDocument5PagesIsAvailable(
   TestWorld.container = container;
   container
       .read(documentRepositoryProvider.notifier)
-      .openDocument(pageCount: 5);
+      .openPicked(pageCount: 5);
 }

test/features/step/a_signature_asset_is_placed_on_the_page.dart

@@ -17,7 +17,7 @@ Future<void> aSignatureAssetIsPlacedOnThePage(WidgetTester tester) async {
   if (!container.read(documentRepositoryProvider).loaded) {
     container
         .read(documentRepositoryProvider.notifier)
-        .openDocument(pageCount: 5);
+        .openPicked(pageCount: 5);
   }
 
   // Get or create an asset

test/features/step/a_signature_placement_is_placed_on_page.dart

@@ -17,7 +17,7 @@ Future<void> aSignaturePlacementIsPlacedOnPage(
   if (!container.read(documentRepositoryProvider).loaded) {
     container
         .read(documentRepositoryProvider.notifier)
-        .openDocument(pageCount: 5);
+        .openPicked(pageCount: 5);
   }
   final page = param1.toInt();
   container

test/features/step/a_signature_placement_is_placed_with_a_position_and_size_relative_to_the_page.dart

@@ -16,7 +16,7 @@ Future<void> aSignaturePlacementIsPlacedWithAPositionAndSizeRelativeToThePage(
   if (!container.read(documentRepositoryProvider).loaded) {
     container
         .read(documentRepositoryProvider.notifier)
-        .openDocument(pageCount: 5);
+        .openPicked(pageCount: 5);
   }
   final currentPage = container.read(pdfViewModelProvider).currentPage;
   container

test/features/step/dragging_or_resizing_one_does_not_change_the_other.dart

@@ -22,7 +22,7 @@ Future<void> draggingOrResizingOneDoesNotChangeTheOther(
   final changedFirst = firstRectBefore.inflate(5);
   container
       .read(documentRepositoryProvider.notifier)
-      .modifyPlacement(page: page, index: 0, rect: changedFirst);
+      .updatePlacementRect(page: page, index: 0, rect: changedFirst);
 
   final after = container
       .read(documentRepositoryProvider.notifier)

test/features/step/number_of_signature_placements_is.dart

@@ -1,33 +0,0 @@
-import 'package:flutter_test/flutter_test.dart';
-import 'package:flutter_riverpod/flutter_riverpod.dart';
-import 'package:pdf_signature/data/repositories/document_repository.dart';
-import 'package:pdf_signature/data/repositories/signature_card_repository.dart';
-import '_world.dart';
-
-/// Usage: number of signature placements is {0}
-Future<void> numberOfSignaturePlacementsIs(
-  WidgetTester tester,
-  num param1,
-) async {
-  final expected = param1.toInt();
-  final c = TestWorld.container ?? ProviderContainer();
-  final doc = c.read(documentRepositoryProvider);
-  final total = doc.placementsByPage.values.fold<int>(
-    0,
-    (sum, list) => sum + list.length,
-  );
-  expect(total, expected);
-  // If we had previous placements recorded, ensure they were non-zero to
-  // validate that a reset actually happened when opening a different doc.
-  if (TestWorld.prevPlacementsCount != null &&
-      TestWorld.prevPlacementsCount!.isNotEmpty) {
-    final prevTotal = TestWorld.prevPlacementsCount!.values.fold<int>(
-      0,
-      (sum, v) => sum + v,
-    );
-    expect(prevTotal, greaterThan(0));
-  }
-  // Also verify that signature cards still exist (persistence across open).
-  final cards = c.read(signatureCardRepositoryProvider);
-  expect(cards.length, greaterThanOrEqualTo(1));
-}

test/features/step/page_becomes_visible_in_the_scroll_area.dart

@@ -0,0 +1,14 @@
+import 'package:flutter_test/flutter_test.dart';
+import 'package:flutter_riverpod/flutter_riverpod.dart';
+import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
+import '_world.dart';
+
+/// Usage: page {5} becomes visible in the scroll area
+Future<void> pageBecomesVisibleInTheScrollArea(
+  WidgetTester tester,
+  num param1,
+) async {
+  final page = param1.toInt();
+  final c = TestWorld.container ?? ProviderContainer();
+  expect(c.read(pdfViewModelProvider).currentPage, page);
+}

test/features/step/signature_cards_exist.dart

@@ -1,12 +0,0 @@
-import 'package:flutter_test/flutter_test.dart';
-import 'package:flutter_riverpod/flutter_riverpod.dart';
-import 'package:pdf_signature/data/repositories/signature_card_repository.dart';
-import '_world.dart';
-
-/// Usage: {2} signature cards exist
-Future<void> signatureCardsExist(WidgetTester tester, num param1) async {
-  final expected = param1.toInt();
-  final c = TestWorld.container ?? ProviderContainer();
-  final cards = c.read(signatureCardRepositoryProvider);
-  expect(cards.length, expected);
-}

test/features/step/signature_placements_exist_on_page.dart

@@ -1,17 +0,0 @@
-import 'package:flutter_test/flutter_test.dart';
-import '_world.dart';
-
-/// Usage: {1} signature placements exist on page {2}
-Future<void> signaturePlacementsExistOnPage(
-  WidgetTester tester,
-  num param1,
-  num param2,
-) async {
-  final expected = param1.toInt();
-  final page = param2.toInt();
-  // Record the expectation as part of scenario context instead of asserting
-  // against current state (the scenario describes placements in the previous
-  // document before opening a new one).
-  TestWorld.prevPlacementsCount ??= {};
-  TestWorld.prevPlacementsCount![page] = expected;
-}

test/features/step/the_left_pages_overview_highlights_page.dart

@@ -3,10 +3,12 @@ import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
 import '_world.dart';
 
-/// Usage: the first page of the new document is displayed
+/// Usage: the left pages overview highlights page {5}
-Future<void> theFirstPageOfTheNewDocumentIsDisplayed(
+Future<void> theLeftPagesOverviewHighlightsPage(
   WidgetTester tester,
+  num param1,
 ) async {
+  final n = param1.toInt();
   final c = TestWorld.container ?? ProviderContainer();
-  expect(c.read(pdfViewModelProvider).currentPage, 1);
+  expect(c.read(pdfViewModelProvider).currentPage, n);
 }

test/features/step/the_page_view_mode_is_set_to_continuous.dart

@@ -0,0 +1,8 @@
+import 'package:flutter_test/flutter_test.dart';
+import '_world.dart';
+
+/// Usage: the Page view mode is set to Continuous
+Future<void> thePageViewModeIsSetToContinuous(WidgetTester tester) async {
+  // Logic-level test: no widget tree; just mark a flag if needed
+  TestWorld.prefs['page_view'] = 'continuous';
+}

test/features/step/the_user_drags_handles_to_resize_and_drags_to_reposition.dart

@@ -26,6 +26,6 @@ Future<void> theUserDragsHandlesToResizeAndDragsToReposition(
       height: currentRect.height + 30,
     );
 
-    pdfN.modifyPlacement(page: currentPage, index: 0, rect: newRect);
+    pdfN.updatePlacementRect(page: currentPage, index: 0, rect: newRect);
   }
 }

test/features/step/the_user_drags_it_on_the_page_of_the_document_to_place_signature_placements_in_multiple_locations_in_the_document.dart

@@ -25,7 +25,7 @@ theUserDragsItOnThePageOfTheDocumentToPlaceSignaturePlacementsInMultipleLocation
   if (!container.read(documentRepositoryProvider).loaded) {
     container
         .read(documentRepositoryProvider.notifier)
-        .openDocument(pageCount: 5);
+        .openPicked(pageCount: 5);
   }
 
   container

test/features/step/the_user_drags_this_signature_card_on_the_page_of_the_document_to_place_a_signature_placement.dart

@@ -21,7 +21,7 @@ theUserDragsThisSignatureCardOnThePageOfTheDocumentToPlaceASignaturePlacement(
   if (!container.read(documentRepositoryProvider).loaded) {
     container
         .read(documentRepositoryProvider.notifier)
-        .openDocument(pageCount: 5);
+        .openPicked(pageCount: 5);
   }
 
   // Get or create an asset

test/features/step/the_user_jumps_to_page.dart

@@ -0,0 +1,15 @@
+import 'package:flutter_test/flutter_test.dart';
+import 'package:flutter_riverpod/flutter_riverpod.dart';
+
+import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
+import '_world.dart';
+
+/// Usage: the user jumps to page {2}
+Future<void> theUserJumpsToPage(WidgetTester tester, num param1) async {
+  final page = param1.toInt();
+  final c = TestWorld.container ?? ProviderContainer();
+  try {
+    c.read(pdfViewModelProvider).jumpToPage(page);
+  } catch (_) {}
+  await tester.pump();
+}

test/features/step/the_user_opens_a_different_document_with_pages.dart

@@ -1,43 +0,0 @@
-import 'package:image/image.dart' as img;
-import 'package:flutter_test/flutter_test.dart';
-import 'package:flutter_riverpod/flutter_riverpod.dart';
-import 'package:pdf_signature/data/repositories/document_repository.dart';
-import 'package:pdf_signature/data/repositories/signature_card_repository.dart';
-import 'package:pdf_signature/domain/models/model.dart';
-import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
-import '_world.dart';
-
-/// Usage: the user opens a different document with {3} pages
-Future<void> theUserOpensADifferentDocumentWithPages(
-  WidgetTester tester,
-  num param1,
-) async {
-  final pageCount = param1.toInt();
-  final container = TestWorld.container ?? ProviderContainer();
-  TestWorld.container = container;
-
-  // Simulate "open a different document": reset placements and set page count.
-  container
-      .read(documentRepositoryProvider.notifier)
-      .openDocument(pageCount: pageCount);
-  // Ensure there are 2 signature cards available as per scenario.
-  final cards = container.read(signatureCardRepositoryProvider);
-  if (cards.length < 2) {
-    final notifier = container.read(signatureCardRepositoryProvider.notifier);
-    while (container.read(signatureCardRepositoryProvider).length < 2) {
-      notifier.add(
-        SignatureCard(
-          asset: SignatureAsset(
-            sigImage: img.Image(width: 1, height: 1),
-            name: 'sig.png',
-          ),
-          rotationDeg: 0,
-          graphicAdjust: const GraphicAdjust(),
-        ),
-      );
-    }
-  }
-  // Moving to a new document should show page 1.
-  container.read(pdfViewModelProvider).currentPage = 1;
-  await tester.pumpAndSettle();
-}

test/features/step/the_user_places_it_in_multiple_locations_in_the_document.dart

@@ -12,7 +12,7 @@ Future<void> theUserPlacesItInMultipleLocationsInTheDocument(
   TestWorld.container = container;
   final notifier = container.read(documentRepositoryProvider.notifier);
   // Always open a fresh doc to avoid state bleed between scenarios
-  notifier.openDocument(pageCount: 6);
+  notifier.openPicked(pageCount: 6);
   // Place two on page 2 and one on page 4
   notifier.addPlacement(page: 2, rect: const Rect.fromLTWH(10, 10, 80, 40));
   notifier.addPlacement(page: 2, rect: const Rect.fromLTWH(120, 50, 80, 40));

test/features/step/the_user_savesexports_the_document.dart

@@ -17,7 +17,7 @@ Future<void> theUserSavesexportsTheDocument(WidgetTester tester) async {
     // Load a minimal sample so the expectation passes in logic-only tests
     container
         .read(documentRepositoryProvider.notifier)
-        .openDocument(bytes: Uint8List(10), pageCount: 2, knownPageCount: true);
+        .openPicked(pageCount: 2, bytes: Uint8List(10));
   }
   expect(pdf.loaded, isTrue, reason: 'PDF must be loaded before export');
   // Check if there are placements

test/features/step/the_user_selects.dart

@@ -10,9 +10,7 @@ Future<void> theUserSelects(WidgetTester tester, dynamic file) async {
   final container = ProviderContainer();
   TestWorld.container = container;
   // Mark page for signing to enable signature ops
-  container
+  container.read(documentRepositoryProvider.notifier).openPicked(pageCount: 1);
-      .read(documentRepositoryProvider.notifier)
-      .openDocument(pageCount: 1);
   // For invalid/unsupported/empty selections we do NOT set image bytes.
   // This simulates a failed load and keeps rect null.
   final token = file.toString();

test/features/step/the_user_types_into_the_go_to_input_and_presses_enter.dart

@@ -0,0 +1,19 @@
+import 'package:flutter_test/flutter_test.dart';
+import 'package:flutter_riverpod/flutter_riverpod.dart';
+
+import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
+import '_world.dart';
+
+/// Usage: the user types {3} into the Go to input and presses Enter
+Future<void> theUserTypesIntoTheGoToInputAndPressesEnter(
+  WidgetTester tester,
+  num param1,
+) async {
+  final target = param1.toInt();
+  final c = TestWorld.container ?? ProviderContainer();
+  TestWorld.container = c;
+  try {
+    c.read(pdfViewModelProvider.notifier).jumpToPage(target);
+  } catch (_) {}
+  await tester.pump();
+}

test/features/step/the_user_uses_rotate_controls.dart

@@ -11,6 +11,10 @@ Future<void> theUserUsesRotateControls(WidgetTester tester) async {
   final currentPage = container.read(pdfViewModelProvider).currentPage;
   final placements = pdfN.placementsOn(currentPage);
   if (placements.isNotEmpty) {
-    pdfN.modifyPlacement(page: currentPage, index: 0, rotationDeg: 45.0);
+    pdfN.updatePlacementRotation(
+      page: currentPage,
+      index: 0,
+      rotationDeg: 45.0,
+    );
   }
 }

test/features/step/three_signature_placements_are_placed_on_the_current_page.dart

@@ -22,9 +22,7 @@ Future<void> threeSignaturePlacementsArePlacedOnTheCurrentPage(
   container.read(signatureCardRepositoryProvider.notifier).state = [
     SignatureCard.initial(),
   ];
-  container
+  container.read(documentRepositoryProvider.notifier).openPicked(pageCount: 5);
-      .read(documentRepositoryProvider.notifier)
-      .openDocument(pageCount: 5);
   final pdfN = container.read(documentRepositoryProvider.notifier);
   final page = container.read(pdfViewModelProvider).currentPage;
   pdfN.addPlacement(

test/widget/export_flow_test.dart

@@ -1,11 +1,12 @@
 import 'dart:typed_data';
-import 'package:cross_file/cross_file.dart';
+import 'package:file_selector/file_selector.dart' as fs;
 
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:shared_preferences/shared_preferences.dart';
 
+import 'package:pdf_signature/data/services/export_service.dart';
 import 'package:pdf_signature/ui/features/pdf/view_model/pdf_export_view_model.dart';
 import 'package:pdf_signature/data/repositories/preferences_repository.dart';
 import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
@@ -13,23 +14,30 @@ import 'package:pdf_signature/ui/features/pdf/view_model/pdf_view_model.dart';
 import 'package:pdf_signature/data/repositories/document_repository.dart';
 import 'package:pdf_signature/ui/features/pdf/widgets/pdf_screen.dart';
 import 'package:pdf_signature/l10n/app_localizations.dart';
+import 'package:image/image.dart' as img;
+import 'package:pdf_signature/domain/models/model.dart';
 
-// A fake export VM that always reports success, so this widget test doesn't
+class RecordingExporter extends ExportService {
-// depend on PDF validity or platform specifics.
+  bool called = false;
-bool exported = false;
-
-class _FakePdfExportViewModel extends PdfExportViewModel {
-  _FakePdfExportViewModel(Ref ref)
-    : super(ref, savePathPicker: () async => 'C:/tmp/output.pdf');
-
   @override
-  Future<bool> exportToPath({
+  Future<Uint8List?> exportSignedPdfFromBytes({
-    required String outputPath,
+    required Uint8List srcBytes,
     required Size uiPageSize,
     required Uint8List? signatureImageBytes,
+    Map<int, List<SignaturePlacement>>? placementsByPage,
+    Map<String, img.Image>? libraryImages,
     double targetDpi = 144.0,
   }) async {
-    exported = true;
+    // Return tiny dummy PDF bytes
+    return Uint8List.fromList([0x25, 0x50, 0x44, 0x46]); // "%PDF" header start
+  }
+
+  @override
+  Future<bool> saveBytesToFile({
+    required bytes,
+    required String outputPath,
+  }) async {
+    called = true;
     return true;
   }
 }
@@ -40,6 +48,7 @@ void main() {
   ) async {
     SharedPreferences.setMockInitialValues({});
     final prefs = await SharedPreferences.getInstance();
+    final fake = RecordingExporter();
     await tester.pumpWidget(
       ProviderScope(
         overrides: [
@@ -48,17 +57,18 @@ void main() {
           ),
           documentRepositoryProvider.overrideWith(
             (ref) =>
-                DocumentStateNotifier()..openDocument(
+                DocumentStateNotifier()
-                  bytes: Uint8List(0),
+                  ..openPicked(pageCount: 5, bytes: Uint8List(0)),
-                  pageCount: 5,
-                  knownPageCount: true,
-                ),
           ),
           pdfViewModelProvider.overrideWith(
             (ref) => PdfViewModel(ref, useMockViewer: true),
           ),
           pdfExportViewModelProvider.overrideWith(
-            (ref) => _FakePdfExportViewModel(ref),
+            (ref) => PdfExportViewModel(
+              ref,
+              exporter: fake,
+              savePathPicker: () async => 'C:/tmp/output.pdf',
+            ),
           ),
         ],
         child: MaterialApp(
@@ -67,7 +77,7 @@ void main() {
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile(''),
+            currentFile: fs.XFile(''),
           ),
         ),
       ),
@@ -77,10 +87,10 @@ void main() {
 
     // Trigger save directly (mark toggle no longer required)
     await tester.tap(find.byKey(const Key('btn_save_pdf')));
-    // Pump a bit to allow async export flow to run.
+    await tester.pumpAndSettle();
-    await tester.pump(const Duration(milliseconds: 100));
+
-    await tester.pump(const Duration(milliseconds: 200));
+    // Expect success UI (localized)
-    // Basic assertion: export was invoked
+    expect(find.textContaining('Saved:'), findsOneWidget);
-    expect(exported, isTrue);
+    expect(fake.called, isTrue);
   });
 }

test/widget/helpers.dart

@@ -1,5 +1,5 @@
 import 'dart:typed_data';
-import 'package:cross_file/cross_file.dart';
+import 'package:file_selector/file_selector.dart' as fs;
 import 'package:flutter/material.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:flutter_test/flutter_test.dart';
@@ -36,7 +36,7 @@ Future<void> pumpWithOpenPdf(WidgetTester tester) async {
         home: PdfSignatureHomePage(
           onPickPdf: () async {},
           onClosePdf: () {},
-          currentFile: XFile(''),
+          currentFile: fs.XFile(''),
         ),
       ),
     ),
@@ -413,7 +413,7 @@ Future<void> pumpWithOpenPdfAndSig(WidgetTester tester) async {
         home: PdfSignatureHomePage(
           onPickPdf: () async {},
           onClosePdf: () {},
-          currentFile: XFile(''),
+          currentFile: fs.XFile(''),
         ),
       ),
     ),

test/widget/pdf_navigation_widget_test.dart

@@ -1,4 +1,4 @@
-import 'package:cross_file/cross_file.dart';
+import 'package:file_selector/file_selector.dart' as fs;
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
@@ -38,7 +38,7 @@ void main() {
           home: PdfSignatureHomePage(
             onPickPdf: () async {},
             onClosePdf: () {},
-            currentFile: XFile(''),
+            currentFile: fs.XFile(''),
           ),
         ),
       ),