Skip to content

Commit 95cd4ff

Browse files
committed
fix: address feedback
1 parent f8c4640 commit 95cd4ff

File tree

8 files changed

+12
-10
lines changed

8 files changed

+12
-10
lines changed

src/lib/button-toggle/button-toggle.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
@import '../core/a11y/a11y';
22
@import '../core/style/elevation';
3-
@import '../core/style/prefixes';
3+
@import '../core/style/vendor-prefixes';
44
@import '../core/style/layout-common';
55

66
$mat-button-toggle-padding: 0 16px !default;

src/lib/core/option/_option.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
@import '../style/menu-common';
2-
@import '../style/prefixes';
2+
@import '../style/vendor-prefixes';
33
@import '../a11y/a11y';
44

55
/**

src/lib/core/style/_button-common.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
@import './prefixes';
1+
@import './vendor-prefixes';
22

33
// Mixin overriding default button styles like the gray background, the border, and the outline.
44
@mixin mat-button-reset {

src/lib/core/style/_prefixes.scss renamed to src/lib/core/style/_vendor-prefixes.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
user-select: none;
77
}
88

9-
@mixin placeholder {
9+
@mixin input-placeholder {
1010
&::placeholder {
1111
@content;
1212
}

src/lib/input/input-container.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
@import '../core/style/variables';
2-
@import '../core/style/prefixes';
2+
@import '../core/style/vendor-prefixes';
33
@import '../core/style/form-common';
44

55

@@ -110,7 +110,7 @@ $mat-input-underline-disabled-background-image:
110110
// Note that we can't use something like visibility: hidden or
111111
// display: none, because IE ends up preventing the user from
112112
// focusing the input altogether.
113-
@include placeholder {
113+
@include input-placeholder {
114114
color: transparent;
115115
}
116116
}

src/lib/select/select.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
@import '../core/style/list-common';
33
@import '../core/style/form-common';
44
@import '../core/style/variables';
5-
@import '../core/style/prefixes';
5+
@import '../core/style/vendor-prefixes';
66
@import '../core/a11y/a11y';
77

88
$mat-select-trigger-height: 30px !default;

src/lib/slide-toggle/slide-toggle.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
@import '../core/style/variables';
22
@import '../core/ripple/ripple';
33
@import '../core/style/elevation';
4-
@import '../core/style/prefixes';
4+
@import '../core/style/vendor-prefixes';
55
@import '../core/a11y/a11y';
66

77
$mat-slide-toggle-thumb-size: 20px !default;

tools/stylelint/no-prefixes/needs-prefix.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,10 @@ const Browsers = require('autoprefixer/lib/browsers');
33
const Prefixes = require('autoprefixer/lib/prefixes');
44

55
/**
6-
* Utility to be used when checking whether a CSS declaration needs to be prefixed.
7-
* Based on https://github.com/stylelint/stylelint/blob/master/lib/utils/isAutoprefixable.js
6+
* Utility to be used when checking whether a CSS declaration needs to be prefixed. Based on
7+
* Stylelint's `no-vendor-prefix` rule, but instead of checking whether a rule has a prefix,
8+
* we check whether it needs one.
9+
* Reference https://github.com/stylelint/stylelint/blob/master/lib/utils/isAutoprefixable.js
810
*/
911
module.exports = class NeedsPrefix {
1012
constructor(browsers) {

0 commit comments

Comments
 (0)