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
Add pointStyle false #10758
Add pointStyle false #10758
Conversation
@etimberg Can you review it now? |
@etimberg String to bool converted, not sure If every place needs a bool false. Let me know if any other changes are needed, would be glad to update you. Haha :) |
Isn't radius: 0 doing this already? |
@kurkle yes it is, but feature was requested and tagged as enhancement. |
@etimberg @kurkle Updated the tests, but still getting an error, maybe you could help me with that. |
@@ -16,11 +16,12 @@ module.exports = { | |||
'dash', | |||
'line', | |||
'rect', | |||
'false' |
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.
shouldn't this be
'false' | |
false |
@@ -16,11 +16,12 @@ module.exports = { | |||
'dash', | |||
'line', | |||
'rect', | |||
'false' |
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.
Same as above
@@ -1,10 +1,10 @@ | |||
var gradient; | |||
|
|||
var datasets = ['circle', 'cross', 'crossRot', 'dash', 'line', 'rect', 'rectRounded', 'rectRot', 'star', 'triangle'].map(function(style, y) { | |||
var datasets = ['circle', 'cross', 'crossRot', 'dash', 'line', 'rect', 'rectRounded', 'rectRot', 'star', 'triangle', 'false'].map(function(style, y) { |
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.
Same
@@ -1,4 +1,4 @@ | |||
const pointStyles = ['circle', 'cross', 'crossRot', 'dash', 'line', 'rect', 'rectRounded', 'rectRot', 'star', 'triangle']; | |||
const pointStyles = ['circle', 'cross', 'crossRot', 'dash', 'line', 'rect', 'rectRounded', 'rectRot', 'star', 'triangle', 'false']; |
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.
Same
This error usually happens when the window running the tests is not active (Chrome does not fire events when hidden) Looks like you still need to update this one: |
@puneetkathar1 will you be able to update this? |
Sorry, I'm getting bunch of errors. I don't think I'll be able to do it.
…On Sun, 13 Nov 2022, 01:24 Jukka Kurkela, ***@***.***> wrote:
@puneetkathar1 <https://github.com/puneetkathar1> will you be able to
update this?
—
Reply to this email directly, view it on GitHub
<#10758 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ARPD5WVRULAKEIS5UIYV7JTWH7YY7ANCNFSM6AAAAAAQ6XJSX4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Added feature - #10755