Ext.ux.Notification

**efattal** · 17 Sep 2008 12:26 AM

Originally Posted by mystix

download with Firefox. do not download with m$IE.

That's right.

I wonder what happens with IE?

**dolittle** · 17 Sep 2008 12:35 AM

The new release is cool.

Maybe it will be better if the close button will be available all the time?

**wm003** · 17 Sep 2008 1:31 AM

When clicking the eject button on the new online demo page firefox 3.0.1 crashes everytime...

**mystix** · 17 Sep 2008 6:50 AM

Originally Posted by efattal

That's right.

I wonder what happens with IE?

it can't handle this forum's gzip compression apparently.

**jmcneese** · 17 Sep 2008 8:13 AM

hey, that stub message looks familiar

**mjlecomte** · 19 Sep 2008 5:20 AM

Anyone notice all of the orphaned elements in the DOM? Looks like the destruction/garbage collection may need some attention?

**evant** · 19 Sep 2008 5:49 AM

Yeah, but it's because the default auto destroy behaviour for the plugin is to hide the window:

Code:

if (this.autoDestroy)
   this.task = new Ext.util.DelayedTask(this.hide, this);
else
   this.closable = true;

**mjlecomte** · 19 Sep 2008 6:01 AM

Too right. I don't understand that part.

if you set autoDestroy = true, wouldn't it be appropriate to destroy it, not hide it?

I'm confused what the intent behind that code is.

**evant** · 19 Sep 2008 6:08 AM

Should probably replace this.hide with this.close, it's not like you're going to reuse the toast.

**efattal** · 19 Sep 2008 6:23 AM

Hi guys,

You're right. I don't remember why I choosed Hide...

I'll correct that soon. Thanks.

Ext.ux.Notification

Thread Tools

Search Thread

Display