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

Commit ecd6837

Browse files
devversionThomasBurleson
authored andcommitted
fix(select): use parsed attribute for md-container-class attribute
Fixes #6973 Closes #6976
1 parent 9a36112 commit ecd6837

File tree

2 files changed

+20
-2
lines changed

2 files changed

+20
-2
lines changed

src/components/select/select.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -430,8 +430,8 @@ function SelectDirective($mdSelect, $mdUtil, $mdTheming, $mdAria, $compile, $par
430430
element[0].querySelector('.md-select-menu-container')
431431
);
432432
selectScope = scope;
433-
if (element.attr('md-container-class')) {
434-
var value = selectContainer[0].getAttribute('class') + ' ' + element.attr('md-container-class');
433+
if (attr.mdContainerClass) {
434+
var value = selectContainer[0].getAttribute('class') + ' ' + attr.mdContainerClass;
435435
selectContainer[0].setAttribute('class', value);
436436
}
437437
selectMenuCtrl = selectContainer.find('md-select-menu').controller('mdSelectMenu');

src/components/select/select.spec.js

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,24 @@ describe('<md-select>', function() {
5757
expect(container.classList.contains('test')).toBe(true);
5858
}));
5959

60+
it('supports passing classes to the container using `data-` attribute prefix', inject(function($document) {
61+
var select = setupSelect('ng-model="val", data-md-container-class="test"').find('md-select');
62+
openSelect(select);
63+
64+
var container = $document[0].querySelector('.md-select-menu-container');
65+
expect(container).toBeTruthy();
66+
expect(container.classList.contains('test')).toBe(true);
67+
}));
68+
69+
it('supports passing classes to the container using `x-` attribute prefix', inject(function($document) {
70+
var select = setupSelect('ng-model="val", x-md-container-class="test"').find('md-select');
71+
openSelect(select);
72+
73+
var container = $document[0].querySelector('.md-select-menu-container');
74+
expect(container).toBeTruthy();
75+
expect(container.classList.contains('test')).toBe(true);
76+
}));
77+
6078
it('sets aria-owns between the select and the container', function() {
6179
var select = setupSelect('ng-model="val"').find('md-select');
6280
var ownsId = select.attr('aria-owns');

0 commit comments

Comments
 (0)