From ed5c79c05d36817e1ff8e70e1938d4a6242086f7 Mon Sep 17 00:00:00 2001
From: Ian Gulliver <ian@flamingcow.tv>
Date: Fri, 12 Jul 2019 00:45:03 +0000
Subject: [PATCH] Stop us from undoing right off the page

---
 Architype.js | 31 ++++++++++++++++++++++++++-----
 Editor.js    | 12 ------------
 2 files changed, 26 insertions(+), 17 deletions(-)

diff --git a/Architype.js b/Architype.js
index c9356f8..c95c88b 100644
--- a/Architype.js
+++ b/Architype.js
@@ -34,13 +34,13 @@ class Architype {
     }
 
     addEventListener('hashchange', (e) => { this.onHashChange(e); });
+    addEventListener('popstate', (e) => { this.onPopState(e); });
+    this.first_ = true;
 
-    let fixUrl = false;
     if (location.hash.length > 1) {
       this.unserialize(JSON.parse(atob(location.hash.substring(1))));
     } else {
       this.unserialize(JSON.parse(localStorage.getItem('currentState')));
-      fixUrl = true;
     }
     if (this.editor_.getEntries().length == 0) {
       this.editor_.addHelpAfter();
@@ -53,9 +53,7 @@ class Architype {
 
     this.saveAndRender();
 
-    if (fixUrl) {
-      history.replaceState(null, null, '#' + btoa(this.serializedStr_));
-    }
+    history.replaceState('first', null, '#' + btoa(this.serializedStr_));
   }
 
   observe() {
@@ -79,6 +77,7 @@ class Architype {
   }
 
   serialize() {
+    // TODO: include selected element info
     return {
       version: 1,
       generation: this.generation_,
@@ -123,6 +122,10 @@ class Architype {
     }
   }
 
+  onPopState(e) {
+    this.first_ = (e.state == 'first');
+  }
+
   onChange() {
     ++this.generation_;
     this.saveAndRender();
@@ -130,6 +133,7 @@ class Architype {
 
   snapshot() {
     history.pushState(null, null, '#' + btoa(this.serializedStr_));
+    this.first_ = false;
   }
 
   saveAndRender() {
@@ -140,6 +144,23 @@ class Architype {
   }
 
   onKeyDown(e) {
+    switch (e.key) {
+      case 'u':
+        // Stop us from backing up out of the page
+        if (!this.first_) {
+          history.back();
+        }
+        e.stopPropagation();
+        e.preventDefault();
+        return;
+
+      case 'U':
+        history.forward();
+        e.stopPropagation();
+        e.preventDefault();
+        return;
+    }
+
     let elem = document.activeElement;
     while (elem) {
       if (elem == this.editorElem_) {
diff --git a/Editor.js b/Editor.js
index 3dc952a..a8fda1d 100644
--- a/Editor.js
+++ b/Editor.js
@@ -194,18 +194,6 @@ class Editor extends List {
         }
         return;
 
-      case 'u':
-        history.back();
-        e.stopPropagation();
-        e.preventDefault();
-        return;
-
-      case 'U':
-        history.forward();
-        e.stopPropagation();
-        e.preventDefault();
-        return;
-
       case '?':
         if (this.addHelpAfter()) {
           e.stopPropagation();