@uppy/informer - npm Package Compare versions

Comparing version 3.0.0-beta.1 to 3.0.0-beta.2

CHANGELOG.md

 # @uppy/informer
 
+## 3.0.0-beta.2
+
+Released: 2022-08-03
+Included in: Uppy v3.0.0-beta.4
+
+- @uppy/informer: simplify `render` method (Antoine du Hamel / #3931)
+
 ## 2.1.0
@@ -4,0 +11,0 @@

lib/Informer.js

@@ -9,3 +9,3 @@ /* eslint-disable jsx-a11y/no-noninteractive-element-interactions  */
 const packageJson = {
-  "version": "3.0.0-beta.1"
+  "version": "3.0.0-beta.2"
 };
@@ -12,0 +12,0 @@ /**

lib/TransitionGroup.js

@@ -296,19 +296,11 @@ /* eslint-disable */
     // by cloning a single child
-    const childrenToRender = [];
-
-    for (const key in children) {
-      if (children.hasOwnProperty(key)) {
-        const child = children[key];
-
-        if (child) {
-          const ref = linkRef(this, key),
-                el = cloneElement(childFactory(child), {
-            ref,
-            key
-          });
-          childrenToRender.push(el);
-        }
-      }
-    }
-
+    const childrenToRender = Object.entries(children).map(_ref3 => {
+      let [key, child] = _ref3;
+      if (!child) return undefined;
+      const ref = linkRef(this, key);
+      return cloneElement(childFactory(child), {
+        ref,
+        key
+      });
+    }).filter(Boolean);
     return h(component, props, childrenToRender);
@@ -315,0 +307,0 @@ }

package.json

 {
   "name": "@uppy/informer",
   "description": "A notification and error pop-up bar for Uppy.",
-  "version": "3.0.0-beta.1",
+  "version": "3.0.0-beta.2",
   "license": "MIT",
@@ -31,5 +31,5 @@ "main": "lib/index.js",
   "peerDependencies": {
-    "@uppy/core": "^3.0.0-beta.1"
+    "@uppy/core": "^3.0.0-beta.3"
   },
   "stableVersion": "2.1.0"
 }

src/TransitionGroup.js

@@ -267,14 +267,9 @@ /* eslint-disable */
     // by cloning a single child
-    const childrenToRender = []
-    for (const key in children) {
-      if (children.hasOwnProperty(key)) {
-        const child = children[key]
-        if (child) {
-          const ref = linkRef(this, key),
-            el = cloneElement(childFactory(child), { ref, key })
-          childrenToRender.push(el)
-        }
-      }
-    }
+    const childrenToRender = Object.entries(children).map(([key, child]) => {
+      if (!child) return undefined
 
+      const ref = linkRef(this, key);
+      return cloneElement(childFactory(child), { ref, key })
+    }).filter(Boolean)
+
     return h(component, props, childrenToRender)
@@ -281,0 +276,0 @@ }

No alert changes

Worsened metrics

Total package byte prevSize

53110

decreased by-0.33%

Lines of code

723

decreased by-1.5%

No dependency changes