From 5b902cd94c3fe41f61987a947859a83f6610249c Mon Sep 17 00:00:00 2001 From: "Alex A. Naanou" Date: Thu, 4 Jan 2018 07:42:09 +0300 Subject: [PATCH] some ui cleanup... Signed-off-by: Alex A. Naanou --- ui (gen4)/features/app.js | 2 +- ui (gen4)/features/collections.js | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/ui (gen4)/features/app.js b/ui (gen4)/features/app.js index f36bad2a..c149ca17 100755 --- a/ui (gen4)/features/app.js +++ b/ui (gen4)/features/app.js @@ -355,7 +355,7 @@ module.PortableAppControl = core.ImageGridFeatures.Feature({ handlers: [ // update window title... // XXX make this generic... - ['focusImage', + ['focusImage clear', function(){ if(this.images){ var img = this.images[this.current] diff --git a/ui (gen4)/features/collections.js b/ui (gen4)/features/collections.js index ea0e2f8f..3428265a 100755 --- a/ui (gen4)/features/collections.js +++ b/ui (gen4)/features/collections.js @@ -2380,6 +2380,7 @@ var UICollectionActions = actions.Actions({ // // XXX would be nice to make this nested (i.e. path list) -- collection grouping... (???) // XXX should we use options object??? + // XXX might need to check (in a standard way) that nothing is loaded... browseCollections: ['Collections/$Collections...', core.doc`Collection list... @@ -2423,6 +2424,12 @@ var UICollectionActions = actions.Actions({ .addClass('highlighted') }) + // nothing loaded... + if(that.data.length == 0 && that.collection == null){ + make.Empty('No collections...') + return + } + var openHandler = function(_, title){ var title = $(this).find('.text').attr('text') || title // create collection if it does not exist...