From 2fd8fbdbeb308a47cc4426514531a1ba63ab9882 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Tue, 3 Oct 2023 16:32:29 +0300
Subject: [PATCH] moved remove action to a method + cleanup and fixes...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 experiments/outline-editor/editor.css |  2 +-
 experiments/outline-editor/editor.js  | 26 ++++++++++++++++++--------
 experiments/outline-editor/index.html |  5 +++--
 3 files changed, 22 insertions(+), 11 deletions(-)

diff --git a/experiments/outline-editor/editor.css b/experiments/outline-editor/editor.css
index 978bd90..76d7e2b 100755
--- a/experiments/outline-editor/editor.css
+++ b/experiments/outline-editor/editor.css
@@ -28,7 +28,7 @@
 	--padding: 0.2em;
 
 	display: block;
-	width: 100%
+	width: 100%;
 	/* XXX this is a tiny bit off and using textarea's height here is off too... */
 	min-height: 1em;
 	padding: var(--padding);
diff --git a/experiments/outline-editor/editor.js b/experiments/outline-editor/editor.js
index 69268b6..ce310de 100755
--- a/experiments/outline-editor/editor.js
+++ b/experiments/outline-editor/editor.js
@@ -121,6 +121,7 @@ var Outline = {
 	// 		-> <nodes>
 	//
 	// XXX add support for node ID...
+	// XXX need to be able to get the next elem on same level...
 	get: function(node='focused', offset){
 		var that = this
 
@@ -193,6 +194,12 @@ var Outline = {
 					return elem.getAttribute('tabindex') != null }) }
 
 		// offset...
+		offset = 
+			offset == 'next' ? 
+				1
+			: offset == 'prev' ?
+				-1
+			: offset
 		if(typeof(offset) == 'number'){
 			nodes = this.get('visible')
 			var i = nodes.indexOf(node) + offset
@@ -207,8 +214,8 @@ var Outline = {
 	at: function(index, nodes='visible'){
 		return this.get(nodes).at(index) },
 
-	focus: function(node='focused'){},
-	edit: function(node='focused'){},
+	focus: function(node='focused', offset){},
+	edit: function(node='focused', offset){},
 
 	indent: function(node='focused', indent=true){
 		// .indent(<indent>)
@@ -261,9 +268,15 @@ var Outline = {
 				elem.updateSize() } }
 		return node },
 
-	// XXX should this handle focus???
 	remove: function(node='focused', offset){
-		this.get(...arguments)?.remove()
+		var elem = this.get(...arguments)
+		var next 
+		if(elem.classList.contains('focused')){
+			// XXX need to be able to get the next elem on same level...
+			this.toggleCollapse(elem)
+			next = this.get(elem, 'next') }
+		elem?.remove()
+		next?.focus()
 		return this },
 
 	// block serialization...
@@ -394,10 +407,7 @@ var Outline = {
 		Delete: function(evt){
 			if(evt.target.isContentEditable){
 				return }
-			this.toggleCollapse(true)
-			var next = this.get('next')
-			this.remove()
-			next?.focus() },
+			this.remove() },
 
 		// select...
 		// XXX add:
diff --git a/experiments/outline-editor/index.html b/experiments/outline-editor/index.html
index d74fd86..108e607 100755
--- a/experiments/outline-editor/index.html
+++ b/experiments/outline-editor/index.html
@@ -51,8 +51,9 @@ var setup = function(){
 	<div class="toolbar">
 		<button onclick="editor.deindent()">&lt;</button>
 		<button onclick="editor.indent()">&gt;</button>
-		<button>indent</button>
-		<button>indent</button>
+		<button onclick="editor.Block('before').focus()">O</button>
+		<button onclick="editor.Block('after').focus()">o</button>
+		<button onclick="editor.remove()">&times;</button>
 	</div>
 </div>