From c6ad689d89bf77c970cf7cd90d32f3b5fc5ccc99 Mon Sep 17 00:00:00 2001 From: Jeffery To Date: Tue, 15 Apr 2014 17:40:29 +0800 Subject: [PATCH] Removed addEvent support As noted in #456, IE8 is no longer supported. Also, undefined is falsey in JavaScript, so the typeof check was unnecessary. --- js/skip-link-focus-fix.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/js/skip-link-focus-fix.js b/js/skip-link-focus-fix.js index e067cd5..044d055 100644 --- a/js/skip-link-focus-fix.js +++ b/js/skip-link-focus-fix.js @@ -3,9 +3,8 @@ is_opera = navigator.userAgent.toLowerCase().indexOf( 'opera' ) > -1, is_ie = navigator.userAgent.toLowerCase().indexOf( 'msie' ) > -1; - if ( ( is_webkit || is_opera || is_ie ) && 'undefined' !== typeof( document.getElementById ) ) { - var eventMethod = ( window.addEventListener ) ? 'addEventListener' : 'attachEvent'; - window[ eventMethod ]( 'hashchange', function() { + if ( ( is_webkit || is_opera || is_ie ) && document.getElementById && window.addEventListener ) { + window.addEventListener( 'hashchange', function() { var element = document.getElementById( location.hash.substring( 1 ) ); if ( element ) {