Closed (fixed)
Project:
Node Gallery Hierarchy
Version:
6.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
28 Sep 2011 at 23:01 UTC
Updated:
13 Nov 2012 at 09:11 UTC
Jump to comment: Most recent file
Comments
Comment #1
scroogie commentedHow do you mean there is an empty view? Is it displayed for you? There should be nothing visible in such galleries.
Comment #2
marktheshark commentedPerhaps it shows because the view has a skinr style applied, however, I would expect that nothing would be rendered in the first place (to apply styles to)... I will post screen-shots when I get back from work.
Comment #3
scroogie commentedI assumed that Views would return empty when the View does not return results, but apparently Iwas wrong. I could certainly build in a check for the number of results if the problem persists, at least for subgalleries.
Comment #4
marktheshark commentedParent gallery has empty view node_gallery_gallery_image_views with display 'View all image thumbnails' from the Node Galleries module.
Child gallery has empty view node_gallery_hierarchy_views with display 'Galleries Cover Grid' from the Node Gallery Hierarchy module.
Attached screenshots (the empty views are the long rounded rectangles with no content).
Comment #5
scroogie commentedI just commited a check for the subgallery view, please test it if you can. I'm not sure how to deal with the empty image view. I could override the behaviour of Node Gallery here in Node Gallery Hierarchy, but I'm not sure if that would be the best solution.
Comment #6
marktheshark commentedThe fix worked, thanks. Now galleries do not display an empty view if they do not have subgalleries. Any recommendation on what to change in Node Gallery to do the same? Would you say that an issue should be opened for that module?
Comment #7
scroogie commentedYes, probably.
Comment #8
marktheshark commentedI opened one, though perhaps it will not be accepted as a valid request, since the real need to hide the empty image view arises when there is a non-empty sub-galleries view on the same page...
Perhaps this enhancement would be more at home in the current module.
Comment #9
scroogie commentedWell, it's possible to do here... I'm just not sure.
Comment #10
scroogie commentedAlready does hide empty views.