-
-
Notifications
You must be signed in to change notification settings - Fork 497
feat($compile): adds labelOverlapSeparator as an option #616
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
feat($compile): adds labelOverlapSeparator as an option #616
Conversation
The labelOverlapSeparator option allows the user to specify what string to use to separate the labels when they overlap. The previous hardcoded string ' - ' could indeed be mistaken as a minus sign. To avoid introducing a breaking change, the default is the previous value: ' - '. Tests and README.md modified accordingly.
Codecov Report
@@ Coverage Diff @@
## master #616 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 1 1
Lines 949 949
=====================================
Hits 949 949
Continue to review full report at Codecov.
|
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.
Great job! Just one problem: please remove the npm log file
I have removed the file, would it be a good idea to add it to .gitignore ? |
Indeed, can you add this as well? :) |
Perfect! |
The labelOverlapSeparator option allows the user to specify what string to use to separate the
labels when they overlap. The previous hardcoded string ' - ' could indeed be mistaken as a minus
sign. To avoid introducing a breaking change, the default is the previous value: ' - '. Tests and
README.md modified accordingly.
Demo: http://jsfiddle.net/sLxkwv9s/
(Compare to original problem: http://jsfiddle.net/04sr3ty0/1/)