Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

分离生命周期和创建浮动框,需要在Application中注册生命周期。 #111

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

baneyue
Copy link

@baneyue baneyue commented Aug 29, 2019

这样就可以在用到浮动框的地方才去创建浮动框。
解决必须在Application中创建浮动框,否则会出问题的bug。
在Applicaion中先注册生命周期:FloatWindow.initLifecycle(this);

西瓜智选股有采用这个库,接入时我们是在Activity中取创建浮层,发现实现不了文档所说的指定页面显示浮层,经过调试发现在Activity中创建浮层会出现生命周期问题,所有我把生命周期注册的代码分离了下,改动非常小。

 这样就可以在用到浮动框的地方才去创建浮动框。
 解决必须在Application中创建浮动框,否则会出问题的bug。
 在Applicaion中先注册生命周期:FloatWindow.initLifecycle(this);
@baneyue
Copy link
Author

baneyue commented Aug 29, 2019

麻烦审阅代码和验证

优化浮框消耗时的内存泄漏
解决浮框销毁的内存泄漏
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant