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

[linear-gradient] Migrate to functional components #8673

Merged

Conversation

EvanBacon
Copy link
Contributor

Why

How

  • migrate to functional components
  • unify native types
  • add doc blocks
  • improve docs
  • fix typo LinearGradienPoint -> LinearGradientPoint (breaking)

Test Plan

  • Snapshot tests return the same rendered results.
  • NCL web preview should still work.

@EvanBacon EvanBacon self-assigned this Jun 5, 2020
@EvanBacon EvanBacon requested a review from ide June 5, 2020 05:41
@github-actions
Copy link
Contributor

github-actions bot commented Jun 5, 2020

Native Component List for this branch is ready

@EvanBacon EvanBacon marked this pull request as ready for review June 5, 2020 05:49
docs/pages/versions/unversioned/sdk/linear-gradient.md Outdated Show resolved Hide resolved
docs/pages/versions/unversioned/sdk/linear-gradient.md Outdated Show resolved Hide resolved
docs/pages/versions/unversioned/sdk/linear-gradient.md Outdated Show resolved Hide resolved
docs/pages/versions/unversioned/sdk/linear-gradient.md Outdated Show resolved Hide resolved
docs/pages/versions/unversioned/sdk/linear-gradient.md Outdated Show resolved Hide resolved
packages/expo-linear-gradient/src/LinearGradient.tsx Outdated Show resolved Hide resolved
@EvanBacon EvanBacon merged commit 257e887 into master Jun 8, 2020
@EvanBacon EvanBacon deleted the @evanbacon/linear-gradient/migrate-to-functional-components branch June 8, 2020 20:18
esamelson pushed a commit that referenced this pull request Jun 24, 2020
* Remove index re-export

* Created shim element for unsupported platforms

* Convert LinearGradient to functional component

* Convert all react imports to star imports

* Unify types and add doc blocks

* Update NativeLinearGradient.web.tsx

* Updated build files

* Updated docs for linear gradient

* Update linear-gradient.md

* Update CHANGELOG.md

* Use nullish

* Clean up function export

* Apply suggestions from code review

Co-authored-by: James Ide <ide@users.noreply.github.com>

* Updated docs

Co-authored-by: James Ide <ide@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

with-storybook demo / expo-linear-gradient breaks with react-native-web 0.12.2
2 participants