address feedback and fix lint

mmalerba · mmalerba · commit c4ee05b505b0 · 2020-02-12T13:13:50.000-08:00
diff --git a/.stylelintrc.json b/.stylelintrc.json
@@ -44,7 +44,7 @@
 
     "unit-case": "lower",
     "unit-no-unknown": true,
-    "unit-whitelist": ["px", "%", "deg", "ms", "em", "vh", "vw", "vmin"],
+    "unit-whitelist": ["px", "%", "deg", "s", "ms", "em", "vh", "vw", "vmin"],
 
     "value-list-comma-space-after": "always-single-line",
     "value-list-comma-space-before": "never",
diff --git a/src/cdk/text-field/_text-field.scss b/src/cdk/text-field/_text-field.scss
@@ -8,12 +8,12 @@
   @keyframes cdk-text-field-autofill-end {/*!*/}
 
   .cdk-text-field-autofill-monitored:-webkit-autofill {
-    // In Chrome 80 we need a 1ms delay, or the animationstart event won't fire.
+    // Since Chrome 80 we need a 1ms delay, or the animationstart event won't fire.
     animation: cdk-text-field-autofill-start 0s 1ms;
   }
 
   .cdk-text-field-autofill-monitored:not(:-webkit-autofill) {
-    // In Chrome 80 we need a 1ms delay, or the animationstart event won't fire.
+    // Since Chrome 80 we need a 1ms delay, or the animationstart event won't fire.
     animation: cdk-text-field-autofill-end 0s 1ms;
   }
 
@@ -55,7 +55,7 @@ $cdk-text-field-autofill-color-frame-count: 0;
   }
 
   &.cdk-text-field-autofill-monitored:-webkit-autofill {
-    // In Chrome 80 we need a 1ms delay for cdk-text-field-autofill-start, or the animationstart
+    // Since Chrome 80 we need a 1ms delay for cdk-text-field-autofill-start, or the animationstart
     // event won't fire.
     animation: cdk-text-field-autofill-start 0s 1ms,
                cdk-text-field-autofill-color-#{$cdk-text-field-autofill-color-frame-count} both;

Original file line number	Diff line number	Diff line change
`@@ -8,12 +8,12 @@`
`8`	`8`	`@keyframes cdk-text-field-autofill-end {/!/}`
`9`	`9`
`10`	`10`	`.cdk-text-field-autofill-monitored:-webkit-autofill {`
`11`		`- // In Chrome 80 we need a 1ms delay, or the animationstart event won't fire.`
	`11`	`+ // Since Chrome 80 we need a 1ms delay, or the animationstart event won't fire.`
`12`	`12`	`animation: cdk-text-field-autofill-start 0s 1ms;`
`13`	`13`	`}`
`14`	`14`
`15`	`15`	`.cdk-text-field-autofill-monitored:not(:-webkit-autofill) {`
`16`		`- // In Chrome 80 we need a 1ms delay, or the animationstart event won't fire.`
	`16`	`+ // Since Chrome 80 we need a 1ms delay, or the animationstart event won't fire.`
`17`	`17`	`animation: cdk-text-field-autofill-end 0s 1ms;`
`18`	`18`	`}`
`19`	`19`
`@@ -55,7 +55,7 @@ $cdk-text-field-autofill-color-frame-count: 0;`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`&.cdk-text-field-autofill-monitored:-webkit-autofill {`
`58`		`- // In Chrome 80 we need a 1ms delay for cdk-text-field-autofill-start, or the animationstart`
	`58`	`+ // Since Chrome 80 we need a 1ms delay for cdk-text-field-autofill-start, or the animationstart`
`59`	`59`	`// event won't fire.`
`60`	`60`	`animation: cdk-text-field-autofill-start 0s 1ms,`
`61`	`61`	`cdk-text-field-autofill-color-#{$cdk-text-field-autofill-color-frame-count} both;`