-
-
Notifications
You must be signed in to change notification settings - Fork 755
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Print number #588
Print number #588
Conversation
4250a7a
to
61c5e2a
Compare
2a9689d
to
dda306a
Compare
packages/plugin-print/src/index.js
Outdated
@@ -263,13 +259,14 @@ export default () => ({ | |||
let alignmentX; | |||
let alignmentY; | |||
|
|||
if (typeof text === 'object') { | |||
if (typeof text === 'object' && text.text) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be :
if (typeof text === 'object' && typeof text.text === "string") {
since we would want : {text:''} , to not error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or maybe text.text != null
@edi9999 I changed it it !== null so a user could still provide non string in the text object |
7b08b12
to
e354ae0
Compare
What's Changing and Why
Allow print to print any type.
closes #568
What else might be affected
Tasks
jimp.d.ts