experimenting with collections + some fixes and tweaks...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>

ui (gen4)/features/collections.js

@@ -7,6 +7,8 @@
 (function(require){ var module={} // make module AMD/node compatible...
 /*********************************************************************/
 
+var data = require('imagegrid/data')
+
 var actions = require('lib/actions')
 var features = require('lib/features')
 
@@ -58,9 +60,18 @@ var CollectionActions = actions.Actions({
 			// XXX need to clear this when exiting crop...
 			this.location.collection = collection
 		}],
-	saveCollection: ['- Collections/Save collection',
-		core.doc`Save current state to collection`,
-		function(collection){
+	saveCollection: ['- Collections/',
+		core.doc`Save current state to collection
+
+			Save Current state as collection
+			.saveCollection(collection)
+				-> this
+
+			Save new empty collection
+			.saveCollection(collection, true)
+				-> this
+		`,
+		function(collection, empty){
 			collection = collection || this.collection
 
 			if(collection == null){
@@ -73,11 +84,15 @@ var CollectionActions = actions.Actions({
 				title: collection,
 
 				// XXX we need to trim .order to only the current images???
-				data: this.data
-					.clone()
-					.removeUnloadedGids(),
+				data: empty ? 
+					(new this.data.constructor())
+					: this.data
+						.clone()
+						.removeUnloadedGids(),
 			}
 		}],
+	newCollection: ['- Collections/',
+		function(collection){ return this.saveCollection(collection, true) }],
 
 	inCollections: ['- Image/',
 		core.doc`Get list of collections containing item`,
@@ -90,11 +105,15 @@ var CollectionActions = actions.Actions({
 		}],
 
 	collect: ['- Collections/',
-		core.doc`Add items to collection`,
+		core.doc`Add items to collection
+
+		NOTE: this will not account for item topology.`,
 		function(gids, collection){
 			var that = this
 
-			gids = gids instanceof Array ? gids : [gids]
+			gids = gids == 'loaded' ? this.data.getImages('loaded')
+				: gids instanceof Array ? gids 
+				: [gids]
 			gids = gids
 				.map(function(gid){ 
 					return gid in that.data.ribbons ? 
@@ -108,6 +127,31 @@ var CollectionActions = actions.Actions({
 			// XXX add to collection...
 			// XXX
 		}],
+	joinCollect: ['- Collections/Merge to collection',
+		core.doc`Merge current state to collection
+
+			Join current state into collection
+			.joinCollect(collection)
+				-> this
+
+			Join current state with specific alignment into collection
+			.joinCollect(align, collection)
+				-> this
+
+		This is like .collect(..) but will preserve topology.
+		
+		NOTE: for align docs see Data.join(..)
+		`,
+		function(align, collection){
+			collection = collection == null ? align : collection
+			if(collection == null){
+				return
+			}
+			this.collections && this.collections[collection] ?
+				this.collections[collection].data.join(align, this.data.clone())
+				: this.saveCollection(collection)
+		}],
+
 	uncollect: ['- Collections/',
 		function(gids, collection){
 			// XXX
@@ -147,28 +191,27 @@ module.Collection = core.ImageGridFeatures.Feature({
 // XXX show collections in image metadata...
 var UICollectionActions = actions.Actions({
 	// XXX highlight current collections....
-	browseCollections: ['Collections|Crop/Collections...',
-		widgets.makeUIDialog(function(gid){
+	browseCollections: ['Collections|Crop/$Collec$tions...',
+		widgets.makeUIDialog(function(action){
 			var that = this
-			gid = gid != null ? this.data.getImage(gid) : gid
-
 			var to_remove = []
 
 			return browse.makeLister(null, 
 				function(path, make){
 					var dialog = this
 
-					//var collections = Object.keys(that.collections || {})
-					var collections = that.inCollections(gid || null)
+					var collections = Object.keys(that.collections || {})
 
 					make.EditableList(collections, 
 						{
 							unique: true,
 							to_remove: to_remove,
 							itemopen: function(title){
-								that.loadCollection(title)
-								gid
-									&& that.focusImage(gid)
+								var gid = that.current
+								action ?
+									action.call(that, title)
+									: that.loadCollection(title)
+								that.focusImage(gid)
 								dialog.close()
 							},
 							normalize: function(title){ 
@@ -177,7 +220,9 @@ var UICollectionActions = actions.Actions({
 								return title.length > 0 },
 
 							itemadded: function(title){
-								that.saveCollection(title) },
+								action ?
+									that.newCollection(title)
+									: that.saveCollection(title) },
 						})
 				})
 				.close(function(){
@@ -231,6 +276,25 @@ var UICollectionActions = actions.Actions({
 				})
 		})],
 
+
+	// XXX should we warn the user when overwriting???
+	saveAsCollection: ['Collections/$Save as collection...',
+		widgets.uiDialog(function(){
+			return this.browseCollections(function(title){
+				this.saveCollection(title) }) })],
+	addToCollection: ['Collections|Image/Add $image to collection...',
+		widgets.uiDialog(function(gids){
+			return this.browseCollections(function(title){
+				this.collect(gids || this.current, title) }) })],
+	addLoadedToCollection: ['Collections/$Add loaded images to collection...',
+		widgets.uiDialog(function(){ return this.addToCollection('loaded') })],
+	// XXX for some reason joining two one ribbon states produces two ribbons...
+	joinToCollection: ['Collections/$Merge view to collection...',
+		widgets.uiDialog(function(){
+			return this.browseCollections(function(title){
+				this.joinCollect(title) }) })],
+
+
 	// XXX this is not used by metadata yet...
 	metadataSection: ['- Image/',
 		function(gid, make){

ui (gen4)/features/keyboard.js

@@ -358,7 +358,8 @@ module.GLOBAL_KEYBOARD = {
 
 
 		// collections...
-		alt_C: 'browseCollections',
+		//alt_C: 'browseCollections',
+		alt_C: 'browseActions: "/Collections/" -- Collections menu...',
 
 
 		// metadata...

ui (gen4)/imagegrid/data.js

@@ -209,8 +209,10 @@ var DataPrototype = {
 	set current(value){
 		this.focusImage(value) },
 
+	// XXX should this default to top or bottom ribbon???
 	get base(){
-		return this.__base || this.ribbon_order.slice(-1)[0] },
+		//return this.__base || this.ribbon_order.slice(-1)[0] },
+		return this.__base || this.ribbon_order[0] },
 	set base(value){
 		this.__base = value },
 
@@ -2172,7 +2174,10 @@ var DataPrototype = {
 	// XXX add a 'gid' align mode...
 	join: function(){
 		var args = Array.apply(null, arguments)
-		var align = typeof(args[0]) == typeof('str') ? args.splice(0, 1)[0] : 'base'
+		var align = typeof(args[0]) == typeof('str') || args[0] == null ? 
+			args.shift() 
+			: 'base'
+		align = align || 'base'
 		args = args[0].constructor === Array ? args[0] : args
 
 		var base = this
@@ -2248,8 +2253,10 @@ var DataPrototype = {
 			})
 		})
 
-		// XXX this is slow-ish...
-		base.removeDuplicateGIDs()
+		base
+			// XXX this is slow-ish...
+			.removeDuplicateGIDs()
+			.removeEmptyRibbons()
 
 		return base
 	},
@@ -2545,10 +2552,42 @@ var DataPrototype = {
 
 	// Remove duplicate gids...
 	//
+	// If a gid is in more than one ribbon, this will keep the top 
+	// occurrence only...
+	//
+	// NOTE: this may result in empty ribbons...
 	// NOTE: this is slow-ish...
 	removeDuplicateGIDs: function(){
+		var that = this
 		this.removeDuplicates(this.order)
 		this.updateImagePositions()
+		// if a gid is in more than one ribbon keep only the top occurence...
+		this.order.forEach(function(gid, i){
+			var found = false
+			that.ribbon_order.forEach(function(r){
+				r = that.ribbons[r]
+
+				if(found){
+					delete r[i]
+
+				} else if(r[i] != null){
+					found = true
+				}
+			})
+		})
+		return this
+	},
+	// Remove empty ribbons...
+	removeEmptyRibbons: function(){
+		var that = this
+		this.ribbon_order = this.ribbon_order
+			.filter(function(r){ 
+				if(that.ribbons[r].len == 0){
+					delete that.ribbons[r]
+					return false
+				} 
+				return true
+			})
 		return this
 	},
 
@@ -2558,6 +2597,7 @@ var DataPrototype = {
 	// 	- images from .data that are not in any ribbon
 	removeUnloadedGids: function(){
 		this.order = this.getImages('loaded')
+		this.updateImagePositions()
 		return this
 	},