Contao-Camp 2024
Ergebnis 1 bis 14 von 14

Thema: Google's lighthouse meckert script an

  1. #1
    Community-Moderator & Contao-Urgestein Avatar von derRenner
    Registriert seit
    23.10.2012.
    Ort
    hoRn|waldviertel
    Beiträge
    2.849
    Contao-Projekt unterstützen

    Support Contao

    Standard Google's lighthouse meckert script an

    Hallo,

    Lighthouse über Chrome meckert zwei Dinge im template an - es geht um folgende Zeile in mod_visitors_fe_invisible.html5
    PHP-Code:
    document.write('<img class="invisible" src="<?= $this->route('visitors_frontend_screencount'?>?vcid=<?= $visitor['VisitorsKatID']; ?>&scrw='+sWidth+'&scrh='+sHeight+'&scriw='+iWidth+'&scrih='+iHeight+'">');
    ... kein 'alt' für Screenreader
    Als Abhilfe habe ich einen dummy-alt eingefügt
    PHP-Code:
    document.write('<img class="invisible" alt="invisible" src="<?= $this->route('visitors_frontend_screencount'?>?vcid=<?= $visitor['VisitorsKatID']; ?>&scrw='+sWidth+'&scrh='+sHeight+'&scriw='+iWidth+'&scrih='+iHeight+'">');
    ... und 'document.write'
    For users on slow connections, external scripts dynamically injected via `document.write()` can delay page load by tens of seconds

    Gibt es hier Verbesserungspotential?
    Grüsse
    Bernhard


  2. #2
    Contao-Urgestein
    Registriert seit
    29.10.2009.
    Ort
    Magdeburg
    Beiträge
    2.020
    Partner-ID
    626
    User beschenken
    Wunschliste

    Standard

    Vermeide document.write und nutze die Dom Manipulation API, https://wiki.selfhtml.org/wiki/JavaS...M-Manipulation

  3. #3
    Contao-Yoda Avatar von MacKP
    Registriert seit
    15.06.2009.
    Ort
    Duisburg
    Beiträge
    13.292
    User beschenken
    Wunschliste
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Wenn das nur ein Zählpixel ist, dann sollte die Alt-Angabe auch leer bleiben. Lighthouse ist nur ein Tool. Die Ergebnisse davon sollte man schon interpretieren ;-)
    Jetzt hast du es im Grunde für Screenreader Nutzer verschlimmert, weil die jetzt invisible vorgelesen bekommen. Vorher wars genau richtig so ;-)

    Viele Grüße
    Contao Pool | C-C-A | MetaModels | [Internetseite -> Mediendepot Ruhr]
    [Arbeitet bei -> Paus Design & Medien]
    "I can EXPLAIN it to you, but I can't UNDERSTAND it for you."

  4. #4
    Contao-Urgestein Avatar von do_while
    Registriert seit
    15.06.2009.
    Ort
    Berlin | Deutschland
    Beiträge
    3.612
    Partner-ID
    1081
    User beschenken
    Wunschliste
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Naja - genau richtig war es nicht,
    denn beim <img>-Tag ist doch wohl das alt-Attribut mandatory, es müsste also alt="" lauten.

  5. #5
    Contao-Yoda Avatar von MacKP
    Registriert seit
    15.06.2009.
    Ort
    Duisburg
    Beiträge
    13.292
    User beschenken
    Wunschliste
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Zitat Zitat von do_while Beitrag anzeigen
    Naja - genau richtig war es nicht,
    denn beim <img>-Tag ist doch wohl das alt-Attribut mandatory, es müsste also alt="" lauten.
    Da hast du natürlich recht. Ich hatte übersehen, dass da gar kein alt-Attribut angegeben war.

    Viele Grüße
    Contao Pool | C-C-A | MetaModels | [Internetseite -> Mediendepot Ruhr]
    [Arbeitet bei -> Paus Design & Medien]
    "I can EXPLAIN it to you, but I can't UNDERSTAND it for you."

  6. #6
    Wandelndes Contao-Lexikon Avatar von BugBuster
    Registriert seit
    15.06.2009.
    Ort
    Berlin
    Beiträge
    10.494
    User beschenken
    Wunschliste

    Standard

    alt Tag kommt mit der nächsten Version
    https://github.com/BugBuster1701/con...ndle/issues/74
    Grüße, BugBuster
    "view source" is your guide.
    Danke an alle Amazon Wunschlisten Erfüller

  7. #7
    Community-Moderator & Contao-Urgestein Avatar von derRenner
    Registriert seit
    23.10.2012.
    Ort
    hoRn|waldviertel
    Beiträge
    2.849
    Contao-Projekt unterstützen

    Support Contao

    Standard

    schaffst du auch die Alternative zu 'document.write'?
    Grüsse
    Bernhard


  8. #8
    Wandelndes Contao-Lexikon Avatar von BugBuster
    Registriert seit
    15.06.2009.
    Ort
    Berlin
    Beiträge
    10.494
    User beschenken
    Wunschliste

    Standard

    muss ich mal anschauen wie das geht, Javascript ist nicht mein Ding.
    Grüße, BugBuster
    "view source" is your guide.
    Danke an alle Amazon Wunschlisten Erfüller

  9. #9
    Wandelndes Contao-Lexikon Avatar von BugBuster
    Registriert seit
    15.06.2009.
    Ort
    Berlin
    Beiträge
    10.494
    User beschenken
    Wunschliste

    Standard

    gehts nur darum das document.write zu ersetzen oder muss das unbedingt über "document.addEventListener('DOMContentLoaded'. ..." laufen um das Laden der Seite nicht zu bremsen?
    Grüße, BugBuster
    "view source" is your guide.
    Danke an alle Amazon Wunschlisten Erfüller

  10. #10
    Community-Moderator & Contao-Urgestein Avatar von derRenner
    Registriert seit
    23.10.2012.
    Ort
    hoRn|waldviertel
    Beiträge
    2.849
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Zitat Zitat von BugBuster Beitrag anzeigen
    gehts nur darum das document.write zu ersetzen oder muss das unbedingt über "document.addEventListener('DOMContentLoaded'. ..." laufen um das Laden der Seite nicht zu bremsen?
    Primär geht's aktuell darum das Laden der Seite zu beschleunigen - ich vermute, dass ich hiermit nicht der einzige bin, der deine Erweiterung einsetzt und Lighthouse drüber laufen lässt.
    Was meckert Lighthouse genau:
    Uses document.write()
    For users on slow connections, external scripts dynamically injected via `document.write()` can delay page load by tens of seconds. Learn more.
    Using document.write() can delay the display of page content by tens of seconds and is particularly problematic for users on slow connections. Chrome therefore blocks the execution of document.write() in many cases, meaning you can't rely on it.
    Grüsse
    Bernhard


  11. #11
    Wandelndes Contao-Lexikon Avatar von BugBuster
    Registriert seit
    15.06.2009.
    Ort
    Berlin
    Beiträge
    10.494
    User beschenken
    Wunschliste

    Standard

    ok, hab da was gebastelt, bin grad am testen, sieht im Quelltext schon mal ok aus.

    Nachtrag: leider ein Logikfehler, ein img Tag nachträglich einfügen über DOMContentLoaded bringt ja nichts, das wird ja nicht mehr angefordert.
    Geändert von BugBuster (21.02.2020 um 20:01 Uhr)
    Grüße, BugBuster
    "view source" is your guide.
    Danke an alle Amazon Wunschlisten Erfüller

  12. #12
    Wandelndes Contao-Lexikon Avatar von BugBuster
    Registriert seit
    15.06.2009.
    Ort
    Berlin
    Beiträge
    10.494
    User beschenken
    Wunschliste

    Standard

    Bin am testen, läuft nun asynchron über XMLHttpRequest. Damit erledigt sich auch der alt Tag beim Image, denn das ist nun kein Bild mehr was da als Antwort kommt.
    https://github.com/BugBuster1701/con...ndle/issues/75
    Grüße, BugBuster
    "view source" is your guide.
    Danke an alle Amazon Wunschlisten Erfüller

  13. #13
    Wandelndes Contao-Lexikon Avatar von BugBuster
    Registriert seit
    15.06.2009.
    Ort
    Berlin
    Beiträge
    10.494
    User beschenken
    Wunschliste
    Grüße, BugBuster
    "view source" is your guide.
    Danke an alle Amazon Wunschlisten Erfüller

  14. #14
    Community-Moderator & Contao-Urgestein Avatar von derRenner
    Registriert seit
    23.10.2012.
    Ort
    hoRn|waldviertel
    Beiträge
    2.849
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Vielen DANK @BugBuster!
    soeben ein Update gemacht und Lighthouse nochmals drüber laufen lassen » Best Practices zeigt nun 100%
    Angehängte Grafiken Angehängte Grafiken
    Grüsse
    Bernhard


Aktive Benutzer

Aktive Benutzer

Aktive Benutzer in diesem Thema: 1 (Registrierte Benutzer: 0, Gäste: 1)

Lesezeichen

Lesezeichen

Berechtigungen

  • Neue Themen erstellen: Nein
  • Themen beantworten: Nein
  • Anhänge hochladen: Nein
  • Beiträge bearbeiten: Nein
  •