[2.0.2][DUP] MessageBox does not hide correctly

[gslender]

Hi,

I noticed (via the samples and code I've tested) that the messagebox will ignore a hide when the window is being dragged (moved). In other words, if you drag/move a messagebox window and the call to hide is sent, then it ignores it and you cannot continue (as the windows has been blocked/masked).

Cheers,
Grant

[gslender]

Perhaps this is by design, and if so, how can you ensure the user has not locked themselves out of your application/site due to the messagebox mask and therefore disabling all other controls... I'm wondering if there is a way you can disable the window move/resize actions to ensure this event blocking does not occur.

[aconran]

I believe what you are describing is by design. For clarity could you please post a code sample so that we can all be on the same page?

[gslender]

There is no real need to provide code as your samples on the website show the problem on all messageboxes easily..

http://extjs.com/deploy/dev/examples...x/msg-box.html

For example, click Show on the Progress Dialog. As it is about to end (nearing 100% on the progress bar) click, and hold the title of the Messagebox - you can even move it about, but it doesn't mater if you do or don't - as long as you keep hold of the title (like a mouse-drag)

Once you have waited past the timeout, when you release the title bar you won't be able to continue with that page... its is locked out and sitting at 100%.

This is a bad design decision as it would leave a user in a state unable to continue with your application. Not a bug in code, but a bug in design.

Does ExtJS have an event-queue ? If so, then this close/hide call should be queued until the widget can handle the event, and then close itself after handling the current move events that are blocking.

Cheers,
Grant

[gslender]

could you please post a code sample ...

Oh, happy to provide a code sample - just tell me what I need to provide that would be different from the existing Samples on the ExtJS website - otherwise use the code in those samples to demonstrate the bug...!!

[[email protected]]

It is fixed in ExtJs1.1(http://extjs.com/deploy/ext-1.1.1/ex...g/msg-box.html) . But this bug is in the extjs2 (http://extjs.com/deploy/dev/examples...x/msg-box.html). so i need the fix for this bug. Coz the user will be dragged the wait screen, while it will not dismissed and the user won't be able to continue with that page.

Cheers,
Riyaz

[halkon_polako]

Hi all

This is a patch/workaround to can hide a message box while is dragging .
The basics on the patch is calling Ext.dd.DragDropMgr.stopDrag() on the method hide at MessageBox:

..............
hide : function(){
    Ext.dd.DragDropMgr.stopDrag(); //PATCH call stopdrag
    if(this.isVisible()){
        dlg.hide();
        handleHide();
    }
    return this;
},
..............

But as a workaround, you can't override this function because hide is accessing private variable(dlg) and functions (handleHide) .
So, you can use this next as a workaround (forever or until this patch or some other solution is included into the official code ).
This workaround consists in creating an interceptor doing the work on the hide function:

Ext.onReady({
.............
    Ext.apply(Ext.MessageBox,{
        hide: Ext.MessageBox.hide.createInterceptor(function(){
            Ext.dd.DragDropMgr.stopDrag();
        })
    });
................
});

Hope this can help you...
Regards

[riyazahameds]

HI halkon_polako,

Thanks for this reply. It is working. Once again thanks.

I have used following code.

Ext.apply(Ext.MessageBox,{
        hide: Ext.MessageBox.hide.createInterceptor(function(){
            Ext.dd.DragDropMgr.stopDrag();
        })
    });

Cheers,
Riyaz.

[brian.moeskau]

This was reported previously and is still on our list to fix. The proper fix needs to be done in Window and is a little trickier than the patches posted above.

[halkon_polako]

This was reported previously and is still on our list to fix. The proper fix needs to be done in Window and is a little trickier than the patches posted above.

Yes... you are right.... it is more a fix on Window than MessageBox, but meanwhile....