CMSimple_XH–Forum

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.

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.

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.

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.

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!

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 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.

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.

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 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.