Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

fix(select): auto-select new option that is marked as selected #8106

Merged
merged 1 commit into from
Jul 8, 2014

Conversation

lgalfaso
Copy link
Contributor

@lgalfaso lgalfaso commented Jul 8, 2014

When adding a new element, if the DOM of this option element
states that the element is marked as selected, then select the new

element

Closes #6828

This is a new PR based on a wrong merge done at #7136

When adding a new <option> element, if the DOM of this option element
states that the element is marked as `selected`, then select the new
<option> element

Closes angular#6828
@mary-poppins
Copy link

Thanks for the PR! Please check the items below to help us merge this faster. See the contributing docs for more information.

  • Uses the issue template (#8106)

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@lgalfaso lgalfaso added cla: yes and removed cla: no labels Jul 8, 2014
@lgalfaso
Copy link
Contributor Author

lgalfaso commented Jul 8, 2014

Merging as per #7136 (comment)

lgalfaso added a commit that referenced this pull request Jul 8, 2014
fix(select): auto-select new option that is marked as selected
@lgalfaso lgalfaso merged commit a296097 into angular:master Jul 8, 2014
mgol pushed a commit to mgol/angular.js that referenced this pull request Jul 9, 2014
ckknight pushed a commit to ckknight/angular.js that referenced this pull request Jul 16, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ngOptions : regression 1.2.15 && 1.3.0-beta.3
2 participants