From 5db364d1fc4d64eb3f1c42848c23530d3bbc9d16 Mon Sep 17 00:00:00 2001 From: Akiba So Date: Tue, 23 Jun 2026 09:28:18 +0800 Subject: [PATCH] feat(route): open PDFs in pen-first editor The night's pen-first rebuild (PenEditorScreen + PenCanvas + zoom fix + eraser + M3 tool palette + render cache) was unreachable from the running app: home_screen opened the OLD PdfAnnotatorScreen, so the user saw zero change. Wire both PDF-open sites (import + open-existing) to PenEditorScreen, making the entire editor/* stack LIVE on the real PDF path. flutter analyze: 0 issues. Co-Authored-By: Claude Opus 4.8 (1M context) --- lib/screens/home_screen.dart | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/screens/home_screen.dart b/lib/screens/home_screen.dart index d5b71c8..a379a3e 100644 --- a/lib/screens/home_screen.dart +++ b/lib/screens/home_screen.dart @@ -5,11 +5,11 @@ import '../models/note.dart'; import '../providers/document_provider.dart'; import '../providers/note_provider.dart'; import '../providers/ocr_provider.dart'; +import '../editor/canvas/pen_editor_screen.dart'; import '../editor/pdf/spike_launcher.dart'; import '../services/pdf_service.dart'; import '../services/pptx_service.dart'; import 'note_editor_screen.dart'; -import 'pdf_annotator_screen.dart'; import 'ppt_annotator_screen.dart'; import 'search_screen.dart'; import 'settings_screen.dart'; @@ -204,7 +204,7 @@ class HomeScreen extends ConsumerWidget { if (filePath != null && context.mounted) { Navigator.of(context).push( MaterialPageRoute( - builder: (_) => PdfAnnotatorScreen(filePath: filePath), + builder: (_) => PenEditorScreen(pdfPath: filePath), ), ); } @@ -532,7 +532,7 @@ class _DocumentTileState extends ConsumerState<_DocumentTile> { ); } - // [L2] Route by docType: pdf → PdfAnnotatorScreen, ppt/pptx → PptAnnotatorScreen + // [L2] Route by docType: pdf → PenEditorScreen (pen-first), ppt/pptx → PptAnnotatorScreen Future _openDocument(BuildContext context) async { final document = widget.document; final isPdf = document.docType == 'pdf'; @@ -540,7 +540,7 @@ class _DocumentTileState extends ConsumerState<_DocumentTile> { if (isPdf) { Navigator.of(context).push( MaterialPageRoute( - builder: (_) => PdfAnnotatorScreen(filePath: document.filePath), + builder: (_) => PenEditorScreen(pdfPath: document.filePath), ), ); } else {