From 7d5c38de9639ab8dba98b363dc4037f4fa5422e4 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Mon, 22 Jun 2015 04:02:58 +0300
Subject: [PATCH] some cleanup...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 ui (gen4)/experiments/browse-dialog.js | 22 ++--------------------
 1 file changed, 2 insertions(+), 20 deletions(-)

diff --git a/ui (gen4)/experiments/browse-dialog.js b/ui (gen4)/experiments/browse-dialog.js
index 14738b11..34a6c3e8 100755
--- a/ui (gen4)/experiments/browse-dialog.js	
+++ b/ui (gen4)/experiments/browse-dialog.js	
@@ -768,15 +768,9 @@ var BrowserPrototype = {
 		return this
 	},
 
-	/*
-	focus: function(){
-		this.dom.focus()
-		return this
-	},
-	*/
-
 	// XXX think about the API...
-	// XXX need to check if openable and if not use .push()
+	// XXX need to check if openable i.e. when to use open and when push...
+	// XXX might be a good idea to add a live traversable check...
 	// XXX trigger an "open" event...
 	action: function(){
 		var elem = this.select('!')
@@ -791,13 +785,6 @@ var BrowserPrototype = {
 
 		path.push(elem.text())
 
-		/* XXX need to sort out several issues:
-		 * 		- if not openable and not traversable this infinitely recurs...
-		if(this.isOpenable && !this.isOpenable(path)){
-			return this.push()
-		}
-		*/
-
 		var res = this.open(path)
 
 		this.trigger('open', path)
@@ -805,11 +792,6 @@ var BrowserPrototype = {
 		return res
 	},
 
-	/*
-	isOpenable: function(path){
-		return false
-	},
-	*/
 
 	// extension methods...