-
Notifications
You must be signed in to change notification settings - Fork 11.9k
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 option to include invisible points #10362
Conversation
Haven't looked at code, but the new option should also be documented in the documentation itself instead of only in the typings |
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.
Only doubt is the option name. It only includes invisible points outside chart area (and those could still be visible if clipping is disabled). But it does not include hidden points inside chart area, right?
I just added an option to suppress |
Entire dataset can be hidden by adding I fail to come up with a better name for the option, so maybe just update the description to mention "outside chartArea"? |
Co-authored-by: Jacco van den Berg <[email protected]>
Co-authored-by: Jacco van den Berg <[email protected]>
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'm ok with includeInvisible
as the param name. I didn't notice anything with the code
Fixes #10361.