From 022ded2a3e6979a0fa7dd456d6f4b113e57293e7 Mon Sep 17 00:00:00 2001
From: Ashita Prasad <ashitaprasad92@gmail.com>
Date: Tue, 14 Jan 2025 09:50:11 +0530
Subject: [PATCH] Update platform_specific_instructions.md
---
doc/dev_guide/platform_specific_instructions.md | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/doc/dev_guide/platform_specific_instructions.md b/doc/dev_guide/platform_specific_instructions.md
index 7752933c..4ee8398b 100644
--- a/doc/dev_guide/platform_specific_instructions.md
+++ b/doc/dev_guide/platform_specific_instructions.md
@@ -44,16 +44,21 @@ For more information on multidex support, you can refer to the Android developer
### Web
-If you're building a Flutter app for the web using the `printing` package, you may encounter a build error like:
+If you're building a Flutter app for the web, you may encounter a build error like:
```
-Error: A value of type 'JSString' can't be assigned to a variable of type 'String'.
+Launching lib/main.dart on Chrome in debug mode...
+../../../.pub-cache/hosted/pub.dev/printing-5.13.4/lib/printing_web.dart:218:16: Error:
+A value of type 'JSString' can't be assigned to a variable of type 'String'.
+ .toJS;
+ ^
+Failed to compile application.
```
This happens because `.toJS` is no longer required for converting Dart strings to JavaScript strings in recent Dart versions.
**Fix:**
-Update the `printing_web.dart` file in the `printing` package by removing `.toJS` from:
+Update the `printing_web.dart` file in the cached `printing` package by removing `.toJS` as done in the PR [here](https://github.com/DavBfr/dart_pdf/pull/1739/files)
```dart
script.innerHTML =
@@ -67,5 +72,4 @@ script.innerHTML =
'''function ${_frameId}_print(){var f=document.getElementById('$_frameId');f.focus();f.contentWindow.print();}''';
```
-This fix ensures compatibility with the latest Dart runtime. For more details, refer to [GitHub Issue #1791](https://github.com/DavBfr/dart_pdf/issues/1791).
Read more about it here - https://github.com/DavBfr/dart_pdf/issues/1791