From bec9bdf3f3fb8588fbb819849949639278c72d7d Mon Sep 17 00:00:00 2001 From: Pig Fang Date: Sat, 16 Dec 2017 11:03:37 +0800 Subject: [PATCH] Fix iterating items for closet and skin library `for...of` is a feature of ES6, however Babel will transpile this syntax to `Symbol.iterator()`, which does not supported on IE11. We won't add `Symbol` polyfill, because it will increase JS file size. `Array.prototype.reduce` is of ES5 and supported on IE11. --- resources/assets/src/js/skinlib/index.js | 4 +--- resources/assets/src/js/user/closet.js | 4 +--- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/resources/assets/src/js/skinlib/index.js b/resources/assets/src/js/skinlib/index.js index 10db1633..132fd504 100644 --- a/resources/assets/src/js/skinlib/index.js +++ b/resources/assets/src/js/skinlib/index.js @@ -60,9 +60,7 @@ function renderSkinlib(items) { } else { $('#skinlib-paginator').show(); - for (const item of items) { - container.append(renderSkinlibItemComponent(item)); - } + container.html(items.reduce((carry, item) => renderSkinlibItemComponent(item), '')); } $('.overlay').hide(); diff --git a/resources/assets/src/js/user/closet.js b/resources/assets/src/js/user/closet.js index 949ec8d9..c14a961b 100644 --- a/resources/assets/src/js/user/closet.js +++ b/resources/assets/src/js/user/closet.js @@ -132,9 +132,7 @@ function renderCloset(items, category) { } else { $('#closet-paginator').show(); - for (let item of items) { - container.append(renderClosetItemComponent(item)); - } + container.html(items.reduce((carry, item) => carry + renderClosetItemComponent(item), '')); } }