target option fix: replacing JS 'style' with jQuery 'css'

jackmoore / zoom

jQuery plugin for zooming images on mouseover.

http://www.jacklmoore.com/zoom/

MIT License

1.54k stars 460 forks source link

target option fix: replacing JS 'style' with jQuery 'css' #73

Closed easingthemes closed 9 years ago

easingthemes commented 9 years ago

Target option was broken.

target: 'selector'

On some places it was used as jQuery object $(selector), and on some places as DOM element selector.style - bug, selector is string, it can't be used like this.

Fixed by changing all to jQuery object.

Target is working fine now.

Thanks, Dragan

easingthemes commented 9 years ago

Issue: #68 - 'target' option.

jacquesletesson commented 9 years ago

Make sense!

jackmoore commented 9 years ago

Thanks

easingthemes commented 9 years ago

np, Thank you for the plugin