You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Mar 15, 2024. It is now read-only.
When using loadAssetIntoView for a video, Flurry will add two views to the viewgroup passed into the method. However, due to an internal behavior, one of these views will overwrite its own layout params with a RelativeLayout.LayoutParams in onConfigurationChanged(). Because of this, if the viewgroup passed in is not a RelativeLayout, it will crash whenever configuration changes (e.g. rotation).
I suggest at the very least documenting this behavior within the loadAssetIntoView documentation, but I additionally suggest either enforce runtime checks telling integrating devs of this requirement or getting rid of this behavior altogether.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
When using
loadAssetIntoView
for a video, Flurry will add two views to the viewgroup passed into the method. However, due to an internal behavior, one of these views will overwrite its own layout params with aRelativeLayout.LayoutParams
inonConfigurationChanged()
. Because of this, if the viewgroup passed in is not a RelativeLayout, it will crash whenever configuration changes (e.g. rotation).I suggest at the very least documenting this behavior within the
loadAssetIntoView
documentation, but I additionally suggest either enforce runtime checks telling integrating devs of this requirement or getting rid of this behavior altogether.The text was updated successfully, but these errors were encountered: