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

[BUG] Min and Max from Time Scale not working properly #4263

Closed
rodrigopavezi opened this issue May 16, 2017 · 3 comments · Fixed by #4556
Closed

[BUG] Min and Max from Time Scale not working properly #4263

rodrigopavezi opened this issue May 16, 2017 · 3 comments · Fixed by #4556

Comments

@rodrigopavezi
Copy link

When setting Min and Max from Time Scale chart options, chart will not show the max time and also not divide the parts into same size steps.

Expected Behavior

Chart should show min and max that was set on the configuration then plot the line between them with a equal size of steps.

Current Behavior

Chat will not show max and parts are not separated by equal steps. See it happening here https://jsfiddle.net/5ekk6b2s/1/

Steps to Reproduce (for bugs)

  1. Create a chart with numerical and time scale data
  2. Configure it to show the numerical data on the Y axis and the time on the X axis
  3. Set min and max with the min and max values from the Array of time values

Context

We have a char that works fine when not setting min and max value besides that it will not start the line on the 0 position an X axis. We would like to see the same as you can see in this example https://jsfiddle.net/zvrqs3vw/3/ but the line starting on 0.

Environment

  • Chart.js version: 2.5.0
  • Browser name and version: Chrome 57.0.2987.133

Cheers

@etimberg
Copy link
Member

It might be a good idea to test this with the latest changes from master to see if it's been fixed by some of the other work we've done

@rodrigopavezi
Copy link
Author

@etimberg I tried using the latest from master but it is still not working properly. It is repeating the max as you can see here
screen shot 2017-05-17 at 10 58 12 am

Cheers

@rodrigopavezi
Copy link
Author

rodrigopavezi commented Jul 25, 2017

@pcsandford FYI. just in case you are going to still use chart.js Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants