jsor / jcarousel

Riding carousels with jQuery.
https://sorgalla.com/jcarousel/
MIT License
1.99k stars 736 forks source link

Recommend Removing alert message, switch to console.log() or console.error() instead #621

Closed paceaux closed 10 years ago

paceaux commented 10 years ago

I visited the website ThinkGeek.com and got a jcarousel error message:http://www.thinkgeek.com/product/b5e5/?srp=2 Chrome 31 on Mac Os

The alert() is blocking and it cannot be easily be suppressed in non-debugging environments. I'd like to recommend that this alert be switched to either console.error() or console.log(), instead.

I can't identify the version that ThinkGeek is using - so I can't identify where the alert is in the existing source.

jsor commented 10 years ago

This has been removed 3 years ago (d229e6de3a3b9deb880b9ed72e21a86274d8c4d0).