Bladeren bron

switch the script tags (js) to defer mode

Olivier Massot 5 jaren geleden
bovenliggende
commit
51f105cc24

+ 10 - 5
ot_templating/Resources/Private/Partials/Classic/Assets.html

@@ -39,19 +39,24 @@ Assets included with the VHS viewhelpers
 
 <f:comment><!-- Script assets --></f:comment>
 <v:asset.script name="jquery"
-                path="{assets_dir}/script/jquery-3.4.1.min.js"/>
+                path="{assets_dir}/script/jquery-3.4.1.min.js"
+                defer="1"/>
 
 <v:asset.script name="datepicker"
-                path="{assets_dir}/script/jquery-ui.min.js"/>
+                path="{assets_dir}/script/jquery-ui.min.js"
+                defer="1"/>
 
 <v:asset.script name="slick"
                 path="{assets_dir}/script/slick.min.js"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="leaflet"
-                path="{assets_dir}/script/leaflet.js"/>
+                path="{assets_dir}/script/leaflet.js"
+                defer="1"/>
 
 <v:asset.script name="main"
                 path="{assets_dir}/script/main.js"
                 dependencies="jquery,datepicker,slick,leaflet"
-                standalone="1"/>
+                standalone="1"
+                defer="1"/>

+ 46 - 23
ot_templating/Resources/Private/Partials/Modern/Assets.html

@@ -153,116 +153,139 @@ Assets included with the VHS viewhelpers
 
 <v:asset.script name="jquery"
                 path="{assets_dir}/script/jquery-3.3.1.min.js"
-                standalone="{force_standalone}"/>
+                standalone="{force_standalone}"
+                defer="1"/>
 
 <f:comment><!-- includes all plugins ; can be removed if none of those plugins are used --></f:comment>
 <v:asset.script name="plugins-jquery"
                 path="{assets_dir}/script/plugins-jquery.js"
                 standalone="{force_standalone}"
-                dependencies="jquery" />
+                dependencies="jquery"
+                defer="1" />
 
 <v:asset.script name="recaptcha"
                 path="https://www.google.com/recaptcha/api.js"
                 external="1"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="jquery-tools"
                 path="{assets_dir}/script/revolution/jquery.themepunch.tools.min.js"
-                standalone="{force_standalone}"/>
+                standalone="{force_standalone}"
+                defer="1"/>
 
 <v:asset.script name="jquery-revolution"
                 path="{assets_dir}/script/revolution/jquery.themepunch.revolution.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-actions"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.actions.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-carousel"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.carousel.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-kenburn"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.kenburn.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-layeranimation"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.layeranimation.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-migration"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.migration.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-parallax"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.parallax.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-navigation"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.navigation.min.js"
-                standalone="{force_standalone}"/>
+                standalone="{force_standalone}"
+                defer="1"/>
 
 <v:asset.script name="revolution-slideanims"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.slideanims.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-navigation"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.navigation.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-video"
                 path="{assets_dir}/script/revolution/extensions/revolution.extension.video.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="revolution-custom"
                 path="{assets_dir}/script/revolution/revolution-custom.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="moment-js"
                 path="{assets_dir}/script/bootstrap-datetimepicker/moment-datepicker.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="datetimepicker-js"
                 path="{assets_dir}/script/bootstrap-datetimepicker/bootstrap-datetimepicker.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery,moment-js"/>
+                dependencies="jquery,moment-js"
+                defer="1"/>
 
 <v:asset.script name="slick-js"
                 path="{assets_dir}/script/slick/slick.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="leaflet-js"
                 path="{assets_dir}/script/leaflet/leaflet.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="magnific-popup"
                 path="{assets_dir}/script/magnific-popup/jquery.magnific-popup.min.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="custom-js"
                 path="{assets_dir}/script/custom.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>
 
 <v:asset.script name="custom-ot-js"
                 path="{assets_dir}/script/custom-ot.js"
                 standalone="{force_standalone}"
-                dependencies="jquery"/>
+                dependencies="jquery"
+                defer="1"/>