Merge pull request #1222 from alphagov/toolkit-upgrade

Upgrade GOV.UK Template, Frontend Tookit, and Elements

app/assets/javascripts/main.js

@@ -2,8 +2,6 @@ $(() => $("time.timeago").timeago());
 
 $(() => GOVUK.modules.start());
 
-$(() => new GOVUK.SelectionButtons('.block-label input, .sms-message-option input'));
-
 $(() => $('.error-message').eq(0).parent('label').next('input').trigger('focus'));
 
 $(() => $('.banner-dangerous').eq(0).trigger('focus'));

app/assets/javascripts/radioSelect.js

@@ -5,54 +5,57 @@
   let states = {
     'initial': Hogan.compile(`
       <div class="radio-select-column">
-        <label class="block-label js-block-label" for="{{name}}-0">
-          <input checked="checked" id="{{name}}-0" name="{{name}}" type="radio" value=""> Now
-        </label>
+        <div class="multiple-choice js-multiple-choice">
+          <input checked="checked" id="{{name}}-0" name="{{name}}" type="radio" value="">
+          <label class="block-label js-block-label" for="{{name}}-0">Now</label>
+        </div>
       </div>
       <div class="radio-select-column">
         {{#categories}}
-          <input type='button' class='button tertiary-button js-category-button' value='{{.}}' />
+          <input type='button' class='js-category-button' value='{{.}}' />
         {{/categories}}
       </div>
     `),
     'choose': Hogan.compile(`
       <div class="radio-select-column">
-        <label class="block-label js-block-label" for="{{name}}-0">
-          <input checked="checked" id="{{name}}-0" name="{{name}}" type="radio" value="" class="js-initial-option"> Now
-        </label>
+        <div class="multiple-choice js-multiple-choice js-initial-option">
+          <input checked="checked" id="{{name}}-0" name="{{name}}" type="radio" value="">
+          <label for="{{name}}-0">Now</label>
+        </div>
       </div>
       <div class="radio-select-column">
         {{#choices}}
-          <label class="block-label js-block-label" for="{{id}}">
-            <input type="radio" value="{{value}}" id="{{id}}" name="{{name}}" class="js-option" />
-            {{label}}
-          </label>
+          <div class="multiple-choice js-multiple-choice js-option">
+            <input type="radio" value="{{value}}" id="{{id}}" name="{{name}}" />
+            <label for="{{id}}">{{label}}</label>
+          </div>
         {{/choices}}
       </div>
     `),
     'chosen': Hogan.compile(`
       <div class="radio-select-column">
-        <label class="block-label js-block-label" for="{{name}}-0">
-          <input id="{{name}}-0" name="{{name}}" type="radio" value="" class="js-initial-option"> Now
-        </label>
+        <div class="multiple-choice js-multiple-choice js-initial-option">
+          <input id="{{name}}-0" name="{{name}}" type="radio" value="">
+          <label for="{{name}}-0">Now</label>
+        </div>
       </div>
       <div class="radio-select-column">
         {{#choices}}
-          <label class="block-label js-block-label" for="{{id}}">
+          <div class="multiple-choice js-multiple-choice">
             <input checked="checked" type="radio" value="{{value}}" id="{{id}}" name="{{name}}" />
-            {{label}}
-          </label>
+            <label for="{{id}}">{{label}}</label>
+          </div>
         {{/choices}}
       </div>
       <div class="radio-select-column">
-        <input type='button' class='button tertiary-button js-reset-button' value='Choose a different time' />
+        <input type='button' class='category-link js-reset-button' value='Choose a different time' />
       </div>
     `)
   };
 
   let focusSelected = function() {
     setTimeout(
-      () => $('[type=radio]:checked').parent('label').blur().trigger('focus').addClass('selected'),
+      () => $('[type=radio]:checked').next('label').blur().trigger('focus').addClass('selected'),
       10
     );
   };
@@ -62,13 +65,16 @@
     this.start = function(component) {
 
       let $component = $(component);
-      let render = (state, data) => $component.html(states[state].render(data));
+      let render = (state, data) => {
+        $component.html(states[state].render(data));
+        new GOVUK.SelectionButtons('.block-label input');
+      };
       let choices = $('label', $component).toArray().map(function(element) {
         let $element = $(element);
         return {
           'id': $element.attr('for'),
           'label': $.trim($element.text()),
-          'value': $element.find('input').attr('value')
+          'value': $element.prev('input').attr('value')
         };
       });
       let categories = $component.data('categories').split(',');
@@ -95,7 +101,7 @@
           if (!event.pageX) return true;
 
           event.preventDefault();
-          let value = $(this).attr('value');
+          let value = $('input', this).attr('value');
           render('chosen', {
             'choices': choices.filter(
               element => element.value == value

app/assets/stylesheets/app.scss

@@ -46,8 +46,7 @@
 }
 
 .beta-badge {
-  @include phase-tag(beta);
-  background: $govuk-blue;
+  @include phase-tag();
   margin: 10px 0 0 0;
 }
 
@@ -151,22 +150,10 @@ td {
   margin-bottom: 5px;
 }
 
-a[rel='external'] {
-  @include external-link-default;
-  @include external-link-16;
-  @include media(tablet) {
-    @include external-link-19;
-  }
-}
-
 .hint {
   color: $secondary-text-colour;
 }
 
-.button {
-  padding: 0.3em 0.6em 0.2em 0.6em;
-}
-
 .list-bullet {
 
   .list-bullet {
@@ -256,10 +243,16 @@ details .arrow {
   margin-top: 5px;
 }
 
-.block-label input[disabled] {
-  opacity: 0.5;
-}
-
 #content.override-elements-content {
   padding-bottom: 0;
 }
+
+.multiple-choice input:disabled+label {
+  opacity: 1;
+  color: $secondary-text-colour;
+  cursor: default;
+}
+
+.multiple-choice input:disabled {
+  opacity: 0.5;
+}

app/assets/stylesheets/components/radio-select.scss

@@ -5,19 +5,25 @@
     display: inline-block;
     vertical-align: top;
 
-    .block-label {
+    .multiple-choice {
       margin-right: 5px;
-      padding-right: $gutter - 10px;
+      padding-right: 10px;
       padding-left: 54px - 10px;
     }
 
   }
 
-  .tertiary-button {
+  .js-reset-button,
+  .js-category-button {
+    background: none;
+    text-decoration: underline;
+    color: $link-colour;
+    //font-weight: bold;
+    border: none;
     display: inline-block;
     vertical-align: top;
     width: auto;
-    padding: 20px $gutter-half 15px $gutter-half;
+    padding: 7px 20px 7px 10px;
     margin-right: 5px;
   }
 
@@ -26,12 +32,12 @@
     height: 60px;
     overflow: visible;
 
-    .block-label {
+    .multiple-choice {
       display: none;
     }
 
-    .js-block-label {
-      display: inline-block;
+    .js-multiple-choice {
+      display: block;
     }
 
   }

app/assets/stylesheets/main.scss

@@ -24,7 +24,7 @@ $path: '/static/images/';
 @import 'elements/elements-typography';
 @import 'elements/forms';
 @import 'elements/forms/form-validation';
-@import 'elements/forms/form-block-labels';
+@import 'elements/forms/form-multiple-choice';
 @import 'elements/forms/form-validation';
 @import 'elements/icons';
 @import 'elements/layout';

app/templates/components/checkbox.html

@@ -1,29 +1,22 @@
 {% macro checkbox(
   field,
   hint=False,
-  help_link=None,
-  help_link_text=None,
-  width='2-3',
-  suffix=None,
-  block=False
+  width='2-3'
 ) %}
-    <label class="{% if block %}block-label{% endif %}" for="{{ field.name }}">
-        {{ field()}}
-        {{ field.label.text }}
-        {% if hint %}
-            <span class="form-hint">
-                {{ hint }}
-            </span>
-        {% endif %}
-        {% if field.errors %}
-            <span class="error-message">
-                {{ field.errors[0] }}
-            </span>
-        {% endif %}
-        {% if help_link and help_link_text %}
-            <p class="textbox-help-link">
-                <a href='{{ help_link }}'>{{ help_link_text }}</a>
-            </p>
-        {% endif %}
+  <div class="multiple-choice">
+    <input
+      id="{{ field.id }}" name="{{ field.name }}" type="checkbox" value="{{ field.data }}"
+      {% if field.checked %}
+        checked
+      {% endif %}
+    >
+    <label for="{{ field.id }}">
+      {{ field.label.text }}
+      {% if hint %}
+        <div class="hint">
+          {{ hint }}
+        </div>
+      {% endif %}
     </label>
+  </div>
 {% endmacro %}

app/templates/components/file-upload.html

@@ -1,5 +1,5 @@
 {% macro file_upload(field, button_text="Choose file", alternate_link=None, alternate_link_text=None) %}
-  <form method="post" enctype="multipart/form-data" class="{% if field.errors %}error{% endif %}" data-module="file-upload">
+  <form method="post" enctype="multipart/form-data" class="{% if field.errors %}form-group-error{% endif %}" data-module="file-upload">
     <label class="file-upload-label" for="{{ field.name }}">
       <span class="visually-hidden">{{ field.label.text }}</span>
       {% if hint %}

app/templates/components/list-entry.html

@@ -8,7 +8,7 @@
   {% if error %}
     <div class="validation-wrapper">
   {% endif %}
-    <fieldset class="form-group{% if field.errors %} error{% endif %}" id="{{ field.name }}">
+    <fieldset class="form-group{% if field.errors %} form-group-error{% endif %}" id="{{ field.name }}">
       <legend>
         <span class="form-label">
           {{ field.label.text }}

app/templates/components/radios.html

@@ -4,7 +4,7 @@
   disable=[],
   option_hints={}
 ) %}
-  <div class="form-group {% if field.errors %} error{% endif %}">
+  <div class="form-group {% if field.errors %} form-group-error{% endif %}">
     <fieldset>
       <legend class="form-label">
         {{ field.label.text|safe }}
@@ -15,7 +15,7 @@
         {% endif %}
       </legend>
       {% for option in field %}
-        <label class="block-label" for="{{ option.id }}">
+        <div class="multiple-choice">
           <input
             id="{{ option.id }}" name="{{ option.name }}" type="radio" value="{{ option.data }}"
             {% if option.data in disable %}
@@ -25,13 +25,15 @@
               checked
             {% endif %}
           >
-          {{ option.label.text }}
-          {% if option_hints[option.data] %}
-            <div class="block-label-hint">
-              {{ option_hints[option.data] }}
-            </div>
-          {% endif %}
-        </label>
+          <label class="block-label" for="{{ option.id }}">
+            {{ option.label.text }}
+            {% if option_hints[option.data] %}
+              <div class="block-label-hint">
+                {{ option_hints[option.data] }}
+              </div>
+            {% endif %}
+          </label>
+        </div>
       {% endfor %}
     </fieldset>
   </div>
@@ -43,7 +45,7 @@
  hint=None,
  wrapping_class='form-group'
 ) %}
- <div class="{{ wrapping_class }} {% if field.errors %} error{% endif %}">
+ <div class="{{ wrapping_class }} {% if field.errors %} form-group-error{% endif %}">
    <fieldset>
      <legend class="form-label">
        {{ field.label.text }}
@@ -56,10 +58,12 @@
      <div class="radio-select" data-module="radio-select" data-categories="{{ field.categories|join(',') }}">
        <div class="radio-select-column">
        {% for option in field %}
-         <label class="block-label" for="{{ option.id }}">
+         <div class="multiple-choice">
            {{ option }}
-           {{ option.label.text }}
-         </label>
+           <label for="{{ option.id }}">
+             {{ option.label.text }}
+           </label>
+         </div>
          {% if loop.first %}
        </div>
        <div class="radio-select-column">
@@ -77,7 +81,7 @@
   hint=None,
   branding_dict={}
 ) %}
-  <div class="form-group {% if field.errors %} error{% endif %}">
+  <div class="form-group {% if field.errors %} form-group-error{% endif %}">
     <fieldset>
       <legend class="form-label">
         {{ field.label.text }}
@@ -88,7 +92,7 @@
         {% endif %}
       </legend>
       {% for value, option, checked in field.iter_choices() %}
-        <label class="block-label" for="{{ field.name }}-{{ loop.index }}">
+        <div class="multiple-choice">
           <input
             type="radio"
             name="{{ field.name }}"
@@ -96,17 +100,19 @@
             value="{{ value }}"
             {% if checked %}checked="checked"{% endif %}
           />
-          {% if branding_dict.get(value, {}).get('colour') %}
-            <span style="background: {{ branding_dict[value].colour }}; display: inline-block; width: 3px; height: 27px"></span>
-          {% endif %}
-          {% if branding_dict.get(value, {}).get('logo') %}
-            <img
-              src="{{ branding_dict[value].logo }}"
-              height="27"
-            />
-          {% endif %}
-          {{option}}
-        </label>
+          <label class="block-label" for="{{ field.name }}-{{ loop.index }}">
+            {% if branding_dict.get(value, {}).get('colour') %}
+              <span style="background: {{ branding_dict[value].colour }}; display: inline-block; width: 3px; height: 27px"></span>
+            {% endif %}
+            {% if branding_dict.get(value, {}).get('logo') %}
+              <img
+                src="{{ branding_dict[value].logo }}"
+                height="27"
+              />
+            {% endif %}
+            {{option}}
+          </label>
+        </div>
       {% endfor %}
     </fieldset>
   </div>

app/templates/components/table.html

@@ -1,7 +1,7 @@
 {% from "components/big-number.html" import big_number %}
 
 {% macro mapping_table(caption='', field_headings=[], field_headings_visible=True, caption_visible=True) -%}
-  <table class="table">
+  <table class="table table-font-xsmall">
     <caption class="heading-medium table-heading{{ ' visuallyhidden' if not caption_visible}}">
       {{ caption }}
     </caption>

app/templates/components/textbox.html

@@ -11,7 +11,7 @@
   safe_error_message=False,
   rows=8
 ) %}
-  <div class="form-group{% if field.errors %} error{% endif %}" {% if autofocus %}data-module="autofocus"{% endif %}>
+  <div class="form-group{% if field.errors %} form-group-error{% endif %}" {% if autofocus %}data-module="autofocus"{% endif %}>
     <label class="form-label" for="{{ field.name }}">
       {% if label %}
         {{ label }}
@@ -29,10 +29,20 @@
         </span>
       {% endif %}
     </label>
+    {%
+      if highlight_tags
+    %}
+      {% set field_class = 'form-control-{} textbox-highlight-textbox'.format(width) %}
+    {% else %}
+      {% set field_class = 'form-control-{} {}'.format(width, 'textbox-right-aligned' if suffix else '') %}
+    {% endif %}
+    {%
+      set field_class = 'form-control ' + field_class + (
+        ' form-control-error' if field.errors else ''
+      )
+    %}
     {{ field(**{
-      'class':
-        'form-control form-control-{} textbox-highlight-textbox'.format(width) if highlight_tags else
-        'form-control form-control-{} {}'.format(width, 'textbox-right-aligned' if suffix else ''),
+      'class': field_class,
       'data-module': 'highlight-tags' if highlight_tags else '',
       'rows': rows|string
     }) }}

app/templates/views/manage-users/permissions.html

@@ -4,9 +4,9 @@
   <legend class="form-label">
     Permissions
   </legend>
-  {{ checkbox(form.send_messages, block=True) }}
-  {{ checkbox(form.manage_service, block=True) }}
-  {{ checkbox(form.manage_api_keys, block=True) }}
+  {{ checkbox(form.send_messages) }}
+  {{ checkbox(form.manage_service) }}
+  {{ checkbox(form.manage_api_keys) }}
 </fieldset>
 
 <div class="form-group">
@@ -18,4 +18,4 @@
     <li>history of sent messages</li>
     <li>who the other team members are</li>
   </ul>
-</div>
+</div>

gulpfile.babel.js

@@ -54,7 +54,6 @@ gulp.task('copy:govuk_template:images', () => gulp.src(paths.template + 'assets/
 gulp.task('javascripts', () => gulp
   .src([
     paths.toolkit + 'javascripts/govuk/modules.js',
-    paths.toolkit + 'javascripts/govuk/selection-buttons.js',
     paths.src + 'javascripts/detailsPolyfill.js',
     paths.src + 'javascripts/apiKey.js',
     paths.src + 'javascripts/autofocus.js',

package.json

@@ -21,9 +21,9 @@
     "babel-core": "6.3.26",
     "babel-preset-es2015": "6.3.13",
     "diff-dom": "2.1.0",
-    "govuk-elements-sass": "1.1.1",
-    "govuk_frontend_toolkit": "4.6.0",
-    "govuk_template_jinja": "https://github.com/alphagov/govuk_template/releases/download/v0.19.1/jinja_govuk_template-0.19.1.tgz",
+    "govuk-elements-sass": "3.0.x",
+    "govuk_frontend_toolkit": "5.2.x",
+    "govuk_template_jinja": "0.19.x",
     "gulp": "3.9.0",
     "gulp-add-src": "0.2.0",
     "gulp-babel": "6.1.1",