Fl_chart: Bug: First interval is drawn over top of the border

Created on 20 Jan 2020  路  14Comments  路  Source: imaNNeoFighT/fl_chart

When chart interval lines are enabled the very first interval line is being painted over top of the border. One of the two need to be removed because together they form a bug.

Screen Shot 2020-01-20 at 4 52 17 PM

Fundamental bug

All 14 comments

I didn't get what exactly did you mean

@imaNNeoFighT the first (and last) interval lines are drawn in the same spot as the border. that's why the first line and the last line are red. they're supposed to be grey like the top and bottom border

Got it,
I think it's okay because users can prevent to draw the first and the last one manually.

I suppose we could just draw borders last? It would take care of the issue

No we just can skip the first and the last lines of grid from drawing

Hi @shamilovtim
I think this issue can be closed,
what do you think?

No matter the size of the interval the first interval will always be painted on every users chart so every user who has an interval and a border will experience this bug and will need to write code to handle it. If the border is enabled there's no reason to leave it to the user to stop the interval from being draw over top of the border. So I would say it's still valid and still a bug.

Okay, it makes sense,
you are completely right.
we will skip the first and last interval,
agree?

Yup exactly

Thanks!

Fixed in 0.8.1, thanks for reporting.

There's a regression where this problem appears again but only for the last border

It will be fixed in the new version,
I think we have an issue for this reported by you?

@imaNNeoFighT The other issue caused this regression again, then the other issue got closed.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

MobileMon picture MobileMon  路  4Comments

davidmarinangeli picture davidmarinangeli  路  3Comments

davidmarinangeli picture davidmarinangeli  路  6Comments

caioflores picture caioflores  路  6Comments

satrio1256 picture satrio1256  路  6Comments