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

Commit

Permalink
fix(select): refresh optionNodes reference
Browse files Browse the repository at this point in the history
Fixes #2109.
  • Loading branch information
ThomasBurleson committed Jun 24, 2015
1 parent c183e9b commit 05119a7
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions src/components/select/select.js
Original file line number Diff line number Diff line change
Expand Up @@ -820,8 +820,11 @@ function SelectProvider($$interimElementProvider) {


function focusOption(direction) {
optionNodes = opts.selectEl[0].getElementsByTagName('md-option');

var optionsArray = $mdUtil.nodesToArray(optionNodes);
var index = optionsArray.indexOf(opts.focusedNode);

if (index === -1) {
// We lost the previously focused element, reset to first option
index = 0;
Expand Down Expand Up @@ -860,11 +863,12 @@ function SelectProvider($$interimElementProvider) {
}

function onRemove(scope, element, opts) {
opts.isRemoved = true;
element.addClass('md-leave')
element
.addClass('md-leave')
.removeClass('md-clickable');
opts.target.attr('aria-expanded', 'false');

opts.target.attr('aria-expanded', 'false');
opts.isRemoved = true;

angular.element($window).off('resize', opts.resizeFn);
angular.element($window).off('orientationchange', opts.resizefn);
Expand Down

0 comments on commit 05119a7

Please sign in to comment.