From 6e4c8e18d582d07f7886e2a2907883a36cf9c9d4 Mon Sep 17 00:00:00 2001 From: "Alex A. Naanou" Date: Mon, 23 Dec 2019 19:11:45 +0300 Subject: [PATCH] make.field(..) refining + docs... Signed-off-by: Alex A. Naanou --- ui (gen4)/features/ui-widgets.js | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/ui (gen4)/features/ui-widgets.js b/ui (gen4)/features/ui-widgets.js index 807a85eb..28d9c95d 100755 --- a/ui (gen4)/features/ui-widgets.js +++ b/ui (gen4)/features/ui-widgets.js @@ -1025,6 +1025,11 @@ browse.items.makeSubContext = function(name, obj){ // XXX EXPERIMENT... +// +// .field(title[, options]) +// .field(title, value[, options]) +// +// // 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 @@ -1039,10 +1044,16 @@ browse.items.makeSubContext = function(name, obj){ // 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], ...)?? -// XXX parse arguments... 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, {