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

🎨 Improve Button component stylesheet; removed duplicate styles #21535

Closed
wants to merge 1 commit into from
Closed

🎨 Improve Button component stylesheet; removed duplicate styles #21535

wants to merge 1 commit into from

Conversation

cHaLkdusT
Copy link
Contributor

@cHaLkdusT cHaLkdusT commented Oct 6, 2018

Updated Button component inspiration. Refactored code takes advantage of Platform Specific Code, as we want to re-use as much code as possible.

Test Plan

  1. Import Button component
  2. Button should work the same

Related PRs

#19752

Release Notes

[GENERAL] [ENHANCEMENT] [Button] - Uses spread operator for platform specific code in creating styles

@cHaLkdusT cHaLkdusT changed the title 🎨 Improve Button component stylesheet; removed duplicate styles 🎨 Improve Button component stylesheet; removed duplicate styles Oct 6, 2018
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 6, 2018
@cHaLkdusT
Copy link
Contributor Author

Creating a new PR to avoid rebase issue. #19752

@facebook-github-bot facebook-github-bot added the Import Started This pull request has been imported. This does not imply the PR has been approved. label Oct 8, 2018
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TheSavior is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@react-native-bot
Copy link
Collaborator

@cHaLkdusT merged commit bbb6a07 into facebook:master.

@facebook facebook locked as resolved and limited conversation to collaborators Oct 9, 2018
@react-native-bot react-native-bot added the Merged This PR has been merged. label Oct 9, 2018
@hramos hramos removed Import Started This pull request has been imported. This does not imply the PR has been approved. labels Feb 6, 2019
t-nanava pushed a commit to microsoft/react-native-macos that referenced this pull request Jun 17, 2019
…book#21535)

Summary:
Updated Button component inspiration. Refactored code takes advantage of Platform Specific Code, as we want to re-use as much code as possible.

1. Import Button component
2. Button should work the same

facebook#19752

[GENERAL] [ENHANCEMENT] [Button] - Uses spread operator for platform specific code in creating styles
Pull Request resolved: facebook#21535

Differential Revision: D10248048

Pulled By: TheSavior

fbshipit-source-id: 7260fa56f15b70b7c9499c8da418db7b2214b0dd
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants