Thanks Cais.
I checked your link and it is indeed working.
No inline style and no extra empty divs being output.
I do still see the inline style happening at this page http://www.nextgen-gallery.com/basic-imagebrowser-gallery/
so I’m wondering what version of nextgen you have (if its a beta) and what method you are using to insert the gallery?
Also happens here: http://nggpro.bycais.com/?page_id=176
It seems to be an older version of pagination, as the non-working ones have the spinning beach ball loader.
@yanwhite – It is the latest beta which is available here: http://nextgen-gallery.com/nextgen-gallery-latest-beta/ … and that particular display was added with installation defaults and the IGW.
I even double-checked my ImageBrowser displays on my test site and they are working correctly as well. I’m also not seeing the problem on our home site either. The site, at the moment, is running NextGEN Gallery 2.0.66.33 and a Genesis based theme.
Perhaps there are more specifics to your view that may need to be taken into consideration … what browser(s) and OS are you specifically seeing this in?
As well as, have you tried using anonymous browser windows and/or clearing all of your caches/cookies (the anonymous browser window should cover this but just in case of something truly persistent)?
– Cais.
It’s only noticeable on firefox, not chrome (OSX)
However, I opened an incognito window in chrome and inspected the element of your main nav after clicking on next at this page: http://nggpro.bycais.com/?page_id=176
<a href="http://nggpro.bycais.com/?page_id=13" style="cursor: auto;">Basic Displays</a>
I imagine the fix that Sergio suggested should work
https://ww.wp.xz.cn/support/topic/cursor-after-changing-picture-via-ajax
However I believe there is a cache issue preventing me from seeing the change.
Thanks, I will try the beta and see if that fixes the other issue as well.
I’m also using Genesis and my own child theme.
Ok, so the cursor bug persists after updating to the beta version.
The problem code is still clearly there:
$('body, a').css('cursor', 'wait');
Actually this acts on the whole body and the next:
$('body, a').css('cursor', 'auto');
For some reason Sergio’s fix is not working for me.
I have tracked down the other issue.
In my CSS I have:
.ngg-galleryoverview .ngg-gallery-thumbnail-box {
float: left;
margin: 0;
padding: 5px;
width: 20%;
}
This is to get some control of a proper responsive layout, where I can change the column count (width %) at different screen sizes.
The padding and width in my CSS trigger layout on those elements that are otherwise hidden because they have none. There are no different classes to hook into to add a display: none.
IMO this should be handled with JQuery.
Anyway, at least I can hardcode a column count through the GUI and have a non-broken desktop view for my site now.
Fail. That doesnt work either, as the margin from empty tags breaks the layout when navigating to a new page.
One question.
On your demo page how did you get only the thumbnail tags on the current page to show? Is that something different in the premium version perhaps?
I have tried switching to 2015 theme and the empty divs still get output.
It’s impossible to style the gallery this way!
@yanwhite –
This is what I am seeing in Firefox: http://take.ms/tjurk
This is the equivalent with Firebug: http://take.ms/XhD2L
I’m not saying you are not seeing what you are describing, I simply cannot see it myself?!
I’m not sure I understand what you are referring to with “on your demo page how did you get only the thumbnail tags on the current page to show” … which specific “demo” page are you referring to?
Thanks!
– Cais.
Thanks for your help.
You didnt specify that you had taken those screenshots after clicking on pagination. Not saying you didnt, but for the sake of clarity its better to be specific. Sergio had the same issue, so it should be acknowledged as a bug.
I saved myself the hassle of trying to style nextgen and picked up Justified Image Grid, which is exactly what I needed, fully responsive and has a nicer licensing model (one time purchase). I didnt have to do any hacking to make it do what I wanted and looks great.
I can now use nextgen as a backend librarian, which is what it does best. The frontend and styling of nextgen still needs a lot of work IMO
@yanwhite – Thanks for the update; and, we definitely agree with …
The frontend and styling of nextgen still needs a lot of work IMO
It’s definitely on our to-do list to improve the overall look and feel of displays.
– Cais.
