-
Notifications
You must be signed in to change notification settings - Fork 44
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
Use Vega-Lite layering to support tooltip on line charts and area charts #1
Comments
This will be partially addressed by fixing vega/vega-lite#508, but we should wait until we have layering support first. |
One thing that we can do now is
|
Partially fixed in PR #16, which only displays qualitative fields for line charts and area charts, and gives warnings in the console. |
Hmm. If it is partially fixed and should be further fixed, I don't think we should close it. |
Dropping quantitative fields does seem to solve the problem for the As a better partial fix, we may drop fields in the x and y channels and only show fields in the other encoding channels (typically color). In this way, if a qualitative field is not used for encoding, it will not show up in the tooltip. I am writing down this thought for logging purpose. I know we will have layering really soon, so we may not even need to implement this "better partial fix" -- we may go straight into "using layering to properly show all fields" for line charts and area charts. I wrote down these thoughts as comments in PR #16. |
Close via #50 |
Currently tooltip for line chart does not report correct data because data is bound to path, not to points. In the screenshot below, price should be a number around 200 and date should be in human-readable format (that's another issue).
The text was updated successfully, but these errors were encountered: