StatusBar Calculation Wrong

[grote]

First of all thanks for this awesome library, I love it! 😄 🙇‍♂️

The latest version introduced a bug (probably by 91e9539).

This is how it looks with 1.4.0

[joseluisametller]

I can reproduce it as well, at least on KitKat. I'm using 1.4.0 for the moment...

[sjwall]

Hi, thanks letting me know about the issue.

It looks like the issue is occurring due to changes made in d70026e, it is incorrectly applying the status bar height when it has already clipped the bounds to the activity.

I hope to have a fix out later today or tomorrow

[sjwall]

Changes released in v1.4.2, could you please confirm that these changes fix this issue?

[joseluisametller]

Using 1.4.2 on my gradle file I can still reproduce it on KitKat.

[sjwall]

The issue is effecting KitKat and lower

[sjwall]

Issue is the call to mClipToView.getLocationOnScreen(location); in updateClipBounds().
It is correctly returning the views screen position from top on KitKat and lower. On Lollipop and newer this returns 0 all the time.

Working on fix for this

[sjwall]

Should be fixed now and released in v1.4.3. Found different way of retrieving the activity screen location. @joseluisametller and @grote could you both please confirm that this fixes the issue for you? I have tested it with Transportr on Kitkat and Nougat, so I have high hopes

[joseluisametller]

@sjwall I can confirm it is working perfectly fine on KitKat. Thanks a lot for your work :D

[sjwall]

@joseluisametller Thanks for confirming. Closing as issue has been fixed