From ca9b1fbe15f9e6a1298e857c28bb561999a66ae4 Mon Sep 17 00:00:00 2001 From: "Alex A. Naanou" Date: Tue, 18 Feb 2020 15:32:22 +0300 Subject: [PATCH] notes... ready to start experimenting... Signed-off-by: Alex A. Naanou --- ui (gen4)/features/ui-widgets.js | 16 +--------------- 1 file changed, 1 insertion(+), 15 deletions(-) diff --git a/ui (gen4)/features/ui-widgets.js b/ui (gen4)/features/ui-widgets.js index 17f26562..a9a900be 100755 --- a/ui (gen4)/features/ui-widgets.js +++ b/ui (gen4)/features/ui-widgets.js @@ -1076,21 +1076,7 @@ browse.items.makeSubContext('field', // // // XXX need to open a list (ro/editable) dialog (currently context is used)... -// ...this can be set via options.list but would be nice to provide -// a reasonable default... -// there are several ways this can be done: -// - generic dialogs in browse.js -// ...the quiestion with this approach is that we need the -// resulting dialogs to get wrapped in the same way as the -// parent... -// ...this can be done via inlining or temporarily redrawing.... -// - a way to define defaults -- global options? -// - access to the .app -- should be configurable... -// ...it would be quite logical to add .app on init to all -// dialogs created via ImageGrid -- this is not trivial as -// we directly calk browse.* most of the time... (refactoring -// this might be a good excuse to migrate to browse2.js) -// - default methods .showEditableList(..) / .showList(..) on make(..) +// ...IMO the way to go here is to move the non-generic editors to browse... // XXX if options.open is defined it will fully override the default open // behavior... browse.items.field.Toggle =