This repository was archived by the owner on Sep 5, 2024. It is now read-only.
Repository navigation
This repository was archived by the owner on Sep 5, 2024. It is now read-only.
select: multiple attribute only checks for existence but docs indicate that it evaluates true/false #3068
Description
If I want to tell wether a md-select should be multiple from a scope key, I must do this :
<md-select ng-if="multiple" multiple .../>
<md-select ng-if="!multiple" .../>
Because all those lines have the same behavior, they set initialize the select as multiple :
<md-select multiple="true" .../>
<md-select multiple="false" .../>
<md-select multiple="{{true}}" .../>
<md-select multiple="{{false}}" .../>
In the source code of md-select directive, I can see this LoC :
https://github.com/angular/material/blob/master/src/components/select/select.js#L387
I can understand why all 4 lines have the same behavior.
So I tried to use ng-multiple and the select started to work correctly... in appearance only : the model is now all messed up.
Here is a codepen where you can see select behavior and the associated model : http://codepen.io/myagoo/pen/OVWGqz
Reacted by VADIM, Valera Melnikov, Faisal Feroz, Michael Prentice and Hadi Rasouli
Activity
dharitdesai commentedon Dec 15, 2015
+1
pdore-netfore commentedon Dec 15, 2015
+1
nextsoftpl commentedon Oct 25, 2016
+1
danicomas commentedon Jun 1, 2018
No news?
5 remaining items
Splaktar commentedon Jun 6, 2018
Re-opening this one as the CodePen is still valid and this issue still exists.
Splaktar commentedon Jun 6, 2018
This seems to be partially related to angular/angular.js@d87fa00 where binding to
multipleonselectelements was disabled in AngularJS 1.2.0.Splaktar commentedon Jun 6, 2018
There is a lot more discussion related to that in:
angular/angular.js#3230
angular/angular.js#3337
Splaktar commentedon Jun 6, 2018
As mentioned in those issues and #9705,
ng-multiplewas also removed in AngularJS 1.2.0.We need to remove it from AngularJS Material as well. This removal is being tracked in #9705.
Splaktar commentedon Jun 6, 2018
The docs are going to be updated to state:
Any value that is passed into the
multipleattribute will be ignored.docs(select): clarify behavior of multiple attribute
docs(select): clarify behavior of multiple attribute (#11316)
docs(select): clarify behavior of multiple attribute (#11316)