Protoshow icon indicating copy to clipboard operation
Protoshow copied to clipboard

option autoPlay:false ignored

Open Nebel54 opened this issue 12 years ago • 3 comments

Thanks for the good work!

I stumbled into a bug today: The option autoPlay:false is ignored, because your initialize Method fires this.play() without considering this option, please wrap an if-statement around it:

    // let's get things going!
    if (this.autoPlay) {
        this.play();
    }

Thanks again, Oliver

Nebel54 avatar Aug 29 '12 14:08 Nebel54

Hi Oliver,

Many thanks for noticing this. It always helps when people take the time to report such things.

Thanks for submitting. I will do my best to get this fixed as soon as possible.

Many thanks,

David

On 29 August 2012 15:17, Nebel54 [email protected] wrote:

Thanks for the good work!

I stumbled into a bug today: The option autoPlay:false is ignored, because your initialize Method fires this.play() without considering this option, please wrap an if-statement around it:

// let's get things going!
if (this.autoPlay) {
    this.play();
}

Thanks again, Oliver

— Reply to this email directly or view it on GitHubhttps://github.com/getdave/Protoshow/issues/12.

getdave avatar Sep 06 '12 19:09 getdave

Hi @Nebel54. Is there any way you could raise a pull request into the develop branch for this? Would be greatly appreciated.

getdave avatar Jun 11 '13 13:06 getdave

Hi @getdave, i just initiated the pull request... hope the request works :)

Nebel54 avatar Jun 12 '13 21:06 Nebel54