Comments (6)
Cool seems like the ball's in their court now. Hope they agree. I will close this thread.
from react-native-webview.
FYI we did contact the authors to ask them if they wanted to contribute as we feel we would all benefit from one package instead of so many: CRAlpha/react-native-wkwebview#191. I will spend some time later to understand if there is still any benefits with using their package, and if so I'll try to implement it in the community package.
WebView being extracted from core does not mean it's support getting dropped, but instead easier contribution from the community
from react-native-webview.
AFAIK React Native's WebView supported WKWebView when this was extracted, so it wasn't needed to look at other WkWebView implementations.
I think it's better to keep this implementation under react-native-community
. It'll also be nice if the maintainers of react-native-wkwebview
want to merge these projects into react-native-community/react-native-webview
so that the work isn't split.
from react-native-webview.
@satya164 @Titozzz thanks for replying so quickly (that too on a Sunday).
WebView being extracted from core does not mean it's support getting dropped, but instead easier contribution from the community
I am not worried about the support being dropped. Moving this out of the core seems to be a step in the right direction.
FYI we did contact the authors to ask them if they wanted to contribute as we feel we would all benefit from one package instead of so many: CRAlpha/react-native-wkwebview#191.
This is great. I hope the authors agree.
I will spend some time later to understand if there is still any benefits with using their package, and if so I'll try to implement it in the community package.
IMO their package is more feature rich. There seem to be issues here already referring to that other repo:
and if so I'll try to implement it in the community package.
In case the owners don't consent to unifying efforts into a single repo, does it make sense to fork the other repo under this organisation instead of implementing it here again or accepting PRs porting individual features?
from react-native-webview.
I think porting individually we will the chosen solution, have you read https://facebook.github.io/react-native/blog/2018/08/27/wkwebview ? There has been a lot merged in 0.57 for the webview, which explains sticking to the react native original implementation. A lot of bug might a been fixed by the new implem, and our work initially here we be to review all the issues IMPORTED and make sure bugs still exists.
from react-native-webview.
@f0rr0 Just FYI:
CRAlpha/react-native-wkwebview#191 (comment)
This project is licensed under MIT, so feel free to take part of the code or fork the whole project. And I am happy to direct the contributors to the new project.
from react-native-webview.
Related Issues (20)
- [v13.10.0] Broken on Windows platform
- Bug: onShouldStartLoadWithRequest not preventing navigation to aliexpress:: links on iOS Prevent deep linking to third party applications Still not solved on IOS HOT 1
- Zoom issue in webview HOT 3
- On Android device not able to auto play video in full screen for embed iframe video url
- Use WebViewCompat.addWebMessageListener instead of @JavascriptInterface to prevent origin spoofing
- Android 10
- Not logging in on iOS initial load HOT 1
- Images from a http:// URL do not load even with mixedContentMode="always" enabled HOT 1
- An SSL error has occurred and a secure connection to the server cannot be made.
- Download is not working for PDF associated with a blob URL.
- Setting connection timeout value
- same error after upgrading to Expo SDK 51 from 49 and rendering `RichEditor` from `react-native-pell-rich-editor`
- Cookies are not being persisted on initial period of seconds of logging in
- The onMessage of the previous webview is not executed when there are multiple webview on a page HOT 1
- node_modules/react-native-webview/android/src/main/java/com/reactnativecommunity/webview/RNCWebViewManager.java:275: error: cannot find symbol view.getSettings().setAppCachePath(ctx.getCacheDir().getAbsolutePath());
- The keyboard is shaking (invisible/visible) on Android only HOT 1
- How do I get Speech recognition permissions in an iOS WebView?
- Migrate Windows to New Architecture
- Sharing session cookies on IOS does not work HOT 2
- Media upload on twitter webview not working
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from react-native-webview.