search

heimrichhannot / contao-slick-bundle

A content slider/carousel for contao based on kenwheelers slick carousel.
3 stars 3 forks source link

Invalide Option randomActive #30

Closed Misco closed 4 years ago

Misco commented 4 years ago

Hallo,

ich nutze Contao 4.4 Im data steht zwar "randomActive":true, aber ohne Wirkung.

Grüße Dennis

koertho commented 4 years ago

@Misco Hi, ich habe gerade mal geschaut und die Option gibt es in Slick gar nicht, ist wohl ein Überbleibsel aus Urzeiten. Werden wir bei Gelegenheit mal entfernen. Falls du eine Zufallsfunktion benötigst, schau mal hier: https://stackoverflow.com/questions/49950262/random-slide-order-in-slick-image-slider/49950398

Misco commented 4 years ago

Danke für den Link. Evtl. kann man es auch wieder hier mit Funktion füllen: mit php prüfen ob Zufall angehackt ist und dann (in etwa):

$("slick").on("init reInit", function() { var i = $('.slick-slide').length; var initalSlide = $("slick").attr("random") ? Math.floor(Math.random() * i) : 0; $("slick").slick('slickGoTo', initalSlide); });

koertho commented 4 years ago

@Misco Du darfst gern einen Pull-Request machen. Wir werden es nicht implementieren, da Slick nicht mehr unser Haupt-Slider ist und wir eine solche Funktion bisher nicht benötigen.

Misco commented 4 years ago

ok, welchen Slider verwendetet ihr jetzt?

koertho commented 4 years ago

Tiny Slider, ist ohne jQuery.

koertho commented 4 years ago

Ich habe gerade mal geschaut, im slick modul war diese Funktion tatsächlich funktional. Aber im Bundle hat dies wohl noch nie funktioniert und wir werden das auch nicht nachziehen. Wie gesagt, das Bundle ist ja erweiterbar gebaut, ist also sehr einfach nachrüstbar. In Version 1.5.0 habe ich die entsprechende Konfiguration nun entfernt.