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

Android leak #448

Closed
everfire130 opened this issue Mar 31, 2020 · 10 comments
Closed

Android leak #448

everfire130 opened this issue Mar 31, 2020 · 10 comments

Comments

@everfire130
Copy link

everfire130 commented Mar 31, 2020

main dependencies:
 "react": "16.11.0",
 "react-native": "0.62.0",
 "react-navigation": "4.3.5",
 "react-navigation-hooks": "1.1.0",
 "react-navigation-stack": "2.3.9",
 "react-navigation-tabs": "2.8.9",
 "react-native-screens": "2.4.0",

and after I have added this code to app.js.

import {enableScreens} from 'react-native-screens';
enableScreens();

and use pushnavigate to route in some pages
I got the leak warnings
WechatIMG1188
WechatIMG1189

@InvictusGar
Copy link

same problem

@WoLewicki
Copy link
Member

Can you provide a repo with minimal configuration needed to reproduce the issue?

@tanjwGit
Copy link

webview 开启硬件加速,安卓9 10 可以百分百重现 @WoLewicki

@WoLewicki
Copy link
Member

@tanjiaweiGitHub could you write in English? I'm sorry but I don't understand what is written there.

@InvictusGar
Copy link

he means to set webview hardwareAccelerated=true, and then this leak would appear @WoLewicki

@tanjwGit
Copy link

enableScreens() and hardwareAccelerated=true

Open any webpage through webView, then exit webview, click on any @WoLewicki

@tanjwGit
Copy link

By adding the style opacity: 0.99 you can avoid the crash problem, but I think this is not a long-term plan @WoLewicki

@tanjwGit
Copy link

And I encountered the problem of flickering after page loading when switching pages, which was temporarily solved by setting opacity: 0.99

@WoLewicki
Copy link
Member

There seem to be some problems with the react-native-webview and react-native-screens integration, but I am not sure that it should be solved anyhow on our side. If you have any ideas, I would love to check PR with such changes.

@WoLewicki
Copy link
Member

I am closing this issue to keep the discussion about react-native-webview in one place. Please see #105 for more information.

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

No branches or pull requests

4 participants