erp5_web_js_style: add a prerender link

This allows crawlers like wget to also fetch the rendering gadget without executing any JS

erp5_web_js_style: add a prerender link
This allows crawlers like wget to also fetch the rendering gadget without executing any JS
604b0ebd · Romain Courteaud · Rafael Monnerat · f3485432 · 604b0ebd · 604b0ebd
Commit 604b0ebd authored Jan 29, 2021 by Romain Courteaud 🐙 Committed by Rafael Monnerat Feb 05, 2021
6 changed files
--- a/bt5/erp5_web_js_style/SkinTemplateItem/portal_skins/erp5_web_js_style_ui/dialog_main.zpt
+++ b/bt5/erp5_web_js_style/SkinTemplateItem/portal_skins/erp5_web_js_style_ui/dialog_main.zpt
@@ -10,6 +10,7 @@
      dialog_category python: '';
      web_site python: here.getWebSiteValue();
      web_section python: here.getWebSectionValue();
+      no_style_gadget_url python: web_site.getLayoutProperty('configuration_style_gadget_url', default='');
      global_definitions_macros here/global_definitions/macros;">
      <tal:block metal:use-macro="global_definitions_macros/header_definitions" />
 <html>
@@ -28,8 +29,11 @@
  <script src="portal_skins/erp5_xhtml_style/renderjs.js"></script>
  <script src="gadget_global.js"></script>
  <script src="jsstyle.js"></script>
+  <tal:block tal:condition="python: no_style_gadget_url != ''">
+  <link rel="prerender" tal:attributes="href no_style_gadget_url">
+  </tal:block>
  </head>
-  <body tal:attributes="data-nostyle-gadget-url python: web_site.getLayoutProperty('configuration_style_gadget_url', default='')">
+  <body tal:attributes="data-nostyle-gadget-url no_style_gadget_url">
    <tal:block tal:content="structure python: web_site.WebSite_generateNavigationHTML()"></tal:block>
    <main>


--- a/bt5/erp5_web_js_style/SkinTemplateItem/portal_skins/erp5_web_js_style_ui/view_main.zpt
+++ b/bt5/erp5_web_js_style/SkinTemplateItem/portal_skins/erp5_web_js_style_ui/view_main.zpt
@@ -9,6 +9,7 @@
      dummy python: request.set('editable_mode', False);
      web_site python: here.getWebSiteValue();
      web_section python: here.getWebSectionValue();
+      no_style_gadget_url python: web_site.getLayoutProperty('configuration_style_gadget_url', default='');
      global_definitions_macros here/global_definitions/macros;">
      <tal:block metal:use-macro="global_definitions_macros/header_definitions" />
 <html>
@@ -27,8 +28,11 @@
  <script src="portal_skins/erp5_xhtml_style/renderjs.js"></script>
  <script src="gadget_global.js"></script>
  <script src="jsstyle.js"></script>
+  <tal:block tal:condition="python: no_style_gadget_url != ''">
+  <link rel="prerender" tal:attributes="href no_style_gadget_url">
+  </tal:block>
  </head>
-  <body tal:attributes="data-nostyle-gadget-url python: web_site.getLayoutProperty('configuration_style_gadget_url', default='')">
+  <body tal:attributes="data-nostyle-gadget-url no_style_gadget_url">
    <tal:block tal:content="structure python: web_site.WebSite_generateNavigationHTML()"></tal:block>
    <p tal:content="request/portal_status_message | nothing" id="portal_status_message"/>


--- a/bt5/erp5_web_js_style_test/PathTemplateItem/portal_tests/js_style_zuite/testJsStyleDemoStyle.zpt
+++ b/bt5/erp5_web_js_style_test/PathTemplateItem/portal_tests/js_style_zuite/testJsStyleDemoStyle.zpt
@@ -32,6 +32,12 @@
  <td></td>
 </tr>

+<tr>
+  <td>assertElementPresent</td>
+  <td>//head/link[@rel='prerender' and @href='jsstyle_demo.html']</td>
+  <td></td>
+</tr>
+
 <tr>
  <td>waitForElementPresent</td>
  <td>//header/h1[text()='JS Style Demo']</td>

--- a/bt5/erp5_web_js_style_test/PathTemplateItem/portal_tests/js_style_zuite/testJsStyleNoStyle.zpt
+++ b/bt5/erp5_web_js_style_test/PathTemplateItem/portal_tests/js_style_zuite/testJsStyleNoStyle.zpt
@@ -31,6 +31,13 @@
  <td>${base_url}/web_site_module/erp5_web_js_style_test_site/</td>
  <td></td>
 </tr>
+
+<tr>
+  <td>assertElementNotPresent</td>
+  <td>//head/link[@rel='prerender']</td>
+  <td></td>
+</tr>
+
 <tr>
  <td>assertElementPresent</td>
  <td>//nav[@id='sitemap']/a[text()='No Style']</td>

--- a/bt5/erp5_web_js_style_test/PathTemplateItem/portal_tests/js_style_zuite/testJsStyleWebSiteForm.zpt
+++ b/bt5/erp5_web_js_style_test/PathTemplateItem/portal_tests/js_style_zuite/testJsStyleWebSiteForm.zpt
@@ -31,6 +31,11 @@
  <td>${base_url}/web_site_module/erp5_web_js_style_test_site/</td>
  <td></td>
 </tr>
+<tr>
+  <td>assertElementPresent</td>
+  <td>//head/link[@rel='prerender' and @href='jsstyle_demo.html']</td>
+  <td></td>
+</tr>
 <tr>
  <td>waitForElementPresent</td>
  <td>//header/h1[text()='JS Style Demo']</td>

--- a/bt5/erp5_web_js_style_test/PathTemplateItem/portal_tests/js_style_zuite/testJsStyleWebSiteNoStyleForm.zpt
+++ b/bt5/erp5_web_js_style_test/PathTemplateItem/portal_tests/js_style_zuite/testJsStyleWebSiteNoStyleForm.zpt
@@ -31,6 +31,11 @@
  <td>${base_url}/web_site_module/erp5_web_js_style_test_site/</td>
  <td></td>
 </tr>
+<tr>
+  <td>assertElementNotPresent</td>
+  <td>//head/link[@rel='prerender']</td>
+  <td></td>
+</tr>
 <tr>
  <td>waitForTextPresent</td>
  <td>No Style Form</td>