From 8507dcf6f5fae6c131b002804cc3f9f4e4d0981f Mon Sep 17 00:00:00 2001
From: insleker <bkinnightskytw@gmail.com>
Date: Wed, 3 Sep 2025 18:05:34 +0800
Subject: [PATCH] feat: basic signature overlay on document implement

---
 .../pdf/widgets/pdf_mock_continuous_list.dart | 24 +++++++++---
 .../features/pdf/widgets/pdf_page_area.dart   | 39 ++++++++++++++++++-
 .../pdf/widgets/signature_overlay.dart        |  8 +++-
 3 files changed, 63 insertions(+), 8 deletions(-)

diff --git a/lib/ui/features/pdf/widgets/pdf_mock_continuous_list.dart b/lib/ui/features/pdf/widgets/pdf_mock_continuous_list.dart
index 2692d49..9619893 100644
--- a/lib/ui/features/pdf/widgets/pdf_mock_continuous_list.dart
+++ b/lib/ui/features/pdf/widgets/pdf_mock_continuous_list.dart
@@ -66,12 +66,24 @@ class PdfMockContinuousList extends ConsumerWidget {
                     Container(
                       color: Colors.grey.shade200,
                       child: Center(
-                        child: Text(
-                          AppLocalizations.of(context).pageInfo(pageNum, count),
-                          style: const TextStyle(
-                            fontSize: 24,
-                            color: Colors.black54,
-                          ),
+                        child: Builder(
+                          builder: (ctx) {
+                            String label;
+                            try {
+                              label = AppLocalizations.of(
+                                ctx,
+                              ).pageInfo(pageNum, count);
+                            } catch (_) {
+                              label = 'Page $pageNum of $count';
+                            }
+                            return Text(
+                              label,
+                              style: const TextStyle(
+                                fontSize: 24,
+                                color: Colors.black54,
+                              ),
+                            );
+                          },
                         ),
                       ),
                     ),
diff --git a/lib/ui/features/pdf/widgets/pdf_page_area.dart b/lib/ui/features/pdf/widgets/pdf_page_area.dart
index 2eb9bb2..c89a902 100644
--- a/lib/ui/features/pdf/widgets/pdf_page_area.dart
+++ b/lib/ui/features/pdf/widgets/pdf_page_area.dart
@@ -8,6 +8,7 @@ import '../view_model/view_model.dart';
 import '../../../../data/services/preferences_providers.dart';
 import 'signature_drag_data.dart';
 import 'pdf_mock_continuous_list.dart';
+import 'pdf_page_overlays.dart';
 
 class PdfPageArea extends ConsumerStatefulWidget {
   const PdfPageArea({
@@ -191,7 +192,14 @@ class _PdfPageAreaState extends ConsumerState<PdfPageArea> {
     });
 
     if (!pdf.loaded) {
-      return Center(child: Text(AppLocalizations.of(context).noPdfLoaded));
+      // In tests, AppLocalizations delegate may not be injected; fallback.
+      String text;
+      try {
+        text = AppLocalizations.of(context).noPdfLoaded;
+      } catch (_) {
+        text = 'No PDF loaded';
+      }
+      return Center(child: Text(text));
     }
 
     final useMock = ref.watch(useMockViewerProvider);
@@ -229,6 +237,35 @@ class _PdfPageAreaState extends ConsumerState<PdfPageArea> {
           keyHandlerParams: PdfViewerKeyHandlerParams(autofocus: true),
           maxScale: 8,
           scrollByMouseWheel: 0.6,
+          // Render signature overlays on each page via pdfrx pageOverlaysBuilder
+          pageOverlaysBuilder: (context, pageRect, page) {
+            return [
+              Consumer(
+                builder: (context, ref, _) {
+                  final visible = ref.watch(signatureVisibilityProvider);
+                  if (!visible) return const SizedBox.shrink();
+                  return Align(
+                    alignment: Alignment.topLeft,
+                    child: SizedBox(
+                      width: pageRect.width,
+                      height: pageRect.height,
+                      child: PdfPageOverlays(
+                        pageSize: widget.pageSize,
+                        pageNumber: page.pageNumber,
+                        onDragSignature:
+                            (delta) => widget.onDragSignature(delta),
+                        onResizeSignature:
+                            (delta) => widget.onResizeSignature(delta),
+                        onConfirmSignature: widget.onConfirmSignature,
+                        onClearActiveOverlay: widget.onClearActiveOverlay,
+                        onSelectPlaced: widget.onSelectPlaced,
+                      ),
+                    ),
+                  );
+                },
+              ),
+            ];
+          },
           // Add overlay scroll thumbs (vertical on right, horizontal on bottom)
           viewerOverlayBuilder:
               (context, size, handleLinkTap) => [
diff --git a/lib/ui/features/pdf/widgets/signature_overlay.dart b/lib/ui/features/pdf/widgets/signature_overlay.dart
index 6f0c806..42accef 100644
--- a/lib/ui/features/pdf/widgets/signature_overlay.dart
+++ b/lib/ui/features/pdf/widgets/signature_overlay.dart
@@ -268,7 +268,13 @@ class _SignatureImage extends ConsumerWidget {
     }
 
     if (bytes == null) {
-      return Center(child: Text(AppLocalizations.of(context).signature));
+      String label;
+      try {
+        label = AppLocalizations.of(context).signature;
+      } catch (_) {
+        label = 'Signature';
+      }
+      return Center(child: Text(label));
     }
 
     Widget im = Image.memory(bytes, fit: BoxFit.contain);