Codecov Report

Merging #2306 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Continue to review full report at Codecov.

Legend - Click here to learn more
D = absolute (impact), o = not affected, ? = missing data
Powered by Codecov. Last update 2eb2df7...fa83099. Read the comment docs.

@rkrabek Thanks for your contribution. Could you add some to tests to ensure this is actually fixing something?

I tried your fix with the following config:

http://jsfiddle.net/moeumaj7/3/

And in this case it's actually break the display.

@panthony Sorry for the delayed response, I've been super busy at work. I've updated the PR to include tests for bar width when tick fit is false.

The config that you provided: http://jsfiddle.net/moeumaj7/3/ fails with my patch for the same reason it fails when you switch tick/fit to true as shown here: http://jsfiddle.net/ox76hL53/
For the sake of not introducing regressions, I've updated the code to take the max of ticks or unique x values so that should address the break in the config you provided.

This patch aims to fix this scenario (with or without the null data points): http://jsfiddle.net/dtttsa9d/