search

DavBfr / dart_pdf

Pdf creation module for dart/flutter
https://pub.dev/packages/pdf
Apache License 2.0
1.42k stars 632 forks source link

Removed print for fonts with no unicode support #1709

Open MarkOSullivan94 opened 3 months ago

MarkOSullivan94 commented 3 months ago

Resolves #1708

DavBfr commented 3 months ago

Why remove this?

MarkOSullivan94 commented 3 months ago

Why remove this?

Why should a 3rd party dependency be printing out console output in my app without my approval?

DavBfr commented 3 months ago

It's only a debug-build message. And I consider that as something to take care of. The embedded fonts works for testing I don't recommend using them for production application.

MarkOSullivan94 commented 3 months ago

It's only a debug-build message. And I consider that as something to take care of. The embedded fonts works for testing I don't recommend using them for production application.

It's a message I don't care about and if anything it's misleading which has lead to multiple people opening issues misunderstanding the purpose of the message which I already highlighted here: #1708

softwaredementor commented 3 months ago

Hello, is there also a definite resolution for above issue with an example on how to solve it using other fonts?

nickolasdeluca commented 1 month ago

Please merge this, it's a simple thing. Or at least lock it behind a debugMode property or something.

It's really annoying trying to debug something and have this message popup, especially since you can't actually fix it.