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

fix: always trigger the switchChange event after a change event #389

Merged
merged 1 commit into from
Dec 16, 2014

Conversation

frapontillo
Copy link
Contributor

With 2 or more radio buttons, a change to the first radio button propagates correctly, while the change to the second one won't (as it is always skipped). This fixes it by propagating switchChange.bootstrapSwitch after every change.bootstrapSwitch.

Everything else keeps working.

Same as #349 but from the latest develop branch.
This is also needed to close frapontillo/angular-bootstrap-switch#46.

@LostCrew
Copy link
Member

@frapontillo thanks. i will take a look soon!

@LostCrew LostCrew self-assigned this Nov 29, 2014
@LostCrew LostCrew added this to the 3.2.3 milestone Nov 29, 2014
@LostCrew LostCrew added the bug label Nov 29, 2014
LostCrew pushed a commit that referenced this pull request Dec 16, 2014
fix: always trigger the switchChange event after a change event
@LostCrew LostCrew merged commit 88434c5 into Bttstrp:develop Dec 16, 2014
@frapontillo
Copy link
Contributor Author

👍

@LostCrew
Copy link
Member

@frapontillo will release this with v3.3.0.

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.

2 participants