fix: carousel not work with variable-width #3977

vueComponent · Apr 27, 2021 · 58ffdd9 · 58ffdd9
1 parent 5eeaaae
commit 58ffdd9
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 6 deletions.
diff --git a/components/vc-slick/src/inner-slider.js b/components/vc-slick/src/inner-slider.js
@@ -116,7 +116,7 @@ export default {
           slideCount: children.length,
         });
         children.forEach(child => {
-          const childWidth = child.props.width.split('px')[0];
+          const childWidth = child.props.style?.width?.split('px')[0] || 0;
           childrenWidths.push(childWidth);
           trackWidth += childWidth;
         });

diff --git a/components/vc-slick/src/slider.js b/components/vc-slick/src/slider.js
@@ -155,7 +155,7 @@ export default defineComponent({
       for (let j = i; j < i + settings.rows * settings.slidesPerRow; j += settings.slidesPerRow) {
         const row = [];
         for (let k = j; k < j + settings.slidesPerRow; k += 1) {
-          if (settings.variableWidth && children[k].props.style) {
+          if (settings.variableWidth && children[k].props?.style) {
             currentWidth = children[k].props.style.width;
           }
           if (k >= children.length) break;

diff --git a/v2-doc b/v2-doc
diff --git a/webpack.config.js b/webpack.config.js
@@ -118,9 +118,7 @@ module.exports = {
         use: [
           {
             loader: MiniCssExtractPlugin.loader,
-            options: {
-              hmr: true,
-            },
+            options: {},
           },
           'css-loader',
         ],