Skip to content

Commit d65e55a

Browse files
committed
add support table in readme
1 parent 9a613da commit d65e55a

File tree

1 file changed

+15
-3
lines changed

1 file changed

+15
-3
lines changed

README.md

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,11 +62,24 @@ takeSnapshot(viewRef, { path: PictureDir+"/foo.png" })
6262
);
6363
```
6464

65+
## Supported views
66+
67+
Model tested: iPhone 6 (iOS), Nexus 5 (Android).
68+
69+
| System | iOS | Android |
70+
|--------------------|--------------------|-------------------|
71+
| View,Text,Image,.. | YES | YES |
72+
| WebView | YES | NO (crash!) |
73+
| gl-react | YES | NO (empty) |
74+
| react-native-video | NO | NO |
75+
| react-native-maps | YES | [NO](https://github.com/gre/react-native-view-shot/issues/36) |
76+
77+
6578
## Caveats
6679

6780
Snapshots are not guaranteed to be pixel perfect. It also depends on the platform. Here is some difference we have noticed and how to workaround.
6881

69-
- Support of special components like Video / GL views remains untested.
82+
- Support of special components like Video / GL views is not guaranteed to work. In case of failure, the `takeSnapshot` promise gets rejected (the library won't crash).
7083
- It's preferable to **use a background color on the view you rasterize** to avoid transparent pixels and potential weirdness that some border appear around texts.
7184

7285
### specific to Android implementation
@@ -112,8 +125,7 @@ react-native link react-native-view-shot
112125

113126
#### Windows
114127

115-
No support yet. Feel free to PR.
116-
128+
Stay tuned, https://github.com/gre/react-native-view-shot/pull/45 will be merged soon!
117129

118130
## Thanks
119131

0 commit comments

Comments
 (0)