Skip to content
This repository was archived by the owner on Aug 12, 2021. It is now read-only.

Conversation

gensc004
Copy link

@gensc004 gensc004 commented Apr 13, 2017

  • Added an image property that can be used by React Native to force an update to the native view with the new function setImage(image) where image is a base64 encoded string.
  • Allowed for transparent background, and switched the default background from blackColor to clearColor.

Similar to what @stantoncbradley did in response to #6, I wanted to be able to draw on top of an image, and I also wanted to add the ability to undo lines that were drawn. I am working in RN ~0.40, so I opened this up against the react-native-0.40 branch.
sketch

@dkretsch12
Copy link

👍

@jgrancher
Copy link
Owner

jgrancher commented Apr 28, 2017

Hey @gensc004,
Thanks for your help! Would you be able to rebase your changes from master please? :)

@gensc004
Copy link
Author

Absolutely!

@jgrancher
Copy link
Owner

Hey @gensc004! I've finally merged #28, which means the module should now works on RN >= 0.40, directly from master. Would you be able to change your destination branch to be master, and rebase? Thanks! 💃

@bshyong
Copy link

bshyong commented Sep 1, 2017

^ looks great! would love to use this once it is merged

@jgrancher jgrancher mentioned this pull request Sep 12, 2017
@hnryjms
Copy link
Contributor

hnryjms commented Dec 28, 2017

My coworker @gensc004 and I cleaned up this PR with replacement PR #33, so we can close this one and continue discussing things over there.

@gensc004 gensc004 closed this Dec 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants