Fix TypeError for shorthand animation in time-min-milliseconds (#4783)

stylelint · May 18, 2020 · 90d4fe2 · 90d4fe2
1 parent f88bcd5
commit 90d4fe2
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/lib/rules/time-min-milliseconds/__tests__/index.js b/lib/rules/time-min-milliseconds/__tests__/index.js
@@ -9,6 +9,15 @@ testRule({
 	config: [MIN_VALUE],
 
 	accept: [
+		{
+			code: 'a { animation: none; }',
+		},
+		{
+			code: 'a { animation: none, my-animation; }',
+		},
+		{
+			code: 'a { transition: opacity var(--x) var(--y) ease-out }',
+		},
 		{
 			code: 'a { transition-delay: 0.1s; }',
 		},
@@ -314,6 +323,15 @@ testRule({
 	config: [MIN_VALUE, { ignore: ['delay'] }],
 
 	accept: [
+		{
+			code: 'a { animation: none; }',
+		},
+		{
+			code: 'a { animation: none, my-animation; }',
+		},
+		{
+			code: 'a { transition: opacity var(--x) var(--y) ease-out }',
+		},
 		{
 			code: 'a { transition-duration: 0.15s; }',
 		},

diff --git a/lib/rules/time-min-milliseconds/index.js b/lib/rules/time-min-milliseconds/index.js
@@ -63,7 +63,7 @@ function rule(minimum, options) {
 						// Check only duration time values
 						const duration = getDuration(valueList);
 
-						if (!isAcceptableTime(duration)) {
+						if (duration && !isAcceptableTime(duration)) {
 							complain(decl, decl.value.indexOf(duration));
 						}
 					} else {