-
Notifications
You must be signed in to change notification settings - Fork 525
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
Fixes #469 - query results error out if no custom font is set #470
Conversation
- if no custom font has been set from the settings modal, the app crashes. - this fixes the issue by defaulting the custom font to the default font
Not sure what this build failure is about -- seems maybe related to a library upgrade? |
Ah, looks like the Travis build is running on node 6 - |
const { customFont } = this.props.config.data.customFont; | ||
const font = `14px '${customFont}', 'Helvetica Neue', Arial, Helvetica, sans-serif`; | ||
const customFont = this.props.config.data.customFont || 'Helvetica Neue'; | ||
const font = `14px '${customFont}' 'Helvetica Neue', Arial, Helvetica, sans-serif`; |
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.
You removed the ,
from the customFont. I think the comma is required there.
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.
yes you are correct. I will fix that.
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.
LGTM 👍
Manually merged in df70052 |
Fixes #469