From f4e161cc9d470a64d3691836ea208d4d8fdf6696 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Sat, 11 Jul 2015 22:16:35 +0300
Subject: [PATCH] removed all the code that was compensating for the
 click/focus propagation bug...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 ui (gen4)/experiments/browse-dialog.html |  4 +--
 ui (gen4)/experiments/browse-dialog.js   | 34 +++---------------------
 2 files changed, 5 insertions(+), 33 deletions(-)

diff --git a/ui (gen4)/experiments/browse-dialog.html b/ui (gen4)/experiments/browse-dialog.html
index 2696e160..c8c366c9 100755
--- a/ui (gen4)/experiments/browse-dialog.html	
+++ b/ui (gen4)/experiments/browse-dialog.html	
@@ -300,8 +300,8 @@ Files: <button id="theme" onclick="toggleFiles()">toggle</button>
 <br>
 Basic key bindings:
 <ul>
-	<li><code>ctrl-a<code> - edit full path
-	<li><code>ctrl-left<code> / <code>ctrl-Backspace<code> - go to root
+	<li><code>ctrl-a<code> - edit full path </li>
+	<li><code>ctrl-left<code> / <code>ctrl-Backspace<code> - go to root </li>
 </ul>
 
 
diff --git a/ui (gen4)/experiments/browse-dialog.js b/ui (gen4)/experiments/browse-dialog.js
index 75c17ce3..569a3cab 100755
--- a/ui (gen4)/experiments/browse-dialog.js	
+++ b/ui (gen4)/experiments/browse-dialog.js	
@@ -347,7 +347,6 @@ var BrowserPrototype = {
 	//
 	// On more info on setting the path see .update(..)
 	//
-	//
 	// NOTE: .path = <some-path> is equivalent to .update(<some-path>)
 	// 		both exist at the same time to enable chaining...
 	// NOTE: to avoid duplicating and syncing data, the actual path is 
@@ -465,32 +464,14 @@ var BrowserPrototype = {
 		var txt
 		p.append($('<div>')
 			.addClass('dir cur')
-			// XXX BUG: for some reason this element keeps the selection
-			// 		but looses focus in IE...
 			.click(function(){
 				event.stopPropagation()
 
 				that.toggleFilter('on')
 				//that.update(path.concat($(this).text())) 
-
-				// XXX HACK: prevents the field from blurring when clicked...
-				that._hold_blur = true
-				setTimeout(function(){ delete that._hold_blur }, 20)
-
-				// XXX HACK: this will work around double triggering of the focus
-				// 		event after a click happens...
-				that._focus_hold = true
-
-				setTimeout(function(){ console.log('>>>', $(':focus')[0]) }, 1000)
 			})
-			// XXX for some reason this gets triggered when clicking ano 
-			// 		is not triggered when entering via '/'
 			.on('blur', function(){
-				// XXX HACK: prevents the field from bluring when clicked...
-				// 			...need to find a better way...
-				if(!that._hold_blur){
-					that.toggleFilter('off')
-				}
+				that.toggleFilter('off')
 			})
 			/* XXX does the right thing (replaces the later .focus(..) 
 			 * 		and .keyup(..)) but does not work in IE...
@@ -499,16 +480,7 @@ var BrowserPrototype = {
 			})
 			*/
 			// only update if text changed...
-			// XXX for some reason this gets triggered when clicking ano 
-			// 		is not triggered when entering via '/'
 			.focus(function(){
-				// XXX HACK: this will work around double triggering of the focus
-				// 		event after a click happens...
-				if(that._focus_hold){
-					delete that._focus_hold
-					return
-				}
-
 				txt = $(this).text()
 			})
 			.keyup(function(){
@@ -1111,6 +1083,8 @@ var BrowserPrototype = {
 
 	// Push an element to path / go down one level...
 	//
+	// NOTE: if the element is not traversable it will be opened.
+	//
 	// XXX might be a good idea to add a live traversable check...
 	// XXX revise event...
 	push: function(pattern){
@@ -1178,8 +1152,6 @@ var BrowserPrototype = {
 	//
 	// This opens (.open(..)) the selected item and if none are selected
 	// selects the default (.select()) and exits.
-	// 
-	// XXX need to check if openable i.e. when to use open and when push...
 	action: function(){
 		var elem = this.select('!')