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 inline radio element used with optionLabelPath (BS3) #766

Merged
merged 1 commit into from Mar 5, 2019

Conversation

kumy
Copy link
Contributor

@kumy kumy commented Mar 4, 2019

I've not used this part, but found something that looks like an issue while exploring the code.

@jelhan
Copy link
Contributor

jelhan commented Mar 4, 2019

Good catch! Was introduced by 508ae1f. Seems like we are missing test coverage for the combination of having an optionLabelPath and inline=true.

Copy link
Contributor

@simonihmig simonihmig left a comment

Choose a reason for hiding this comment

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

Thanks again!

jelhan added a commit to jelhan/ember-bootstrap that referenced this pull request Mar 5, 2019
A test for BS4 is not needed cause `inline=true` only affects rendering
an additional class for that one.
@simonihmig simonihmig changed the title Fix typo in radio.hbs template Fix inline radio element used with optionLabelPath (BS3) Mar 5, 2019
@jelhan
Copy link
Contributor

jelhan commented Mar 5, 2019

@kumy Opened a pull request against your branch to add missing tests for this case: kumy#1

@simonihmig simonihmig added the bug label Mar 5, 2019
@simonihmig
Copy link
Contributor

@jelhan thanks for putting the test together. I think I will merge this quickly, as the fix itself is fine and I wanted to cut a release today. Can you target your PR to the main repo's master instead?

@simonihmig simonihmig merged commit 49357fe into ember-bootstrap:master Mar 5, 2019
@jelhan
Copy link
Contributor

jelhan commented Mar 5, 2019

@simonihmig Sure. Please find it as #768.

simonihmig added a commit that referenced this pull request Mar 5, 2019
test(BsForm): adds failing test for pull request #766
@kumy kumy deleted the patch-2 branch March 5, 2019 13:07
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.

None yet

3 participants