Skip to content
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

node.fontSize value now overrides config.node.fontSize value #376

Conversation

lironhl
Copy link
Contributor

@lironhl lironhl commented Oct 4, 2020

closes #330

This fixes the issue where individual nodes couldn't get specific fontSize or highlightFontSize values.

Here's the before & after of the behaviors.

Before

fontSizeBefore

After

fontSizeAfter

@lironhl lironhl force-pushed the fix/node_font_size_overrides_config_node_font_size branch from 1b1617c to 3dc029d Compare October 4, 2020 18:51
@lironhl lironhl force-pushed the fix/node_font_size_overrides_config_node_font_size branch from 3dc029d to 27abe77 Compare October 4, 2020 19:00
Copy link
Owner

@danielcaldas danielcaldas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks great! Thanks @lironhl 👍

@danielcaldas danielcaldas merged commit c6c76a1 into danielcaldas:master Oct 7, 2020
@danielcaldas danielcaldas added the Hacktoberfest Hacktoberfest label label Oct 7, 2020
@danielcaldas danielcaldas added the hacktoberfest-accepted Label for accepted PRs for Hacktoberfest 2020 label Oct 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hacktoberfest Hacktoberfest label hacktoberfest-accepted Label for accepted PRs for Hacktoberfest 2020
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fontSize parameter should be working for single node
2 participants