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

Button toggling - trigger change event on input #18900

Merged
merged 1 commit into from
Nov 26, 2016

Conversation

vlastikcz
Copy link

Bootstrap’s .button styles can be applied to other elements, such as labels, to provide checkbox or radio style button toggling.

And when the checkbox or radio state is changed, there should be triggered
the change event. Currently, the change event is triggered on the Button, which is not correct. Only input fields do support the change event.

@wolfy1339
Copy link
Contributor

Please exclude dist file changes

@vlastikcz
Copy link
Author

@wolfy1339 done

Bootstrap’s .button styles can be applied to other elements, such as labels, to provide checkbox or radio style button toggling.

When the checkbox or radio state is changed, there should be triggered the change event. Currently, the change event is triggered on the Button, which is not correct. Only input fields do support the change event.
@vlastikcz
Copy link
Author

are there any other issues with the Pull Request?

@mdo mdo added this to the v4.0.0-alpha.6 milestone Nov 26, 2016
@mdo
Copy link
Member

mdo commented Nov 26, 2016

I believe this fixes #16673 and #17599.

@alexventuraio
Copy link

alexventuraio commented Jul 1, 2018

Hey there!
Has the issue being fixed? Because I still having it on Bootstrap v4.1.1. This is my example for reference.
The radio inputs do not get the checked attribute when clicking on them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants