Review fixes

mozilla · May 28, 2024 · fe5790d · fe5790d
1 parent 1b2c234
commit fe5790d
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 6 deletions.
diff --git a/bedrock/base/templates/base-protocol.html b/bedrock/base/templates/base-protocol.html
@@ -121,20 +121,23 @@
     {% block sentry_client %}
       {% if settings.SENTRY_FRONTEND_DSN and switch('sentry-js') %}
         <!--[if !IE]><!-->
+          {# Sentry only runs on evergreen browsers so we exclude it from IE. #}
           {{ js_bundle('sentry') }}
         <!--<![endif]-->
       {% endif %}
     {% endblock %}
 
     {% block site_js %}
       <!--[if !IE]><!-->
+        {# Standard site JS bundles served to all evergreen browsers plus IE 10 and above. #}
         {{ js_bundle('lib') }}
         {{ js_bundle('ui') }}
         {{ js_bundle('fxa') }}
         {{ js_bundle('data') }}
       <!--<![endif]-->
 
       <!--[if IE 9]>
+        {# IE9 has its own lib JS bundle containing dependencies for anaytics and consent management. #}
         {{ js_bundle('lib-ie') }}
       <![endif]-->
     {% endblock %}
@@ -143,12 +146,14 @@
     {% block stub_attribution %}
       {% if settings.STUB_ATTRIBUTION_RATE %}
         <!--[if IE 9]><!-->
+          {# Stub attribution is run on all evergreen browsers plus IE9 and above. #}
           {{ js_bundle('stub-attribution') }}
         <!--<![endif]-->
       {% endif %}
     {% endblock %}
 
     <!--[if !IE]><!-->
+    {# Glean only runs on evergreen browsers so we exclude it from IE. #}
     {% block glean %}
       {% if switch('glean-analytics') %}
         {{ js_bundle('glean') }}
@@ -162,6 +167,7 @@
 
     {% block consent_banner_js %}
     <!--[if IE 9]><!-->
+      {# Consent banner is shown to all evergreen browsers plus IE9 and above #}
       {{ js_bundle('consent-banner') }}
     <!--<![endif]-->
     {% endblock %}

diff --git a/bedrock/mozorg/templates/mozorg/robots.txt b/bedrock/mozorg/templates/mozorg/robots.txt
@@ -6,6 +6,5 @@ disallow: /*/firstrun/
 disallow: /*/newsletter/existing/
 disallow: /*/whatsnew/
 disallow: /*/etc/
-disallow: /*/privacy/websites/cookie-settings/
 {% endif -%}
 Sitemap: {{ request.scheme }}://{{ request.get_host() }}/sitemap.xml
diff --git a/bedrock/privacy/templates/privacy/cookie-settings.html b/bedrock/privacy/templates/privacy/cookie-settings.html
@@ -44,11 +44,6 @@
 
       <h1>{{ ftl('cookie-settings-page-title') }}</h1>
 
-      {# noscript warning to enable JavaScript #}
-      <noscript>
-        <p><strong>{{ ftl('cookie-settings-turn-on-javascript') }}</strong></p>
-      </noscript>
-
       <p>{{ ftl('cookie-settings-explainer') }}</p>
 
       <p>{{ ftl('cookie-settings-page-intro-v2', fallback='cookie-settings-page-intro') }}</p>
@@ -158,6 +153,11 @@ <h4 class="cookie-summary-title">
           {{ ftl('cookie-settings-save-changes') }}
         </button>
 
+        {# noscript warning to enable JavaScript #}
+        <noscript>
+          <p><strong>{{ ftl('cookie-settings-turn-on-javascript') }}</strong></p>
+        </noscript>
+
         <p class="hidden cookie-consent-form-submit-success" tabindex="-1">{{ ftl('cookie-settings-your-cookie-settings-have') }}</p>
       </div>
 
@@ -181,6 +181,7 @@ <h2>{{ ftl('cookie-settings-privacy-heading') }}</h2>
 {% block consent_banner_js %}
   {{ super() }}
   <!--[if IE 9]> -->
+    {# Cookie settings page is supported on all evergreen browsers plus IE9 and above #}
     {{ js_bundle('cookie-settings-form') }}
   <!--<![endif]-->
 {% endblock %}