Skip to content

Conversation

@styd
Copy link
Contributor

@styd styd commented Jul 15, 2017

Hi! Do we still need Platform here? It seems unnecessary to me since we already have isAndroid from './shared' for that.

@mastermoo
Copy link
Owner

Ah I see, you are a man of clean code, as well
manofculture

@styd
Copy link
Contributor Author

styd commented Jul 18, 2017

Indeed. I bath my code every once in a while. =)

@styd
Copy link
Contributor Author

styd commented Jul 18, 2017

Because, you know, dirty code usually runs a little bit slower than the clean one.

@mastermoo mastermoo merged commit 49c6283 into mastermoo:master Aug 31, 2017
eladgel pushed a commit to HoneyBook/react-native-action-button that referenced this pull request Oct 13, 2017
* commit 'a73637cca3f43fa07d065e48ca62e828e66f3da8':
  2.8.0
  Switch MainButton to use Margin not Padding, and fix shadow prop warning (mastermoo#212)
  Fix shadow style on Android (mastermoo#215)
  Make non-string value/component as title (mastermoo#211)
  Add reminder for linking `react-native-vector-icons`. (mastermoo#214)
  add props onPressIn and onPressOut (mastermoo#216)
  Remove unnecessary import (mastermoo#200)
  2.7.2
  Migrate deprecated  React.PropTypes (mastermoo#195)
  2.7.1
  - revert some changes made in mastermoo#185 - use prettier on code
  2.7.0
  Package of fixes and new features (Read on): (mastermoo#185)
  2.6.10
  Update README.md
  Add support to hide ActionButtonItem and/or its label shadows

# Conflicts:
#	ActionButton.js
#	ActionButtonItem.js
@styd styd deleted the remove-platform branch November 17, 2020 06:18
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.

2 participants