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 =