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

Consider adding onChage method with button values #56

Open
vasilich6107 opened this issue Dec 31, 2021 · 4 comments
Open

Consider adding onChage method with button values #56

vasilich6107 opened this issue Dec 31, 2021 · 4 comments
Labels
enhancement New feature or request help wanted Extra attention is needed in progress

Comments

@vasilich6107
Copy link
Contributor

It would be great to have the api that allows setting selected buttons not by index but with actual buttons values.
For example while using in forms

Also it would be great to have onChange method with selected button values

@Frezyx
Copy link
Owner

Frezyx commented Jan 5, 2022

I'm going to make a GroupButton with possibility of generic value types in 5.0.0 version
The idea is very good! I also need this functionality in my personal projects.

@Frezyx Frezyx added enhancement New feature or request help wanted Extra attention is needed in progress labels Jan 5, 2022
@Frezyx
Copy link
Owner

Frezyx commented Apr 9, 2022

Hello @vasilich6107 !
This feature is available in new 5.0.0 package release

@vasilich6107
Copy link
Contributor Author

@Frezyx this is not what was requested.
The onSelected method gives only the current selected value.
What is required - onChange method with all selected values.

It is not comfortable to operate with indexes.
While using the onSelected method user have to get values by indexes, he also have to handle index deletions rather than simply using the control.

@VIP-Dev-Invisible
Copy link

@Frezyx this is not what was requested. The onSelected method gives only the current selected value. What is required - onChange method with all selected values.

It is not comfortable to operate with indexes. While using the onSelected method user have to get values by indexes, he also have to handle index deletions rather than simply using the control.

I agree too. and @Frezyx please consider adding an option to choose regular old buttons

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed in progress
Projects
None yet
Development

No branches or pull requests

3 participants