Merge pull request #6 from HubSpot/ie8

Fix a bug with IE8 compatibility
HubSpot · Dec 3, 2013 · 5e1257a · 5e1257a
2 parents 763dac8 + 5721265
commit 5e1257a
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 4 deletions.
diff --git a/coffee/sortable.coffee b/coffee/sortable.coffee
@@ -6,6 +6,12 @@ trimRegExp = /^\s+|\s+$/g
 touchDevice = 'ontouchstart' of document.documentElement
 clickEvent = if touchDevice then 'touchstart' else 'click'
 
+addEventListener = (el, event, handler) ->
+  if el.addEventListener?
+    el.addEventListener event, handler, false
+  else
+    el.attachEvent "on#{ event }", handler
+
 sortable =
   init: (options={}) ->
     options.selector ?= SELECTOR
@@ -30,7 +36,7 @@ sortable =
   setupClickableTH: (table, th, i) ->
     type = sortable.getColumnType table, i
 
-    th.addEventListener clickEvent, (e) ->
+    addEventListener th, clickEvent, (e) ->
       sorted = @getAttribute('data-sorted') is 'true'
       sortedDirection = @getAttribute 'data-sorted-direction'
 

diff --git a/js/sortable.js b/js/sortable.js
diff --git a/js/sortable.min.js b/js/sortable.min.js