diff --git a/ui (gen4)/data.js b/ui (gen4)/data.js
index 4636a8b9..616eb915 100755
--- a/ui (gen4)/data.js	
+++ b/ui (gen4)/data.js	
@@ -355,6 +355,7 @@ var DataPrototype = {
 	// Clear elements from data...
 	//
 	// 	Clear all data:
+	// 	.clear()
 	// 	.clear('*')
 	// 	.clear('all')
 	// 		-> data
@@ -390,14 +391,10 @@ var DataPrototype = {
 	// 		other than null?
 	// XXX should this return this or the removed gids???
 	clear: function(gids, deep, clear_empty){
-		// defaults...
+		gids = gids || 'all'
 		deep = deep == null ? true : false
 		clear_empty = clear_empty == null ? true : false 
 
-		if(gids == null){
-			return this
-		}
-
 		// clear all data...
 		if(gids == '*' || gids == 'all'){
 			this._reset()
diff --git a/ui (gen4)/features/filesystem.js b/ui (gen4)/features/filesystem.js
index 4a89384d..3f9d7a19 100755
--- a/ui (gen4)/features/filesystem.js	
+++ b/ui (gen4)/features/filesystem.js	
@@ -758,6 +758,7 @@ var FileSystemWriterActions = actions.Actions({
 	// 		...both denodeify(..)'ed
 	// XXX export current state as a full loadable index
 	// XXX might be interesting to unify this and .exportView(..)
+	// XXX should this return a promise??? ...a clean promise???
 	// XXX local collections???
 	exportCollection: ['File/Export as collection',
 		function(path, logger){
diff --git a/ui (gen4)/features/ui.js b/ui (gen4)/features/ui.js
index 844777d9..ef62e217 100755
--- a/ui (gen4)/features/ui.js	
+++ b/ui (gen4)/features/ui.js	
@@ -263,7 +263,7 @@ module.ViewerActions = actions.Actions({
 			this.ribbons.updateImage(gids)
 		}],
 	clear: [
-		function(){ this.ribbon && this.ribbons.clear() }],
+		function(){ this.ribbons && this.ribbons.clear() }],
 	clone: [function(full){
 		return function(res){
 			if(this.ribbons){