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

refactor: general refactor about spread usage, menu item, optional icon component #38

Merged
merged 4 commits into from
Aug 7, 2021

Conversation

enesozturk
Copy link
Owner

This PR includes several fixes and refactors in the following;

  • Removed spread operator usage from menu styles because of a issue in Reanimated.
  • Refactored menu item and created worklet for calculating menu item text and icon color.
  • Refactored Icon component as optional that provided to HoldMenuProvider component.

@enesozturk enesozturk merged commit 0fb5284 into develop Aug 7, 2021
@enesozturk enesozturk deleted the refactor/general branch August 7, 2021 14:09
enesozturk added a commit that referenced this pull request Aug 7, 2021
…mponent (#38)

* fix: remove spread operator usage in styles

* refactor: add worklet for color calculation

* refactor: use color worklet in menu item comp

* refactor: make icon component props optional
enesozturk added a commit that referenced this pull request Aug 7, 2021
…mponent (#39)

* fix: set icon's color is same with text (#33)

* Set icon's color is same with text

Co-authored-by: @booyeu

* fix Synchronously call function error (#34)

* fix Synchronously call function error

* fix Synchronously call function error

* fix Synchronously call function error

Co-authored-by: @booyeu

* fix: general refactor about spread usage, menu item, optional icon component (#38)

* fix: remove spread operator usage in styles

* refactor: add worklet for color calculation

* refactor: use color worklet in menu item comp

* refactor: make icon component props optional

Co-authored-by: 刘力瑞 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant