Fix link navigation reloading & causing trouble

fixes #4
Hopefully impacts matestack/matestack-docs#34

Author: PragTob

app/concepts/matestack/ui/core/app/app.js

@@ -1,6 +1,7 @@
 import Vue from 'vue/dist/vue.esm'
 import VRuntimeTemplate from "v-runtime-template"
 import Vuex from 'vuex'
+import isNavigatingToAnotherPage from "app/location"
 
 const componentDef = {
   props: ['appConfig', 'params'],
@@ -11,9 +12,10 @@ const componentDef = {
     asyncTemplate: state => state.pageTemplate,
   }),
   mounted: function(){
-    const self = this;
-    window.onpopstate = function(event) {
-      self.$store.dispatch("navigateTo", {url: document.location.pathname, backwards: true} );
+    window.onpopstate = (event) => {
+      if (isNavigatingToAnotherPage(window.location, event)) {
+        this.$store.dispatch("navigateTo", {url: document.location.pathname, backwards: true} );
+      };
     }
   },
   components: {

app/concepts/matestack/ui/core/app/location.js

@@ -0,0 +1,10 @@
+const isNavigatingToAnotherPage = function(currentLocation, popstateEvent) {
+  const targetLocation = popstateEvent.target.location;
+
+  // omits hash by design
+  return currentLocation.pathname !== targetLocation.pathname ||
+    currentLocation.origin !== targetLocation.origin ||
+    currentLocation.search !== targetLocation.search
+}
+
+export default isNavigatingToAnotherPage

spec/usage/components/link_spec.rb

@@ -262,15 +262,11 @@ def set_app_class
       element = page.find("#my-div")
       before_content = element.text
 
-      puts before_content
-
       # don't you rerender on me!
       expect(ExamplePage).not_to receive(:new)
 
       page.click_link("my-link")
 
-      # binding.pry
-
       # if the page reloaded we'd have different content here but as we don't want reloads
       # we want the sime
       expect(page).to have_css("#my-div", text: before_content)