Skip to content

Commit 1d1a903

Browse files
author
Cari Spruiell
committed
MC-2413: Fix Slider Issues
- update frontend code to use true/false instead of 0/1 for attribute values - update master format documentation
1 parent 9d6f6ea commit 1d1a903

File tree

4 files changed

+17
-17
lines changed

4 files changed

+17
-17
lines changed

app/code/Magento/PageBuilder/docs/master-format.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -487,8 +487,8 @@ Attributes
487487
## Slider
488488

489489
```
490-
<div class="pagebuilder-slider" data-role="slider" data-appearance="default" data-autoplay="0"
491-
data-autoplay-speed="4000" data-fade="0" data-show-arrows="0" data-show-dots="1" style="...">
490+
<div class="pagebuilder-slider" data-role="slider" data-appearance="default" data-autoplay="false"
491+
data-autoplay-speed="4000" data-fade="false" data-show-arrows="false" data-show-dots="true" style="...">
492492
```
493493

494494
Attributes

app/code/Magento/PageBuilder/view/adminhtml/ui_component/pagebuilder_slider_form.xml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@
7979
<field name="autoplay" sortOrder="1" formElement="checkbox">
8080
<argument name="data" xsi:type="array">
8181
<item name="config" xsi:type="array">
82-
<item name="default" xsi:type="number">0</item>
82+
<item name="default" xsi:type="string">false</item>
8383
</item>
8484
</argument>
8585
<settings>
@@ -118,7 +118,7 @@
118118
<field name="fade" sortOrder="20" formElement="checkbox">
119119
<argument name="data" xsi:type="array">
120120
<item name="config" xsi:type="array">
121-
<item name="default" xsi:type="number">0</item>
121+
<item name="default" xsi:type="string">false</item>
122122
</item>
123123
</argument>
124124
<settings>
@@ -140,7 +140,7 @@
140140
<field name="is_infinite" sortOrder="30" formElement="checkbox">
141141
<argument name="data" xsi:type="array">
142142
<item name="config" xsi:type="array">
143-
<item name="default" xsi:type="number">0</item>
143+
<item name="default" xsi:type="string">false</item>
144144
</item>
145145
</argument>
146146
<settings>
@@ -162,7 +162,7 @@
162162
<field name="show_arrows" sortOrder="40" formElement="checkbox">
163163
<argument name="data" xsi:type="array">
164164
<item name="config" xsi:type="array">
165-
<item name="default" xsi:type="number">0</item>
165+
<item name="default" xsi:type="string">false</item>
166166
</item>
167167
</argument>
168168
<settings>
@@ -184,7 +184,7 @@
184184
<field name="show_dots" sortOrder="50" formElement="checkbox">
185185
<argument name="data" xsi:type="array">
186186
<item name="config" xsi:type="array">
187-
<item name="default" xsi:type="number">1</item>
187+
<item name="default" xsi:type="string">true</item>
188188
</item>
189189
</argument>
190190
<settings>

app/code/Magento/PageBuilder/view/frontend/web/js/content-type/slider/appearance/default/widget.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,12 @@ define([
2121
}
2222

2323
$element.slick({
24-
autoplay: $element.data('autoplay') === 1,
24+
autoplay: $element.data('autoplay'),
2525
autoplaySpeed: $element.data('autoplay-speed') || 0,
26-
fade: $element.data('fade') === 1,
27-
infinite: $element.data('is-infinite') === 1,
28-
arrows: $element.data('show-arrows') === 1,
29-
dots: $element.data('show-dots') === 1
26+
fade: $element.data('fade'),
27+
infinite: $element.data('is-infinite'),
28+
arrows: $element.data('show-arrows'),
29+
dots: $element.data('show-dots')
3030
});
3131
};
3232
});

dev/tests/js/jasmine/tests/app/code/Magento/PageBuilder/view/frontend/web/js/content-type/slider/appearance/default/widget.test.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,12 @@ define([
2626

2727
spyOn($.fn, 'slick');
2828

29-
el.setAttribute('data-autoplay', 1);
29+
el.setAttribute('data-autoplay', "true");
3030
el.setAttribute('data-autoplay-speed', 500);
31-
el.setAttribute('data-fade', 1);
32-
el.setAttribute('data-is-infinite', 1);
33-
el.setAttribute('data-show-arrows', 1);
34-
el.setAttribute('data-show-dots', 1);
31+
el.setAttribute('data-fade', "true");
32+
el.setAttribute('data-is-infinite', "true");
33+
el.setAttribute('data-show-arrows', "true");
34+
el.setAttribute('data-show-dots', "true");
3535

3636
sliderWidgetInitializer(undefined, el);
3737

0 commit comments

Comments
 (0)