From 3a1e5b5784d306132b4dc20355448696c848a82f Mon Sep 17 00:00:00 2001 From: "Alex A. Naanou" Date: Wed, 24 Apr 2019 15:07:21 +0300 Subject: [PATCH] .render2(..) is done... Signed-off-by: Alex A. Naanou --- ui (gen4)/lib/widget/browse2.js | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/ui (gen4)/lib/widget/browse2.js b/ui (gen4)/lib/widget/browse2.js index 9394c230..b6793e42 100755 --- a/ui (gen4)/lib/widget/browse2.js +++ b/ui (gen4)/lib/widget/browse2.js @@ -1474,7 +1474,7 @@ var BaseBrowserPrototype = { : items }, - // XXX still need to wrap inline lists of items (groups and friends)... + // XXX should this be able to render a specific sub-path??? render2: function(options, renderer){ var that = this // XXX Q: should options and context be distinguished only via @@ -1499,32 +1499,27 @@ var BaseBrowserPrototype = { options = context.options renderer = renderer || this - var getValue = function(item){ - return item.value || item } - var items = this .walk( function(i, path, item, nested, sublist){ - var indent = path.map(e => ' ').join('') - // XXX call renderers... return ( // inline... - // XXX need to wrap groups/inline stuff... (item == null && sublist) ? // NOTE: here we are forcing rendering of the // inline browser/list, i.e. ignoring // options.skipNested for inline stuff... - nested(true) + // NOTE: we here do not distinguish between + // inlined lists and browsers... (XXX ???) + [ renderer.renderGroup(nested(true), context) ] // nested... : sublist ? - [renderer.renderNested( + [ renderer.renderNested( renderer.renderNestedHeader(item, i, context), nested(), item, - context)] + context) ] // normal item... - : [renderer.renderItem(item, i, context)] - ) }, + : [ renderer.renderItem(item, i, context) ] ) }, function(func, i, path, sublist, options){ return sublist.render2(context) }, options)