nicinabox / superslides

A fullscreen, hardware accelerated slider for jQuery.
http://archive.nicinabox.com/superslides
MIT License
1.51k stars 442 forks source link

Wrong slider position and height on a responsive layout #231

Open liverpoollt opened 10 years ago

liverpoollt commented 10 years ago

Look at this website on a large screen: [inactive url]. All works good. Now resize the window so that the horizontal menu falls into a separed line from the logo. It happens that the current slide is positioned correctly, but all the others have a wrong top property and a wrong height. At the end of the loop, you'll see that the slider starts again with the same issue. If you resize a little the window, the current slide position is fixed (even in the responsive layout) and the fix is remembered on the loop, but again all the other slides are out of place. As soon as you exit from the responsive layout, all works good. I think there's something wrong with the image height evaluation, because it seems to me that the top position is half the height of the container as if the image height were 0px. By the way, I've added a "full-sheet-height" slider by means of some jquery script, but this is not the problem because I tested it without any container too.

Thanks

liverpoollt commented 10 years ago

I have temorarily resolved by adding setTimeout(function() { that.css.images(); }, 10); in the "fade" function definition after the variables declaration. Still waiting for an official patch.

Thanks