Trade Republic - Provisionsfrei Aktien handeln
Neuigkeiten
  • Die modified eCommerce Shopsoftware ist kostenlos, aber nicht umsonst.
    Spenden
  • Damit wir die modified eCommerce Shopsoftware auch zukünftig kostenlos anbieten können:
    Spenden
  • Thema: Responsive Template für Modified

    noRiddle (revilonetz)

    • Experte
    • Beiträge: 13.707
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #570 am: 23. Juni 2021, 13:13:27
    @Markus
    @FräuleinGarn

    Hat sich bzgl. meiner Antwort #563 etwas aus euren Tests ergeben ?

    Gruß,
    noRiddle
    Trade Republic - Provisionsfrei Aktien handeln

    Markus

    • modified Team
    • Beiträge: 1.361
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #571 am: 23. Juni 2021, 13:26:18
    Hi noRiddle,

    ich konnte jetzt keine Fehler oder Einschränkungen feststellen ... es funktioniert in den beiden Shops wo ich das mal eingebaut hatte ohne Probleme.

    @Timm
    Hattest du es auch mal ausprobiert?

    Markus

    Timm

    • Fördermitglied
    • Beiträge: 6.318
    Re: Responsive Template für Modified
    Antwort #572 am: 23. Juni 2021, 13:36:27
    @Markus
    Ich hatte es nicht getestet, da ich keine Testdatei von dir bekommen habe, siehe Antwort #565. :flee-mrgreen:

    Gruß Timm

    Markus

    • modified Team
    • Beiträge: 1.361
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #573 am: 23. Juni 2021, 13:41:35
    Hi Timm,

    da ist ranzukommen ... anbei die Datei einfach zum austauschen. :-)

    Markus

    Timm

    • Fördermitglied
    • Beiträge: 6.318
    Re: Responsive Template für Modified
    Antwort #574 am: 24. Juni 2021, 09:57:06
    Moin

    Danke dir.

    Generell funktioniert es, aber mir ist aufgefallen, dass die

    Code: Javascript  [Auswählen]
    fastLoadedClass:"ls-is-cached"

    auch entfernt wird neben den gewollten Leerzeichen.

    Aus

    Code: CSS  [Auswählen]
    <img class="lazyload" data-src=".......>

    wird

    Code: CSS  [Auswählen]
    <img class="lazyloaded" data-src=".......>

    statt wie im Demoshop (dort noch mit dem ungewollten Leerzeichen am Anfang)

    Code: CSS  [Auswählen]
    <img class="ls-is-cached lazyloaded" data-src=".......>

    Und es gibt ein Update des Skripts auf Version 5.3.1.

    Gruss Timm

    Markus

    • modified Team
    • Beiträge: 1.361
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #575 am: 24. Juni 2021, 10:52:11
    Hi Timm,

    die 5.3.1 kannst du einfach reinkopieren ... das sind nur marginale Änderungen welche für unseren Gebrauch keinerlei Auswirkungen haben.

    Das ist halt der Grund warum ich ungern an solchen Scripten was ändere. Ab dem Moment wo wir in an solchen Core-Dateien was machen haben wir immer ein Problem und müssen höllisch aufpassen bei Updates.

    Markus

    Timm

    • Fördermitglied
    • Beiträge: 6.318
    Re: Responsive Template für Modified
    Antwort #576 am: 24. Juni 2021, 11:28:06
    Sorry, grad gesehen, dass das

    Code: CSS  [Auswählen]
     <img class="ls-is-cached lazyloaded" data-src=".......>

    bei mir nie erscheint. Auch nicht mit dem ungeänderten Script. Muss also bei mir was falsch sein.

    Könntest du bitte mal da wo du es geändert hast schauen, ob das

    Code: CSS  [Auswählen]
    ls-is-cached

    mit entfernt wird neben den Leerzeichen? Oder mal testweise im dev Demoshop.

    Gruss Timm

    Markus

    • modified Team
    • Beiträge: 1.361
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #577 am: 24. Juni 2021, 11:46:10
    Hi Timm,

    bei mir kommt das auch nicht. Ich weiss nicht wo das bei kommt.
    Das sollte aber eigentlich uninteressant sein .. wichtig ist das "lazyloaded".

    Da kann nicht viel falsch sein Timm. Das Script wird hochgeladen .. fertig. Da gibts es keine Konfiguration oder sonstwas.
    Das ganze zündet lediglich auf die Klasse "lazyload".

    Markus

    noRiddle (revilonetz)

    • Experte
    • Beiträge: 13.707
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #578 am: 24. Juni 2021, 13:22:57
    Das Skript scheint eine Art Cache-Funktion zu haben (habe es nicht komplett analysiert).
    Das ls-is-cached kommt erst wenn man z.B. folgendes macht:
    • Rufe eine Kategorie auf und schaue mittels Browser-Tool in die Klasse eines Bildes.
      => ls-is-cached  ist nicht vorhanden.
    • Klicke einmal auf die Startseite.
    • Klicke erneut auf die Kategorie aus Punkt 1 und schaue mittels Browser-Tool in die Klasse eines Bildes.
      => ls-is-cached  ist vorhanden.

    Was die Update-Sicherheit anbelangt:
    Wenn Ihr bestätigen könnt, daß mein Vorschlag okay funktioniert schlage ich dem Ersteller des Skriptes die Verbesserung vor. Er scheint das Teil ja noch zu pflegen wenn es bereits eine neuere Version gibt.

    Gruß,
    noRiddle

    Markus

    • modified Team
    • Beiträge: 1.361
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #579 am: 24. Juni 2021, 13:54:56
    Hi noRiddle,,

    ich kann nichts negatives sagen und hatte auch in keinem Shop Probleme.
    Da hat alles wie gewohnt sauber funktioniert.
    Von daher wäre es schon super, wenn der Entwickler das mit aufnehmen würde.

    Von mir schonmal ein dickes Dankeschön. :-)

    Markus

    Timm

    • Fördermitglied
    • Beiträge: 6.318
    Re: Responsive Template für Modified
    Antwort #580 am: 24. Juni 2021, 13:58:29
    Moin

    Der Trick mit der Startseite klappt bei mir nicht. In Safari und Chrome schaffe ich es nie "ls-is-cached" in meinem Testshop zu erzeugen, während es im dev Demoshop immer kommt. In Firefox erscheint "ls-is-cached" auch in meinem Testshop und das ohne den Umweg auf die Startseite.

    Auf jeden Fall wird das "ls-is-cached" nicht abgeschnitten sondern nur das Leerzeichen davor entfernt. Deine Änderung scheint also auch da zu funktionieren.

    Schlag ihm das doch bitte vor. Vielleicht passt es so, oder er sieht da noch was. Wäre doch gut, wenn es dann mit einer neuen Version des Skriptes ohne modified Änderung gehen würde. Danke dir/Euch.

    Gruss Timm

    noRiddle (revilonetz)

    • Experte
    • Beiträge: 13.707
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #581 am: 24. Juni 2021, 14:37:18
    Habe jetzt nicht die Zeit mir das Skript bzgl. ls-is-cached nochmal genau anzusehen.
    Bei mir erscheint die Klasse erst nachdem ich einmal auf einer anderen Seite war und dann zurückkehre, vorausgesetzt, daß ich den Browser-Cache vorher geleert habe.
    Bei solchen Tests sollte man darauf achten immer von genau den gleichen Voraussetzungen auszugehen.

    Habe das Thema als "Feature Request" auf GitHub gepostet: Issue 905
    Wollte es nicht undiskutiert als Bug posten.

    Gruß,
    noRiddle

    noRiddle (revilonetz)

    • Experte
    • Beiträge: 13.707
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #582 am: 04. Juli 2021, 19:05:39
    Offensichtlich konnte ich den Macher bislang nicht hinter dem Ofen vorlocken. Komplettes Ignorieren könnte man sagen.

    Gruß,
    noRiddle

    Tomcraft

    • modified Team
    • Gravatar
    • Beiträge: 46.161
    • Geschlecht:
    Re: Responsive Template für Modified
    Antwort #583 am: 04. Juli 2021, 21:32:34
    Danke für dein Engagement!

    Grüße

    Torsten

    Timm

    • Fördermitglied
    • Beiträge: 6.318
    Re: Responsive Template für Modified
    Antwort #584 am: 10. Juli 2021, 11:51:53
    Moin @Markus

    Im dev Demoshop ist beim Öffnen des Lieferzeit Popups auf dem Smartphone das Wort Information abgeschnitten. In 2.0.5.1 war das Wort noch komplett sichtbar.

    Gruss Timm
    Trade Republic - Provisionsfrei Aktien handeln
    48 Antworten
    40441 Aufrufe
    18. Juni 2017, 11:01:06 von Timm
    1 Antworten
    3085 Aufrufe
    24. April 2018, 09:36:44 von lukass
    10 Antworten
    8832 Aufrufe
    28. November 2018, 23:00:26 von cayuco
    3 Antworten
    1906 Aufrufe
    21. Juli 2018, 14:02:31 von Tomcraft