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

For the button to add the activeOpacity attribute interface #25

Merged
merged 4 commits into from
May 25, 2016
Merged

For the button to add the activeOpacity attribute interface #25

merged 4 commits into from
May 25, 2016

Conversation

mxh-mo
Copy link
Contributor

@mxh-mo mxh-mo commented May 3, 2016

For the button to add the activeOpacity attribute interface, let the button click effect is more flexible.
Example of the second button Add the attribute test.
Add a description of the property the Readme.md.

mxh-mo added 3 commits May 2, 2016 23:09
add activeOpacity Props interface , let the button pressingStyle more flexible
add activeOpacity props describe
Add the activeOpacity props the test
@alvaromb
Copy link
Collaborator

alvaromb commented May 3, 2016

Thanks for the PR @moxiaohui!

Can you try to squash your commits?

@matteomunaretto
Copy link

Hi there, any news about that? Thanks.

@@ -3,6 +3,12 @@ import Button from 'apsl-react-native-button'

export default class Example extends React.Component {
render () {
var onPressProps;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Better use let if possible.

@alvaromb
Copy link
Collaborator

Yes! sorry folks @moxiaohui @matteomunaretto

I can merge this later today.

@alvaromb alvaromb added this to the 3.0 milestone May 20, 2016
@alvaromb alvaromb self-assigned this May 20, 2016
@alvaromb
Copy link
Collaborator

Merging.

@alvaromb alvaromb merged commit e04628b into APSL:master May 25, 2016
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.

None yet

3 participants