Permalink
Browse files

Fix IE8 error when a notification with effect closes

use closure instead of IE's broken setTimeout argument passing
  • Loading branch information...
1 parent becf5db commit 475e147f6738f72e8f8dc0248ca7fefcee72c3c6 @jpjoyal jpjoyal committed Jun 4, 2012
Showing with 2 additions and 2 deletions.
  1. +2 −2 lib/components/notification/notification.js
@@ -217,9 +217,9 @@ Notification.prototype.hide = function(ms){
// hide / remove
this.el.addClass('hide');
if (this._effect) {
- setTimeout(function(self){
+ setTimeout(function(){
self.remove();
- }, 500, this);
+ }, 500);
} else {
self.remove();
}

0 comments on commit 475e147

Please sign in to comment.