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

Commit 4aa7160

Browse files
crisbetoThomasBurleson
authored andcommitted
fix(slider): fix a js error if the slider container doesn't have a md-input-container
* Fixes the slider trying to call getComputedStyle if it doesn't have child md-input-container. * Uses a slightly faster way of finding the child md-input-container. Fixes #8174, #7728. Closes #8225
1 parent a9bca2b commit 4aa7160

File tree

1 file changed

+12
-9
lines changed

1 file changed

+12
-9
lines changed

src/components/slider/slider.js

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -70,19 +70,22 @@ function SliderContainerDirective() {
7070
var initialMaxWidth;
7171

7272
ctrl.fitInputWidthToTextLength = function (length) {
73-
var input = element.find('md-input-container');
74-
var computedStyle = getComputedStyle(input[0]);
75-
var minWidth = parseInt(computedStyle['min-width']);
76-
var padding = parseInt(computedStyle['padding']) * 2;
77-
initialMaxWidth = initialMaxWidth || parseInt(computedStyle['max-width']);
73+
var input = element[0].querySelector('md-input-container');
7874

79-
var newMaxWidth = Math.max(initialMaxWidth, minWidth + padding + (minWidth / 2 * length));
75+
if (input) {
76+
var computedStyle = getComputedStyle(input);
77+
var minWidth = parseInt(computedStyle.minWidth);
78+
var padding = parseInt(computedStyle.padding) * 2;
8079

81-
input.css('max-width', newMaxWidth + 'px');
80+
initialMaxWidth = initialMaxWidth || parseInt(computedStyle.maxWidth);
81+
var newMaxWidth = Math.max(initialMaxWidth, minWidth + padding + (minWidth / 2 * length));
82+
83+
input.style.maxWidth = newMaxWidth + 'px';
84+
}
8285
};
83-
}
86+
};
8487
}
85-
}
88+
};
8689
}
8790

8891
/**

0 commit comments

Comments
 (0)