Skip to content

Comments

feat: Reverse Horizontal Button Option Added#7

Merged
kuraydev merged 1 commit intokuraydev:masterfrom
deLTreeTech:Reverse_Horizontal_Buttons
Apr 10, 2023
Merged

feat: Reverse Horizontal Button Option Added#7
kuraydev merged 1 commit intokuraydev:masterfrom
deLTreeTech:Reverse_Horizontal_Buttons

Conversation

@deLTreeTech
Copy link
Contributor

No description provided.

@kuraydev kuraydev self-requested a review February 4, 2023 10:04
@kuraydev kuraydev self-assigned this Feb 4, 2023
Copy link

@cyril36 cyril36 left a comment

Choose a reason for hiding this comment

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

It would be nice to have this PR merged
is someone able to approve it and merge it ?

@WrathChaos

min?: number;
max?: number;
borderRadius?: number;
reverseButton?: boolean;
Copy link
Owner

Choose a reason for hiding this comment

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

Naming should be different something like reverseCounterButtons. I approved this and will change the naming.

@kuraydev kuraydev merged commit daa9f05 into kuraydev:master Apr 10, 2023
@kuraydev
Copy link
Owner

Version 2 is released with much better integration for this PR. I changed how it should have been implemented, please take a look:

https://github.com/WrathChaos/react-native-counter-input/releases/tag/2.0.0

Also, the whole library is converted to a modern functional component.

@deLTreeTech
@cyril36

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.

3 participants