Hi John,
Apologies, this is probably a bug in the code where it thinks that the variable hasn’t been set. Variables set to 0 can evaluate as being false, giving this kind of behaviour.
I’m not sure that this is a huge problem as I don’t think that many people can tell the difference between 0 delay and a 1ms delay 🙂 Feel free to add it as a bug though, and I’ll get to it when I can:
https://github.com/ewels/cpt-bootstrap-carousel/issues
Phil
Yup, just had a look – the offending code is here:
https://github.com/ewels/cpt-bootstrap-carousel/blob/master/trunk/cptbc-settings.php#L303-L306
This is part of the data cleanup – if you add a text string in that box instead, the absint() function will return 0, so I replace 0 with the default value.
I don’t think I’ll bother to fix this if that’s ok, but feel free to convince me otherwise if you think I should!
Phil
Cool! Oh I don’t care if you fix it. It’s not my plugin! It is a little confusing but whatever. I discovered that I can achieve the effect I want by adding “interval=”0” to the shortcode. Works just as it should so – no worries!
Maybe save it for the next big update? Or if anyone has trouble, hopefully they’ll find this forum post.
Thanks for responding!
No worries – just made the change in github so it will go out with the next release. Thanks for reporting it!
https://github.com/ewels/cpt-bootstrap-carousel/commit/b0603c8dcd9093db2f0d7b2f062a11786d8042bf
Phil
Hi John,
I’ve just pushed the latest version of the plugin to wordpress.org with your suggested changes: https://wordpress.org/plugins/cpt-bootstrap-carousel/
Thanks again!
