Skip to content

Commit 78251fc

Browse files
committed
Updated with feedback from @axe-fb
1 parent 7540914 commit 78251fc

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

proposals/0001-template.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,11 @@ This would move it to an external package, like so:
2525
import { WebView } from '@infinitered/react-native-webview';
2626
```
2727

28+
(Where this third-party package will live is an open question. See below.)
29+
2830
## Motivation
2931

30-
The current WebView implementation is (to my knowledge) not used by Facebook internally. This leads (understandably) to a lack of first-class support for this core API. By moving the component to a third party that is more invested in supporting it, there will be better support.
32+
The current WebView implementation is used by Facebook internally but not actively updated. This leads (understandably) to a lack of first-class support for this core API. By moving the component to a third party that is more invested in supporting it, there will be better support.
3133

3234
Additionally, with this component no longer included in core, there will be an opening for alternative third party WebView implementations, should anyone else wish to provide one.
3335

@@ -57,4 +59,6 @@ We will need to update the React Native documentation to point out that WebView
5759

5860
## Unresolved questions
5961

60-
None so far.
62+
There are a number of [WebView-related issues](https://github.com/facebook/react-native/search?q=webview&state=open&type=Issues) in the core React Native repo. We will need to have a plan to migrate those over to the new WebView repo.
63+
64+
Additionally, we need to decide where this new repo will live. I am willing to host it under the `infinitered` organization (this motivates me more to keep it up to date and supported), but am open to other ideas such as `react-native-community`.

0 commit comments

Comments
 (0)