From a14a09adf5a1cf8d958fd331e8b48ec556af9d01 Mon Sep 17 00:00:00 2001 From: vlazh Date: Mon, 31 Aug 2020 21:28:31 +0600 Subject: [PATCH 1/2] Fix prefix after value-function Prefixing of value-function is not worked. So it fix it. But I'm not sure if I did it right. --- packages/jss-plugin-vendor-prefixer/src/index.js | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/packages/jss-plugin-vendor-prefixer/src/index.js b/packages/jss-plugin-vendor-prefixer/src/index.js index d1db12530..b68a4d630 100644 --- a/packages/jss-plugin-vendor-prefixer/src/index.js +++ b/packages/jss-plugin-vendor-prefixer/src/index.js @@ -48,6 +48,13 @@ export default function jssVendorPrefixer(): Plugin { function onChangeValue(value, prop) { return vendor.supportedValue(prop, toCssValue(value)) || value } + + function onUpdate(data, rule, sheet, options) { + // Prefix after value-function + if (rule.style) { + prefixStyle(rule.style); + } + } - return {onProcessRule, onProcessStyle, onChangeValue} + return {onProcessRule, onProcessStyle, onChangeValue, onUpdate} } From 6db4ddfe0e5a24fad02ca5980bb76cb9c49ee0a3 Mon Sep 17 00:00:00 2001 From: vlazh Date: Tue, 1 Sep 2020 11:14:25 +0600 Subject: [PATCH 2/2] Try fix types --- packages/jss-plugin-vendor-prefixer/src/index.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/jss-plugin-vendor-prefixer/src/index.js b/packages/jss-plugin-vendor-prefixer/src/index.js index b68a4d630..a9cdd7c82 100644 --- a/packages/jss-plugin-vendor-prefixer/src/index.js +++ b/packages/jss-plugin-vendor-prefixer/src/index.js @@ -1,6 +1,6 @@ // @flow import * as vendor from 'css-vendor' -import {toCssValue, type Plugin, type KeyframesRule} from 'jss' +import {toCssValue, type Plugin, type KeyframesRule, type StyleRule} from 'jss' /** * Add vendor prefix to a property name when needed. @@ -51,8 +51,8 @@ export default function jssVendorPrefixer(): Plugin { function onUpdate(data, rule, sheet, options) { // Prefix after value-function - if (rule.style) { - prefixStyle(rule.style); + if (rule.type === 'style') { + prefixStyle((rule: StyleRule).style); } }