ifvisible is not defined

[henryruhs]

ifvisible is not defined

[jasonzissman]

Good idea! Do you have any recommendations on what to do if ifvisible is not defined? I could include a console.log() message, but I don't know if people typically want log messages (even during exceptional cases like this) to be written to the console. Perhaps I should throw an exception instead?

[henryruhs]

I suggest to write an internal method getVisible or ifVisible for the external library like:

(function (ifvisible) {
..
..
..
getVisible: function() {
    if (typeof ifvisible === 'object') {
      //return the ifvisible object
    }
    else {
      throw 'error, warning or whatever you think';
    }
}
..
..
})(this.ifvisible);

Later on I suggest to use this internal method / wrapper inside a listen method that is made for the event handling... ifvisible.on(...)

listen: function () {
     ..
    TimeMe.getVisible.on('...')
     ..
}

Finally you let the user start the listen method or add it to your init method.

[jasonzissman]

I put together a pull request: #11. Care to take a look at it? Thanks as always for the help.

[henryruhs]

Looks good

[henryruhs]

Please create a new tag like 1.1.0

[jasonzissman]

Done!