Slideshow previous button disabled fix

[ludoboludo]

Why are these changes introduced?

Fixes an issue where the previous button on the slideshow is disabled. It doesn't come up when you first add the section in the editor but does once you've saved.

Didn't notice it before, so I'm wondering what could potentially have triggered it 🤔 Seems like an issue with the order in which functions/values are being called.

I think it's due to the fact that the slideshow component is extending the slider component and has a different value for this.enableSliderLooping. But when it runs update() it's using the false value set in sliderComponent first which disables the previous button and only after that it notices the different value set within the slideshow component.

What approach did you take?

I added a check in the update() within the slideshow component. So that if it sees the disabled attribute it removes it.

Other considerations

Demo links

• Store
• Editor

Checklist

• Followed theme code principles
• Linted with Theme Check
• Tested on mobile
• Tested on multiple browsers
• Tested for accessibility

[LucasLacerdaUX]

The solution works. I think we can get rid of the if, though :D