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

Extra points created with the steps option enabled conveys erroneous information #707

Closed
dnschnur opened this issue Sep 28, 2012 · 1 comment
Assignees
Milestone

Comments

@dnschnur
Copy link
Member

Original author: Feldhac...@gmail.com (November 09, 2011 14:45:29)

When using the "steps:true" option, the extra data points that are apparently added in order properly draw the chart with steps are erroneously treated as normal/legitimate data points. Specifically, they are hoverable and clickable when they should not be, thus providing false information when tooltips or click actions are implemented.

Flot v0.7, all browsers.
Similar to issue #258.

Example screenshot attached. The data series only includes the point (14:10, 3), but Flot also adds the point (14:10, 4), which is false data. Not only is incorrect information conveyed in the tooltip depending on where the user hovers, but this chart is part of a master-detail relationship -- clicking on a data point retrieves and displays record details below. By clicking on the (extra) data point (14:10,4), a user would expect to see 4 records displayed when there are only 3.

Possible solution: extra data points added by Flot for drawing purposes could be specially flagged so they are not treated as part of the data series and are not afforded the same attributes as legitimate data points (such as hoverable and clickable, others?).

Original issue: http://code.google.com/p/flot/issues/detail?id=628

@dnschnur
Copy link
Member Author

From vu2...@gmail.com on July 10, 2012 23:23:11
Hi! We've attempted a patch that solves this problem.

The pull request is @
#59

Do let us know if there is a better way of going about it.

dnschnur added a commit that referenced this issue Nov 17, 2013
@ghost ghost assigned dnschnur Nov 17, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant