govuk_frontend_toolkit 4.18.3 → 4.18.4

data/app/assets/spec/unit/shim-links-with-button-role.spec.js

@@ -1,36 +1,41 @@
-describe("shim-links-with-button-role", function () {
-  var $body;
-  var $buttonLink;
-  var keyupEvent;
+/* global describe it expect beforeEach afterEach */
+
+var $ = window.jQuery
+
+describe('shim-links-with-button-role', function () {
+  'use strict'
+  var GOVUK = window.GOVUK
+
+  var $buttonLink
+  var keyupEvent
 
   beforeEach(function () {
-    $buttonLink = $('<a role="button">Button</a>');
-    $buttonLink.on('click',function(){
-      $buttonLink.addClass('clicked');
-    });
-    $(document.body).append($buttonLink);
-    keyupEvent = $.Event('keyup');
-    keyupEvent.target = $buttonLink.get(0);
-    GOVUK.shimLinksWithButtonRole.init();
-  });
+    $buttonLink = $('<a role="button">Button</a>')
+    $buttonLink.on('click', function () {
+      $buttonLink.addClass('clicked')
+    })
+    $(document.body).append($buttonLink)
+    keyupEvent = $.Event('keyup')
+    keyupEvent.target = $buttonLink.get(0)
+    GOVUK.shimLinksWithButtonRole.init()
+  })
 
   afterEach(function () {
-    $buttonLink.remove();
-    $(document).off('keyup');
-  });
+    $buttonLink.remove()
+    $(document).off('keyup')
+  })
 
-  it('should trigger event on space', function(){
+  it('should trigger event on space', function () {
     // Ideally we’d test the page loading functionality but that seems hard to
     // do within a Jasmine context. Settle for checking a bound event trigger.
-    keyupEvent.which = 32; // Space character
-    $(document).trigger(keyupEvent);
-    expect($buttonLink.hasClass('clicked')).toBe(true);
-  });
-
-  it('should not trigger event on tab', function(){
-    keyupEvent.which = 9; // Tab character
-    $(document).trigger(keyupEvent);
-    expect($buttonLink.hasClass('clicked')).toBe(false);
-  });
+    keyupEvent.which = 32 // Space character
+    $(document).trigger(keyupEvent)
+    expect($buttonLink.hasClass('clicked')).toBe(true)
+  })
 
-});
+  it('should not trigger event on tab', function () {
+    keyupEvent.which = 9 // Tab character
+    $(document).trigger(keyupEvent)
+    expect($buttonLink.hasClass('clicked')).toBe(false)
+  })
+})

data/app/assets/spec/unit/show-hide-content.spec.js

@@ -1,8 +1,12 @@
+/* global describe it expect beforeEach afterEach jasmine */
+
+var $ = window.jQuery
+
 describe('show-hide-content', function () {
   'use strict'
+  var GOVUK = window.GOVUK
 
   beforeEach(function () {
-
     // Sample markup
     this.$content = $(
 

data/app/assets/spec/unit/stick-at-top-when-scrolling.spec.js

@@ -1,121 +1,118 @@
-describe("stick-at-top-when-scrolling", function(){
-  var $stickyElement,
-      $stickyWrapper;
-
-  beforeEach(function(){
-    $stickyElement = $('<div class="stick-at-top-when-scrolling"></div>');
-    $stickyWrapper = $('<div>').append($stickyElement);
-
-    $('body').append($stickyWrapper);
-  });
-
-  afterEach(function(){
-    $stickyWrapper.remove();
-  });
-
-  describe('when stick is called', function(){
-
-    it('should add fixed class on stick', function(){
-      expect(!$stickyElement.hasClass('content-fixed')).toBe(true);
-      GOVUK.stickAtTopWhenScrolling.stick($stickyElement);
-      expect($stickyElement.hasClass('content-fixed')).toBe(true);
-    });
-
-    it('should insert shim when sticking the element', function(){
-      expect($('.shim').length).toBe(0);
-      GOVUK.stickAtTopWhenScrolling.stick($stickyElement);
-      expect($('.shim').length).toBe(1);
-    });
-
-    it('should insert shim with minimum height', function(){
-      GOVUK.stickAtTopWhenScrolling.stick($stickyElement);
-      expect($('.shim').height()).toBe(1);
-    });
-
-  });
-
-  describe('when release is called', function(){
-
-    it('should remove fixed class', function(){
-      $stickyElement.addClass('content-fixed');
-      GOVUK.stickAtTopWhenScrolling.release($stickyElement);
-      expect($stickyElement.hasClass('content-fixed')).toBe(false);
-    });
-
-    it('should remove the shim', function(){
-      $stickyElement = $('<div class="stick-at-top-when-scrolling content-fixed"></div>');
-      GOVUK.stickAtTopWhenScrolling.release($stickyElement);
-      expect($('.shim').length).toBe(0);
-    });
-
-  });
-
-  describe('for larger screens (>768px)', function(){
-
-    beforeEach(function() {
-      GOVUK.stickAtTopWhenScrolling.getWindowPositions = function(){
+/* global describe it expect beforeEach afterEach */
+
+var $ = window.jQuery
+
+describe('stick-at-top-when-scrolling', function () {
+  'use strict'
+  var GOVUK = window.GOVUK
+
+  var $stickyElement
+  var $stickyWrapper
+
+  beforeEach(function () {
+    $stickyElement = $('<div class="stick-at-top-when-scrolling"></div>')
+    $stickyWrapper = $('<div>').append($stickyElement)
+
+    $('body').append($stickyWrapper)
+  })
+
+  afterEach(function () {
+    $stickyWrapper.remove()
+  })
+
+  describe('when stick is called', function () {
+    it('should add fixed class on stick', function () {
+      expect(!$stickyElement.hasClass('content-fixed')).toBe(true)
+      GOVUK.stickAtTopWhenScrolling.stick($stickyElement)
+      expect($stickyElement.hasClass('content-fixed')).toBe(true)
+    })
+
+    it('should insert shim when sticking the element', function () {
+      expect($('.shim').length).toBe(0)
+      GOVUK.stickAtTopWhenScrolling.stick($stickyElement)
+      expect($('.shim').length).toBe(1)
+    })
+
+    it('should insert shim with minimum height', function () {
+      GOVUK.stickAtTopWhenScrolling.stick($stickyElement)
+      expect($('.shim').height()).toBe(1)
+    })
+  })
+
+  describe('when release is called', function () {
+    it('should remove fixed class', function () {
+      $stickyElement.addClass('content-fixed')
+      GOVUK.stickAtTopWhenScrolling.release($stickyElement)
+      expect($stickyElement.hasClass('content-fixed')).toBe(false)
+    })
+
+    it('should remove the shim', function () {
+      $stickyElement = $('<div class="stick-at-top-when-scrolling content-fixed"></div>')
+      GOVUK.stickAtTopWhenScrolling.release($stickyElement)
+      expect($('.shim').length).toBe(0)
+    })
+  })
+
+  describe('for larger screens (>768px)', function () {
+    beforeEach(function () {
+      GOVUK.stickAtTopWhenScrolling.getWindowPositions = function () {
         return {
           scrollTop: 300
-        };
-      };
-      GOVUK.stickAtTopWhenScrolling.getElementOffset = function(){
+        }
+      }
+      GOVUK.stickAtTopWhenScrolling.getElementOffset = function () {
         return {
           top: 300
-        };
-      };
-      GOVUK.stickAtTopWhenScrolling.getWindowDimensions = function(){
+        }
+      }
+      GOVUK.stickAtTopWhenScrolling.getWindowDimensions = function () {
         return {
           height: 768,
           width: 769
-        };
-      };
-      GOVUK.stickAtTopWhenScrolling.$els = $stickyElement;
-      GOVUK.stickAtTopWhenScrolling._hasScrolled = true;
-      GOVUK.stickAtTopWhenScrolling.checkScroll();
-    });
-
-    it('should stick, if the scroll position is past the element position', function(){
-      expect($stickyElement.hasClass('content-fixed')).toBe(true);
-    });
-
-    it('should unstick, if the scroll position is less than the point at which scrolling started', function(){
-      GOVUK.stickAtTopWhenScrolling.getWindowPositions = function() {
+        }
+      }
+      GOVUK.stickAtTopWhenScrolling.$els = $stickyElement
+      GOVUK.stickAtTopWhenScrolling._hasScrolled = true
+      GOVUK.stickAtTopWhenScrolling.checkScroll()
+    })
+
+    it('should stick, if the scroll position is past the element position', function () {
+      expect($stickyElement.hasClass('content-fixed')).toBe(true)
+    })
+
+    it('should unstick, if the scroll position is less than the point at which scrolling started', function () {
+      GOVUK.stickAtTopWhenScrolling.getWindowPositions = function () {
         return {
           scrollTop: 0
-        };
-      };
-      GOVUK.stickAtTopWhenScrolling.$els = $stickyElement;
-      GOVUK.stickAtTopWhenScrolling._hasScrolled = true;
-      GOVUK.stickAtTopWhenScrolling.checkScroll();
-      expect($stickyElement.hasClass('content-fixed')).toBe(false);
-    });
-
-  });
-
-  describe('for smaller screens (<=768px)', function(){
-
-    beforeEach(function() {
-      GOVUK.stickAtTopWhenScrolling.getWindowDimensions = function() {
+        }
+      }
+      GOVUK.stickAtTopWhenScrolling.$els = $stickyElement
+      GOVUK.stickAtTopWhenScrolling._hasScrolled = true
+      GOVUK.stickAtTopWhenScrolling.checkScroll()
+      expect($stickyElement.hasClass('content-fixed')).toBe(false)
+    })
+  })
+
+  describe('for smaller screens (<=768px)', function () {
+    beforeEach(function () {
+      GOVUK.stickAtTopWhenScrolling.getWindowDimensions = function () {
         return {
           height: 768,
           width: 767
-        };
-      };
-      GOVUK.stickAtTopWhenScrolling.getElementOffset = function() {
+        }
+      }
+      GOVUK.stickAtTopWhenScrolling.getElementOffset = function () {
         return {
           top: 300
-        };
-      };
-      GOVUK.stickAtTopWhenScrolling.$els = $stickyElement;
-      GOVUK.stickAtTopWhenScrolling._hasScrolled = true;
-      GOVUK.stickAtTopWhenScrolling.checkScroll();
-    });
-
-    it('should unstick the element', function(){
-      expect($stickyElement.hasClass('content-fixed')).toBe(false);
-    });
-
-  });
-
-});
-
+        }
+      }
+      GOVUK.stickAtTopWhenScrolling.$els = $stickyElement
+      GOVUK.stickAtTopWhenScrolling._hasScrolled = true
+      GOVUK.stickAtTopWhenScrolling.checkScroll()
+    })
+
+    it('should unstick the element', function () {
+      expect($stickyElement.hasClass('content-fixed')).toBe(false)
+    })
+  })
+})

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: govuk_frontend_toolkit
 version: !ruby/object:Gem::Version
-  version: 4.18.3
+  version: 4.18.4
 platform: ruby
 authors:
 - Government Digital Service
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-07 00:00:00.000000000 Z
+date: 2016-10-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails