make attributeStyleMap optional

Author: kof

packages/jss/.size-snapshot.json

@@ -1,30 +1,30 @@
 {
   "dist/jss.js": {
-    "bundled": 59659,
-    "minified": 22190,
-    "gzipped": 6640
+    "bundled": 59644,
+    "minified": 22179,
+    "gzipped": 6636
   },
   "dist/jss.min.js": {
-    "bundled": 58493,
-    "minified": 21272,
-    "gzipped": 6198
+    "bundled": 58478,
+    "minified": 21261,
+    "gzipped": 6190
   },
   "dist/jss.cjs.js": {
-    "bundled": 55063,
-    "minified": 24149,
-    "gzipped": 6694
+    "bundled": 55048,
+    "minified": 24138,
+    "gzipped": 6689
   },
   "dist/jss.esm.js": {
-    "bundled": 54567,
-    "minified": 23745,
-    "gzipped": 6614,
+    "bundled": 54552,
+    "minified": 23734,
+    "gzipped": 6607,
     "treeshaked": {
       "rollup": {
-        "code": 19332,
+        "code": 19321,
         "import_statements": 281
       },
       "webpack": {
-        "code": 20747
+        "code": 20736
       }
     }
   }

packages/jss/src/StyleSheet.js

@@ -9,7 +9,8 @@ import type {
   UpdateArguments,
   JssStyle,
   Classes,
-  KeyframesMap
+  KeyframesMap,
+  JssStyles
 } from './types'
 
 export default class StyleSheet {
@@ -29,7 +30,7 @@ export default class StyleSheet {
 
   queue: ?Array<Rule>
 
-  constructor(styles: Object, options: StyleSheetOptions) {
+  constructor(styles: JssStyles, options: StyleSheetOptions) {
     this.attached = false
     this.deployed = false
     this.classes = {}
@@ -124,7 +125,7 @@ export default class StyleSheet {
    * Create and add rules.
    * Will render also after Style Sheet was rendered the first time.
    */
-  addRules(styles: Object, options?: RuleOptions): Array<Rule> {
+  addRules(styles: JssStyles, options?: RuleOptions): Array<Rule> {
     const added = []
     for (const name in styles) {
       const rule = this.addRule(name, styles[name], options)

packages/jss/src/renderers/DomRenderer.js

@@ -41,7 +41,7 @@ function getPropertyValue(
 ): string {
   try {
     // Support CSSTOM.
-    if ('attributeStyleMap' in cssRule) {
+    if (cssRule.attributeStyleMap) {
       return cssRule.attributeStyleMap.get(prop)
     }
     return cssRule.style.getPropertyValue(prop)
@@ -72,7 +72,7 @@ function setProperty(
     }
 
     // Support CSSTOM.
-    if ('attributeStyleMap' in cssRule) {
+    if (cssRule.attributeStyleMap) {
       cssRule.attributeStyleMap.set(prop, cssValue)
     } else {
       cssRule.style.setProperty(prop, cssValue)
@@ -93,7 +93,7 @@ function removeProperty(
 ) {
   try {
     // Support CSSTOM.
-    if ('attributeStyleMap' in cssRule) {
+    if (cssRule.attributeStyleMap) {
       cssRule.attributeStyleMap.delete(prop)
     } else {
       cssRule.style.removeProperty(prop)

packages/jss/src/types/dom.js

@@ -5,5 +5,5 @@ import {StylePropertyMap} from './cssom'
 export type DOMString = string
 
 export interface HTMLElementWithStyleMap extends HTMLElement {
-  +attributeStyleMap: StylePropertyMap;
+  +attributeStyleMap?: StylePropertyMap;
 }

packages/jss/src/types/jss.js

@@ -61,6 +61,8 @@ export type GenerateId = (rule: Rule, sheet?: StyleSheet) => string
 // Find a way to declare all types: Object|string|Array<Object>
 export type JssStyle = Object
 
+export type JssStyles = {[string]: JssStyle}
+
 export type JssValue =
   | string
   | number