search

dklemmt / contao_dk_mmenu

Implementation of mmenu as extension for Contao Open Source CMS
15 stars 16 forks source link

Improvements #62

Open exscorp opened 3 years ago

exscorp commented 3 years ago

Hallo, habe heute die extension bei einer größeren Seite eingebaut und nach einem Lighthouse/web.dev Test ging der score fast in jedem Bereich runter.

Links to cross-origin destinations are unsafe Im nav-Template fehlt <?= $item['rel'] ?> aus dem Standard-Contao-Template, somit wird den target="_blank" Seiten kein rel="noopener" übergeben.

Links are not crawlable Das erste Element oben im Menü mit dem Namen "Menü" ist ein Link ohne funktion, hier meckert der Test. Hier könnte man bei diesem ersten Element den <a class="mm-navbar__title"></a> Tag entfernen und einfach <span>Menü</span> lassen. Leider kann ich das in keinem Template finden/ändern. Hier geht z. B. der score nur wegen diesen einen "Fehler" von 100 auf 92.

Eliminate render-blocking resources Besteht die Möglichkeit, das die scripte in den Body-Tag per default kommen, ohne es manuell umschreiben zu müssen?

fritzmg commented 3 years ago

Pull Requests are always welcome ;)

exscorp commented 3 years ago

Bis auf den ersten Punkt wüsste ich nicht wie man das "korrigieren" könnte, bin kein Programmierer. Sonst hätte ich ein Pull Request gemacht 😀

fritzmg commented 3 years ago

Unfortunately this repository is not really actively maintained anymore bei either me or Dirk. I can still review and merge Pull Requests if needed though, when provided.

rabauss commented 1 year ago

The template error should be solved with v4.0.0 🚀