Merge pull request #142 from shimomura1004/id/141

replace history only when unload event happens refs #141
codefirst · Oct 2, 2013 · fdd4979 · fdd4979
2 parents 221f55d + cbac7a0
commit fdd4979
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 4 deletions.
diff --git a/app/assets/javascripts/jquery-replacestate.js b/app/assets/javascripts/jquery-replacestate.js
@@ -0,0 +1,13 @@
+/**
+ * update browser history before leaving the page
+ * @author codefirst
+ */
+(function($, document, undefined) {
+    if (window.history && window.history.replaceState) {
+        $(window).on("unload", function(){
+            var latestId = $(".message:last").attr("message-id");
+            window.history.replaceState(latestId ,"", location.pathname+"?latest="+latestId);
+        });
+    }
+})(jQuery, document);
+
diff --git a/app/assets/javascripts/jquery-websocket.js b/app/assets/javascripts/jquery-websocket.js
@@ -20,10 +20,6 @@
 
             target.trigger("websocket::"+name, data);
 
-            if (isMessageEvent(name) && window.history && window.history.replaceState) {
-                window.history.replaceState(data.id ,"", location.pathname+"?latest="+data.id);
-            }
-
             if(window.postMessage) {
                 window.postMessage({ 'type': name,
                                      'current': AsakusaSatellite.current,

diff --git a/app/assets/javascripts/room.js b/app/assets/javascripts/room.js
@@ -7,6 +7,7 @@
 //= require "jquery-multiline"
 //= require "jquery-onthespot"
 //= require "jquery-textcomplete"
+//= require "jquery-replacestate"
 
 $(function() {
     // on the spot