Don't remove noscript from body on page change

turbolinks · May 9, 2015 · 86518e1 · 86518e1
1 parent 09e47f4
commit 86518e1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 ## Turbolinks (master)
 
+*   Noscript tags are no longer removed from body on page changes.
+
+    *René Hansen*
+
 *   Deprecated `redirect_via_turbolinks_to` in favor of `redirect_to url, turbolinks: true`.
 
     *Thibaut Courouble*

diff --git a/lib/assets/javascripts/turbolinks.js.coffee b/lib/assets/javascripts/turbolinks.js.coffee
@@ -197,10 +197,6 @@ executeScriptTags = (selector) ->
     parentNode.insertBefore copy, nextSibling
   return
 
-removeNoscriptTags = (node) ->
-  node.innerHTML = node.innerHTML.replace /<noscript[\S\s]*?<\/noscript>/ig, ''
-  node
-
 # Firefox bug: Doesn't autofocus fields that are inserted via JavaScript
 setAutofocusElement = ->
   autofocusElement = (list = document.querySelectorAll 'input[autofocus], textarea[autofocus]')[list.length - 1]
@@ -308,7 +304,7 @@ processResponse = ->
 
 extractTitleAndBody = (doc) ->
   title = doc.querySelector 'title'
-  [ title?.textContent, removeNoscriptTags(doc.querySelector('body')), CSRFToken.get(doc).token ]
+  [ title?.textContent, doc.querySelector('body'), CSRFToken.get(doc).token ]
 
 CSRFToken =
   get: (doc = document) ->