Add option 'limit' to limit the number of items that can be selected

[fdevibe]

In multiple mode, use this option to limit the number of items that can be selected at the same time.

Quite possibly a fairly naïve approach. The tests also aren't optimal, but I haven't found a better way to test them without touching more code. At least it is a suggestion. :-)

This addresses #178.

[YidingW]

+1

[amoshche]

At the moment the workaround is extending the directive, it also work with mouse clicks

    .directive('uiSelect', function (){
      return {
        restrict: 'EA',
        require: 'uiSelect',
        link: function($scope, $element, $attributes, ctrl) {
            $scope.$select.limit = (angular.isDefined($attributes.limit)) ? parseInt($attributes.limit, 10) : undefined;
            var superSelect = ctrl.select;
            ctrl.select = function() {
                if(ctrl.multiple && ctrl.limit !== undefined && ctrl.selected.length >= ctrl.limit) {
                    console.log("booooo");
                } else {
                    superSelect.apply(ctrl, arguments);
                }
            };
        }
      }
    })

[yaneq]

Thanks for this @amoshche

[maxisam]

+1 Thanks for this @amoshche

[davidstump]

👍 Thanks @amoshche!

[evillemez]

👍 Yeah, thanks @amoshche!

[KennyKaplanHP]

Rather than doing another directive I adjusted the ctrl.activate function by adding in an if block around the remainder of the function. Using the attribute set logic from @fdevibe to add the limit to $select you can use the following to prevent the dropdown list from showing up:

if (ctrl.limit === undefined || ctrl.selected.length < ctrl.limit)

[icfantv]

I don't see this added yet to the current code base - probably due to the merge conflict above. Is there any intent of adding this feature to the released version? Thanks.

[voznik]

@amoshche +1 Thanks

[user378230]

Superseded by #1110 (merged)