John Levon wrote:

> On Thu, Jan 08, 2004 at 02:18:35PM +0100, Alfredo Braunstein wrote:
> 
>> > I don't think that this is any better/may well be impossible to
>> > encode. Have a go.
>> 
>> Ugh. You convinced me (I understand near to nothing about qt).
>> Alternatively we can also solve it only for the ones that really
>> need it.
> 
> Um, I thought I had already done that. What has changed since I
> introduced the reject() in the necessary places ?

Alfredo makes the point that all inset dialogs are registered in the 
Dialogs::open_insets_ map. Thus, when a dialog is closed it should 
invoke the modified Dialog::hide:

void Dialog::hide()
{
        if (!view().isVisible())
                return;

        controller().clearParams();
        view().hide();
+       kernel().disconnect(name());
}

Ie, _all_ inset dialogs should invoke CancelButton however they are 
closed.

All this is new in the 1.4.x scheme of things of course.

Incidentally, reject() exists only for QSpellcheckerDialog.C...

-- 
Angus

Reply via email to