jaukia / zoomooz

An easy-to-use jQuery plugin for making zooming web pages.
http://janne.aukia.com/zoomooz/
1.79k stars 275 forks source link

'Prev' and 'Next' button functionality broken #66

Open rhodriwls opened 11 years ago

rhodriwls commented 11 years ago

(original title - 'prev' and 'next' buttons behave like 'first' and 'last')

I've been trying to create a slideshow-type effect with previous and next buttons, but it seems to be behaving wrongly - the 'prev' button works more like a 'first' button, with the 'next' button working like a 'last' button, and not stepping through the individual elements at all. Elements are defined as on the homepage , eg:

<div class="zoomButton" data-type="prev" id="prev" data-root=".zoomViewport"><</div>

Using latest build as of 3/5/13, with jQuery 1.9.

Suggestions?

jaukia commented 11 years ago

I think there is something buggy with the prev and next. Unfortunately I don't have time to look at the issue right now but I will get back to it at some point.

-janne

On 3 May 2013 18:51, rhodriwls notifications@github.com wrote:

I've been trying to create a slideshow-type effect with previous and next buttons, but it seems to be behaving wrongly - the 'prev' button works more like a 'first' button, with the 'next' button working like a 'last' button, and not stepping through the individual elements at all. Elements are defined as on the homepage , eg:

Using latest build as of 3/5/13, with jQuery 1.9.

Suggestions?

— Reply to this email directly or view it on GitHubhttps://github.com/jaukia/zoomooz/issues/66 .

rhodriwls commented 11 years ago

Looks like something to do with the CSS class, ".selectedZoomTarget" not getting applied - when an item is clicked on, and I do "$(".zoomContainer").find(".selectedZoomTarget");" in the web console, I get an empty array.

jaukia commented 11 years ago

I did a small fix, let me know if that helps.

jaukia commented 11 years ago

Your example code has a mistake, "<

" has two "<":s. have you tried removing one of them?