From a1e5fa851671a0d95ab678b3cdd2f1082c5e85c4 Mon Sep 17 00:00:00 2001
From: lordfriend <lordfriend@outlook.com>
Date: Sat, 8 Nov 2014 23:43:53 +0800
Subject: [PATCH] add data-size support

---
 README.md                    |  1 -
 examples/app.js              |  4 ++--
 examples/index.html          | 19 ++++++++++++++++++-
 src/nya-bs-select.js         | 29 +++++++++++++++++++++++++++++
 test/spec/dynamic-feature.js |  2 +-
 5 files changed, 50 insertions(+), 5 deletions(-)

diff --git a/README.md b/README.md
index db9fd2f..41e76b0 100644
--- a/README.md
+++ b/README.md
@@ -90,7 +90,6 @@ Like vanilla `<select>` we can also generate option group with any property in a
 
 ##TODO##
 
-- data-size support
 - data-header support
 - data-container support
 - data-max-options support
diff --git a/examples/app.js b/examples/app.js
index 12d02a2..366af03 100644
--- a/examples/app.js
+++ b/examples/app.js
@@ -4,7 +4,7 @@ angular.module('demoApp',['nya.bootstrap.select'])
   .controller('MainCtrl', function($scope){
 
     var options = ['Alpha', 'Bravo', 'Charlie', 'Delta',
-      'Echo', 'Foxtrot', 'Golf', 'Hotel', 'Juliet', 'Kilo', 'Lima',
+      'Echo', 'Foxtrot', 'Golf', 'Hotel', 'India', 'Juliet', 'Kilo', 'Lima',
       'Mike', 'November', 'Oscar', 'Papa', 'Quebec', 'Romeo', 'Sierra',
       'Tango', 'Uniform', 'Victor', 'Whiskey', 'X-ray', 'Yankee', 'Zulu'
     ];
@@ -117,4 +117,4 @@ angular.module('demoApp',['nya.bootstrap.select'])
     $scope.addElement = function(options) {
       $scope[options].unshift(JSON.parse($scope.element));
     };
-  });
\ No newline at end of file
+  });
diff --git a/examples/index.html b/examples/index.html
index 65b4bc7..caaee04 100644
--- a/examples/index.html
+++ b/examples/index.html
@@ -441,6 +441,23 @@ <h3>Custom option content</h3>
       </ol>
     </form>
 
+    <h3>Dropdown Size</h3>
+    <p>set <code>size</code> to an number, which will limit the dropdown menu size to limit number of element. the default value is <code>'auto'</code></p>
+
+    <form class="form-inline">
+      <ol class="nya-bs-select" ng-model="model7" data-size="3">
+        <li class="nya-bs-option" value="a"><a>Alpha<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+        <li class="nya-bs-option" value="b"><a>Bravo<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+        <li class="nya-bs-option" value="c"><a>Charlie<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+        <li class="nya-bs-option" value="d"><a>Delta<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+        <li class="nya-bs-option" value="d"><a>Echo<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+        <li class="nya-bs-option" value="d"><a>Fox<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+        <li class="nya-bs-option" value="d"><a>Golf<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+        <li class="nya-bs-option" value="d"><a>Hotel<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+        <li class="nya-bs-option" value="d"><a>India<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+        <li class="nya-bs-option" value="d"><a>Juliet<span class="glyphicon glyphicon-ok check-mark"></span></a></li>
+      </ol>
+    </form>
   </section>
 </div>
 <footer class="footer">
@@ -459,4 +476,4 @@ <h3>Custom option content</h3>
 <script src="../src/nya-bs-option.js"></script>
 <script src="app.js"></script>
 </body>
-</html>
\ No newline at end of file
+</html>
diff --git a/src/nya-bs-select.js b/src/nya-bs-select.js
index 94705e5..7a2a9cf 100644
--- a/src/nya-bs-select.js
+++ b/src/nya-bs-select.js
@@ -80,6 +80,8 @@ nyaBsSelect.directive('nyaBsSelect', ['$parse', '$document', '$timeout', functio
         var ngCtrl = ctrls[0];
         var nyaBsSelectCtrl = ctrls[1];
 
+        var liHeight;
+
         // for debug
         nyaBsSelectCtrl.setId($element.attr('id'));
 
@@ -237,6 +239,9 @@ nyaBsSelect.directive('nyaBsSelect', ['$parse', '$document', '$timeout', functio
 
         dropdownToggle.on('click', function() {
           $element.toggleClass('open');
+          if($element.hasClass('open') && typeof liHeight === 'undefined') {
+            calcMenuSize();
+          }
           if($attrs.liveSearch === 'true' && $element.hasClass('open')) {
             searchBox.children().eq(0)[0].focus();
           }
@@ -458,6 +463,30 @@ nyaBsSelect.directive('nyaBsSelect', ['$parse', '$document', '$timeout', functio
 
         }
 
+        // will called only once.
+        function calcMenuSize(){
+
+          var liElements = dropdownMenu.find('li'),
+            length = liElements.length,
+            liElement,
+            i;
+          for(i = 0; i < length; i++) {
+            liElement = liElements.eq(i);
+            if(liElement.hasClass('nya-bs-option') || liElement.attr('nya-bs-option')) {
+              liHeight = liElement[0].clientHeight;
+              console.log('liHeight', liHeight);
+              break;
+            }
+          }
+
+          if(/\d+/.test($attrs.size)) {
+            var dropdownSize = parseInt($attrs.size, 10);
+            dropdownMenu.css('max-height', (dropdownSize * liHeight) + 'px');
+            dropdownMenu.css('overflow-y', 'auto');
+          }
+
+        }
+
       };
     }
   };
diff --git a/test/spec/dynamic-feature.js b/test/spec/dynamic-feature.js
index 5275ae5..2f3cebb 100644
--- a/test/spec/dynamic-feature.js
+++ b/test/spec/dynamic-feature.js
@@ -9,7 +9,7 @@ describe('Features contains ngModel, option auto generate, etc;', function() {
 
 
   var options = ['Alpha', 'Bravo', 'Charlie', 'Delta',
-    'Echo', 'Foxtrot', 'Golf', 'Hotel', 'Juliet', 'Kilo', 'Lima',
+    'Echo', 'Foxtrot', 'Golf', 'Hotel', 'India', 'Juliet', 'Kilo', 'Lima',
     'Mike', 'November', 'Oscar', 'Papa', 'Quebec', 'Romeo', 'Sierra',
     'Tango', 'Uniform', 'Victor', 'Whiskey', 'X-ray', 'Yankee', 'Zulu'
   ];