Merge branch 'master' of github.com:flynx/ImageGrid

ui (gen4)/features/ui-widgets.js

@@ -1025,6 +1025,15 @@ browse.items.makeSubContext = function(name, obj){
 
 
 // XXX EXPERIMENT...
+//
+// 	.field(title[, options])
+// 	.field(title, value[, options])
+//
+//
+// NOTE: essentially make.field(..) is almost identical to make(..), the
+// 		later is needed to provide context for field items and a transparent
+// 		fallback for make(..) calls from within their context...
+//
 // XXX Q: should we add an ImageGrid context to make(..)???
 // 		...something like .app for making it generic-ish for example...
 // 		....a different approach to this would be to create a list editor 
@@ -1034,14 +1043,16 @@ browse.items.makeSubContext = function(name, obj){
 // 				...this can be problematic as the wrapper is external to the browser...
 // 			- as a sub-path...
 // 				...this is hard without side-effects...
-// XXX Q: do we actually need .field(..), it does everything make(..) 
-// 		does already???
-// XXX Q: should title/value args be optional???
-// 		...and should we break the make(..) convention of passing an arg 
-// 		array for multiple .text blocks, i.e. make([title, value], ...)??
 browse.items.makeSubContext('field',
 	function(title, value, options){
-		options = options || {}
+		// parse arguments...
+		var args = [...arguments].slice(1)
+		value = (args[0] instanceof Function 
+				|| !(args[0] instanceof Object)) ?
+			args.shift()
+			: undefined
+		options = args.shift() || {}
+		value = value || options.value
 		Object.assign(
 			options, 
 			{