Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

Commit

Permalink
Merge pull request #1040 from hightower/bugfix/focus-on-activate
Browse files Browse the repository at this point in the history
Use opacity, not visibility to hide element
  • Loading branch information
dimirc committed Jul 28, 2015
2 parents 2d1422d + 58de0d7 commit 76bf9da
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/uiSelectDirective.js
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ uis.directive('uiSelect',
}

// Hide the dropdown so there is no flicker until $timeout is done executing.
dropdown[0].style.visibility = 'hidden';
dropdown[0].style.opacity = 0;

// Delay positioning the dropdown until all choices have been added so its height is correct.
$timeout(function(){
Expand All @@ -277,7 +277,7 @@ uis.directive('uiSelect',
}

// Display the dropdown once it has been positioned.
dropdown[0].style.visibility = '';
dropdown[0].style.opacity = 1;
});
} else {
if (dropdown === null) {
Expand Down

0 comments on commit 76bf9da

Please sign in to comment.