Slideshow_XH

Ein CMSimple Support Forum für deutsch sprechende Nutzer und Entwickler
olape
Posts: 1822
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Slideshow_XH

Post by olape » Sat May 01, 2021 5:58 pm

cmb wrote:
Sat May 01, 2021 5:01 pm
Hm, ist das $img Array unter bestimmten Umständen vielleicht nicht fortlaufend indexiert (0, 1, 2, …). Das wäre natürlich ein Bug. Muss ich noch genauer prüfen.
Wie kommst du darauf?
cmb wrote:
Sat May 01, 2021 4:46 pm
Ich las neulich, dass die Erde eine Scheibe ist, weil sonst ja die Menschen auf der Südhalbkugel herunter fallen würden.
Es geht mir da nicht um absolute Werte, die ändern sich bei jedem Test. Aber auffällig ist schon, dass es bei mehrfachen Tests einen deutlichen Trend gibt.
cmb wrote:
Sat May 01, 2021 4:46 pm
Das sollte eigentlich genau denselben Effekt (i.e. das selbe HTML wird erzeugt) wie mein Patch haben. Das erste Bild "eager", alle weiteren "lazy".
Ja, und wenn ich beim Lesen aufgepasst hätte, dann hätte ich nicht mal selber denken müssen. :oops:

cmb
Posts: 13761
Joined: Tue Jun 21, 2011 11:04 am
Location: Mü-Sa, RLP, DE
Contact:

Re: Slideshow_XH

Post by cmb » Sat May 01, 2021 8:39 pm

olape wrote:
Sat May 01, 2021 5:58 pm
cmb wrote:
Sat May 01, 2021 5:01 pm
Hm, ist das $img Array unter bestimmten Umständen vielleicht nicht fortlaufend indexiert (0, 1, 2, …). Das wäre natürlich ein Bug. Muss ich noch genauer prüfen.
Wie kommst du darauf?
Es hätte sein können, dass deshalb mein Patch bei dir nicht funktioniert hat. Aber wie es scheint, hatte das andere Gründe. :)
Christoph M. Becker – Plugins for CMSimple_XH

olape
Posts: 1822
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Slideshow_XH

Post by olape » Sat May 01, 2021 9:10 pm

cmb wrote:
Sat May 01, 2021 8:39 pm
Es hätte sein können, dass deshalb mein Patch bei dir nicht funktioniert hat. Aber wie es scheint, hatte das andere Gründe
Genau, den Link habe ich nämlich gar nicht für voll genommen, sondern nur den Hinweis auf loading=„lazy“.
Erst nach dem zweiten Hinweis habe ich gesehen, dass du schon viel weiter warst. :oops:

olape
Posts: 1822
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Slideshow_XH

Post by olape » Mon May 03, 2021 7:55 am

cmb wrote:
Fri Apr 30, 2021 10:51 pm
Probier's aber bitte mal aus. Mir scheint, das ist sowohl mit aktuellem Chrome als auch Firefox eine Verbesserung, selbst wenn sich die Slideshow "above the fold" befindet.
Wird das so in die nächste Version kommen?
Ich habe das jetzt bei einer Seite so eingefügt, funktioniert prima.

cmb
Posts: 13761
Joined: Tue Jun 21, 2011 11:04 am
Location: Mü-Sa, RLP, DE
Contact:

Re: Slideshow_XH

Post by cmb » Mon May 03, 2021 9:25 pm

olape wrote:
Mon May 03, 2021 7:55 am
cmb wrote:
Fri Apr 30, 2021 10:51 pm
Probier's aber bitte mal aus. Mir scheint, das ist sowohl mit aktuellem Chrome als auch Firefox eine Verbesserung, selbst wenn sich die Slideshow "above the fold" befindet.
Wird das so in die nächste Version kommen?
Ich habe das jetzt bei einer Seite so eingefügt, funktioniert prima.
Ja, gerne. Mit einem neuen Release würde ich vielleicht noch eine Weile warten, es sei denn es pressiert jemandem. Auf jeden Fall vielen Dank fürs Testen!
Christoph M. Becker – Plugins for CMSimple_XH

olape
Posts: 1822
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Slideshow_XH

Post by olape » Tue May 04, 2021 7:08 am

frase hat einen Fehler entdeckt, der im Zusammenspiel mit CNC entsteht.
CNC generiert einen Endtag für <source ...> bei der Ausgabe der Slideshow.

Falls das also jemand so nutzt ...

olape
Posts: 1822
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Slideshow_XH

Post by olape » Tue May 04, 2021 6:58 pm


cmb
Posts: 13761
Joined: Tue Jun 21, 2011 11:04 am
Location: Mü-Sa, RLP, DE
Contact:

Re: Slideshow_XH

Post by cmb » Tue May 04, 2021 9:42 pm

olape wrote:
Tue May 04, 2021 7:08 am
frase hat einen Fehler entdeckt, der im Zusammenspiel mit CNC entsteht.
CNC generiert einen Endtag für <source ...> bei der Ausgabe der Slideshow.
Ich hab's mir nicht amgeschaut, vermute aber, dass CnC das DOM nutzt, und das ist diesbezüglich tatsächlich buggy. Da müsste also eine Nachbearbeitung her, möglicherweise einfach ein str_replace(["</source>", "</track>"], "", …) oder ähnlich.
Christoph M. Becker – Plugins for CMSimple_XH

olape
Posts: 1822
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Slideshow_XH

Post by olape » Wed May 05, 2021 8:40 am

cmb wrote:
Tue May 04, 2021 9:42 pm
und das ist diesbezüglich tatsächlich buggy.
Den Link hätte ich gestern Abend gebraucht. Ich habe dazu keine konkrete Aussage gefunden. :cry:

OK, dann könnte man als schnellen Patch tatsächlich str_replace, dann auch gleich für - command - embed - keygen - source - track - wbr reinnehmen.
Auch nicht schön, aber sicher wesentlich performanter als meine Lösung.

olape
Posts: 1822
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Slideshow_XH

Post by olape » Wed May 05, 2021 7:32 pm

olape wrote:
Wed May 05, 2021 8:40 am
OK, dann könnte man als schnellen Patch tatsächlich str_replace, dann auch gleich für - command - embed - keygen - source - track - wbr reinnehmen.
Auch nicht schön, aber sicher wesentlich performanter als meine Lösung.
Diese Variante funktioniert ohne merkbarem Performanceverlust. Sicherheitshalber habe ich aber str_ireplace genutzt.

Post Reply