Avoid that the last label entry in the x-labels "clip" off the edge of the screen #3819

[longup]

Modified the “drawLabels” methon in class XAxisRenderer, i should be i/2 in line 210, so the last label can be handled correctly.

[longup]

Please review this commit and merge to fix this bug, thank you.

[AliGolmirzaei]

I have the same issue and I think this is the fix. please review it. thank you

[almic]

This bug is not caused by the XAxisRenderer. The value you changed, i to i / 2 is checking if the current drawn label is the last label, and if so, it correctly shifts it back width / 2, since the label is already centered and only half clips off.

Please use xAxis.setAvoidFirstLastClipping(true). If that still doesn't fix the problem, you can simply increase the visible X range with xAxis.axisMaximum(float), just add 0.5f or similar to the last X value.

~ nothing to see here ~

[longup]

Hello, I have use xAxis.setAvoidFirstLastClipping(true) and increase the visible X range with xAxis.axisMaximum(float), but it crash, like this:
float lastValue = lineChart.getXAxis().getAxisMaximum(); lineChart.getXAxis().setAxisMaximum(lastValue + 1.0f);

And I have try a lot of values, but it doesn't work. Only if I changed the value i to i / 2, it worked fine.

Thank you，please review it.

[almic]

I just double-checked, and it looks like I was wrong. Line 199 sneaked by!

Anyway, this change looks correct, as the point array is always going to be double the size of mXAxis.mEntryCount, it needs to be divided by 2 when checking for the last entry. Thanks for contributing!

Merging.