Thumbnails controlNav update #620

Closed
wants to merge 2 commits into
from

2 participants

@thelamborghinistory

When controlNav has been set to 'thumbnails' until now, for no
reason the images in the nav has been used as selectors, so I've wrapped
them into a link. This is better, because You now can inject some hover
into the thumbnails control nav  without breaking it.

See http://demo.dotwired.de/necon/ and hover the thumbnails to see what I mean by "breaking" it. As soon as you hover the "View" circle, You cannot change the slide any more.

PS.: I've set the li as a new .flex-active selector, you may have other thoughts about this.

Maxim Zubarev added some commits Apr 4, 2013
Maxim Zubarev updated thumbnails coltrolNav
When controlNav has been set to 'thumbnails' until now, for no 
reason the images in the nav has been used as selectors, so I've wrapped
them into a link. This is better, because You now can inject some hover
into the thumbnails control nav  without breaking it.
3bc1cb5
Maxim Zubarev Wrapped not only thumb nav images with an a tag, but also common navi…
…gation in the commit before, so fixed that.
2be1bb9
@mattyza
WooThemes member

I see your use case here, @dotwired. We'll explore this further in due course. At present, we feel this is a bit of a niche use case.

Thanks for your contribution. We look forward to more and welcome them. :)

@mattyza mattyza closed this Jul 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment