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

more notes...

Browse Source 
Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
This commit is contained in:
Alex A. Naanou 2018-01-05 05:20:36 +03:00
parent 5b902cd94c
commit ed605fe5d8
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
ui (gen4)/features/collections.js
View File

@ -2379,7 +2379,7 @@ var UICollectionActions = actions.Actions({
// UI...
//
// XXX would be nice to make this nested (i.e. path list) -- collection grouping... (???)
// XXX should we use options object???
// XXX should we use options object instead of arguments???
// XXX might need to check (in a standard way) that nothing is loaded...
browseCollections: ['Collections/$Collections...',
core.doc`Collection list...
@ -2425,6 +2425,9 @@ var UICollectionActions = actions.Actions({
})
// nothing loaded...
// NOTE: we have to check both .data and .collection as
// we can have an empty collection loaded -- empty
// .data but a set .collection...
if(that.data.length == 0 && that.collection == null){
make.Empty('No collections...')
return
@ -2704,6 +2707,10 @@ var UICollectionActions = actions.Actions({
// XXX also do:
// .saveCollection(..)
// XXX EXPERIMENTAL...
// ...we might not actually need this as this essentially will
// do the same thing as .browseCollections(..)
// ...combining this with .browseCollections(..) might complicate
// things as we still need to reuse the later for other things...
loadCollection: [
collectionGetterWrapper(function(title){ this.loadCollection(title) })],
loadMainCollection: ['Collections/Exit collection view',