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

tweaking .focused and friends...

Browse Source 
Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
This commit is contained in:
Alex A. Naanou 2019-03-18 02:45:56 +03:00
parent 46931af4da
commit d2d2dbe8a3
1 changed files with 19 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -229,6 +229,7 @@ var callItemEventHandlers = function(item, event, ...args){
// XXX revise call signature... // XXX revise call signature...
handler.call(item, evt, item, ...args) }) } handler.call(item, evt, item, ...args) }) }
// XXX use .find(..) instead of .get(..) here....
var makeItemEventMethod = function(event, handler){ var makeItemEventMethod = function(event, handler){
return makeEventMethod(event, function(evt, item, ...args){ return makeEventMethod(event, function(evt, item, ...args){
item = item ? item = item ?
@ -320,11 +321,12 @@ var BaseBrowserPrototype = {
// XXX what should these return??? (item, id, ...) // XXX what should these return??? (item, id, ...)
__focused: undefined, __focused: undefined,
get focused(){ get focused(){
return this.__focused return this.__focused && this.__focused.focused ?
|| (this.__focused = this this.__focused
: (this.__focused = this
// XXX should we simple bailout when we find an item??? // XXX should we simple bailout when we find an item???
.filter(function(e){ .filter(function(e){
return e.focused })[0]) }, return e.focused }).shift()) },
set focused(value){ set focused(value){
// XXX // XXX
}, },
@ -742,11 +744,15 @@ var BaseBrowserPrototype = {
// XXX item API... // XXX item API...
// //
/* XXX this is a more complicated version of .get(..) that should be
// a bit faster on very large lists -- o(1) on direct non-nested
// indexing vs o(n) for the same case in the new .get(..)
// implementation...
// XXX add path support... // XXX add path support...
// XXX add literal item support (???) // XXX add literal item support (???)
// XXX do not get .subtree elements of a .collapsed item... // XXX do not get .subtree elements of a .collapsed item...
// XXX skip .noniterable items... // XXX skip .noniterable items...
get2: function(key, options){ get: function(key, options){
key = key == null ? 0 : key key = key == null ? 0 : key
// index... // index...
@ -834,6 +840,7 @@ var BaseBrowserPrototype = {
} }
return undefined return undefined
}, },
//*/
// Get item... // Get item...
// //
@ -844,7 +851,7 @@ var BaseBrowserPrototype = {
// -> item // -> item
// -> undefined // -> undefined
// //
// options format: the same as for .map(..). // options format: the same as for .map(..) see that for details.
// //
// XXX this is not too fast for indexing very long lists... // XXX this is not too fast for indexing very long lists...
get: function(key, options){ get: function(key, options){
@ -1063,8 +1070,13 @@ var BaseBrowserPrototype = {
prev: function(){}, prev: function(){},
// XXX should there return an array or a .constructor(..) instance?? // XXX should there return an array or a .constructor(..) instance??
forEach: function(){}, forEach: function(func, options){
filter: function(){}, this.map(...arguments)
return this },
filter: function(func, options){
return this.map(function(e, p, b){
return func.call(this, e, p, b) ? [e] : [] })
.flat() },
reduce: function(){}, reduce: function(){},