Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSS/JS inline ausgeben #144

Closed
alxndr-w opened this issue Mar 2, 2021 · 7 comments
Closed

CSS/JS inline ausgeben #144

alxndr-w opened this issue Mar 2, 2021 · 7 comments

Comments

@alxndr-w
Copy link
Member

alxndr-w commented Mar 2, 2021

Der CSS/JS wird nur benötigt, wenn er tatsächlich dargestellt wird. Wenn nicht, braucht es ihn auch nicht. Wenn man ihn braucht, dann sofort.

Entspr. Google PageSpeed Insights würde ich empfehlen, den Code inline auszugeben:

image

@aeberhard
Copy link
Member

@alxndr-w mit der neuen Version ist schon viel optimiert worden. Fontello ist raus, CSS und JS wurde auf jeweils eine (1) Datei eingedampft.

Die Standard-CSS kann man ja in den Einstellungen ausschalten und dann z.B. das CSS inline ins Template schreiben.
Die JS-Datei bastelt mehrere JS-Dateien und das Box-Template zusammen und wird auch gecached.

Kannst ja gerne einen PR beisteuern. Aber dann bitte auf jeden Fall optional als zusätzliche Einstellung in den Addon-Optionen ;-)

Ich bin ja nicht unbedingt ein Freund davon alles in den HTML-Code einzubauen. Google hin oder her.
Siehe auch CSP ... disable inline Scripte ...

@IngoWinter
Copy link
Member

IngoWinter commented Mar 14, 2021

https://github.com/FriendsOfREDAXO/consent_manager/compare/async
Das wäre mein Vorschlag zu der Sache - da hat Google auch kein Problem mit. Das REX_CONSENT_MANAGER[] kommt dabei einfach vor das schliessende body Element.
Edit: ich habe aus dem ursprünglichen "async" im Skriptaufruf ein "defer" gemacht - dann sollte es bei Einbindung im head auch kein Problem geben.

@aeberhard
Copy link
Member

@IngoWinter gerade ausprobiert. Sieht gut aus und funktioniert alles!

@IngoWinter
Copy link
Member

Den Schluck CSS könnte man inline laden wie @alxndr-w schreibt. Dafür müsste man hier: https://github.com/FriendsOfREDAXO/consent_manager/blob/master/fragments/consent_manager_box_cssjs.php#L24 nur ein file_get_contents auf die CSS Datei machen. Damit sparen wir uns zukünftige Page Speed Anfragen - und die werden/würden 100%ig kommen.
Wenns keine Einwände gibt mach ich das gerne.

@aeberhard
Copy link
Member

Ja mach mal Ingo :)

@IngoWinter
Copy link
Member

@alxndr-w magst du mal testen?

@aeberhard
Copy link
Member

Getestet. Top. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants