From ccf93118ba74daa77e122bf5a0ce3815be498cf5 Mon Sep 17 00:00:00 2001
From: Jeremy Ruston <jeremy@osmosoft.com>
Date: Wed, 24 Oct 2012 23:03:06 +0100
Subject: [PATCH] Fixes for new tiddler positioning

---
 core/modules/macros/list/list.js         |  2 +-
 core/modules/macros/navigationcatcher.js | 23 +++++++++++++----------
 2 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/core/modules/macros/list/list.js b/core/modules/macros/list/list.js
index f168a1a99..58897ee56 100644
--- a/core/modules/macros/list/list.js
+++ b/core/modules/macros/list/list.js
@@ -117,7 +117,7 @@ exports.createListElement = function(title) {
 		}};
 	node.execute(this.parents,this.tiddlerTitle);
 	var listElement = $tw.Tree.Element(this.isBlock ? "div" : "span",{"class": ["tw-list-element"]},[node],{
-			events: ["tw-navigate","tw-EditTiddler","tw-SaveTiddler","tw-CloseTiddler"],
+			events: ["tw-navigate","tw-EditTiddler","tw-SaveTiddler","tw-CloseTiddler","tw-NewTiddler"],
 			eventHandler: eventHandler
 		});
 	// Save our data inside the list element node
diff --git a/core/modules/macros/navigationcatcher.js b/core/modules/macros/navigationcatcher.js
index 44fe3ad29..09d447207 100644
--- a/core/modules/macros/navigationcatcher.js
+++ b/core/modules/macros/navigationcatcher.js
@@ -32,6 +32,15 @@ exports.saveList = function(title,list) {
 	},storyTiddler,{text: list.join("\n")}));
 };
 
+exports.findTitleInStory = function(title,defaultIndex) {
+	for(var t=0; t<this.story.length; t++) {
+		if(this.story[t] === title) {
+			return t;
+		}
+	}	
+	return defaultIndex;
+};
+
 exports.handleEvent = function(event) {
 	if(this.eventMap[event.type]) {
 		this.eventMap[event.type].call(this,event);
@@ -54,14 +63,7 @@ exports.eventMap["tw-navigate"] = function(event) {
 	// If not we need to add it
 	if(tiddler === undefined) {
 		// First we try to find the position of the story element we navigated from
-		slot = 0;
-		if(event.navigateFromTitle !== undefined) {
-			for(t=0; t<this.story.length; t++) {
-				if(this.story[t] === event.navigateFromTitle) {
-					slot = t + 1;
-				}
-			}	
-		}
+		slot = this.findTitleInStory(event.navigateFromTitle,-1) + 1;
 		// Add the tiddler
 		this.story.splice(slot,0,event.navigateTo);
 		// Save the story
@@ -179,8 +181,9 @@ exports.eventMap["tw-NewTiddler"] = function(event) {
 			"draft.of": title
 		});
 	this.wiki.addTiddler(draftTiddler);
-	// Update the story to put the new draft at the top
-	this.story.splice(0,0,draftTitle);
+	// Update the story to insert the new draft at the top
+	var slot = this.findTitleInStory(event.navigateFromTitle,-1) + 1;
+	this.story.splice(slot,0,draftTitle);
 	// Save the updated story
 	this.saveList(this.storyTitle,this.story);
 	// Add a new record to the top of the history stack