flynx/ImageGrid
1
1
Fork 0
mirror of https://github.com/flynx/ImageGrid.git synced 2025-10-28 18:00:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

notes...

Browse Source 
Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
This commit is contained in:
Alex A. Naanou 2019-08-05 17:30:39 +03:00
parent 8be7058c76
commit b9f30d7a40
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
ui (gen4)/lib/widget/browse2.js
View File

@ -5205,6 +5205,11 @@ var HTMLBrowserPrototype = {
that.dom
&& that.dom.focus() }) },
__open__: function(evt, elem){ this.focus(elem) },
// XXX when expanding an element at the bottom of the screen (i.e.
// when the expanded tree is not visible) need to nudge the
// element up to reveal the expanded subtree...
// ...would also be logical to "show" the expanded tree but
// keeping the focused elem in view...
/* XXX there is a problem with .update() propagation up the nested
// dialogs -- we lose context...
// ...see .renderContext(..) / .renderFinalize(..) for details...
@ -5214,13 +5219,13 @@ var HTMLBrowserPrototype = {
__expand__: function(){ this.update() },
__collapse__: function(){ this.update() },
/*/
// XXX this is a hack -- we should fix the actual update propagation and revert to the above...
// XXX make this a "local" update -- i.e. only update the subtree...
__expand__: function(){ this.root.update() },
__collapse__: function(){ this.root.update() },
//*/
__select__: updateElemClass('add', 'selected'),
__deselect__: updateElemClass('remove', 'selected'),
// XXX would be more logical to update the actual sepcific elements...
// XXX would be more logical to update only the sepcific elements...
__disable__: updateElemClass('add', 'disabled', function(){ this.update() }),
__enable__: updateElemClass('remove', 'disabled', function(){ this.update() }),
__hide__: updateElemClass('add', 'hidden'),