BackgroundPopUplabel never show if enableYAxisLabel is set NO

[MattiaConfalonieri]

Hi,
First, thank four the library.
secondo, i have problem with "alwaysDisplayPopUpLabels", if i set it on YES but also set enableYAxisLabel YES, BackgroundPopUplabel is not visible.

If i set BackgroundPopUplabel to NO, everything work fine.

You can reply your issue in your sampleProject adding
self. myGraph.alwaysDisplayPopUpLabels = YES;

Is that normal?

[Boris-Em]

Hi @MattiaConfalonieri,
This is not normal, they should all be displayed. I will take a look a this bug tonight, and hopefully come up with a fix.
Thank you for pointing this out!

[Boris-Em]

This issue should be fixed with the latest commit 3da4fb7. @MattiaConfalonieri, could you confirm that this bug doesn't occur anymore in order to be able to close this issue?
Thanks!

[MattiaConfalonieri]

CONFIRMED!!
Great work, thanks very much.

EDIT:
I know that the library is not optimist for iOS8, but just for know:

In iOS PopUpLabels are showed double after that fix, i say after that fix, because in the sample app this issue not appear

EDIT 2
the problem is in the line

if ([self checkOverlapsForView:permanentPopUpView] == YES) {
permanentPopUpLabel.center = CGPointMake(self.xCenterLabel, circleDot.center.y + circleDot.frame.size.height/2 + 15);
}

because if i comment it everything work great.

I hope that's help.

excuse me, but i am new in GitHub and i don't know how help with commit and this things.

[Boris-Em]

Thank you!
Just to make sure, regarding your first edit, the issue with the popup labels displaying as double appear only in iOS8?

[MattiaConfalonieri]

Yeah i confirm, popup labels displaying as double appear only in iOS8 (GM).

[Boris-Em]

Alright, I will close this issue and open a new one specifically for this.
Thanks again for spotting this. We need to support iOS8 as soon as possible!