Juergen Vigna <[EMAIL PROTECTED]> writes:

| On 13-May-2002 Lars Gullik Bj�nnes wrote:
>> John Levon <[EMAIL PROTECTED]> writes:
>> 
>>| On Thu, May 09, 2002 at 03:23:44PM +0200, Juergen Vigna wrote:
>>>
>>>> This patch fixes #387 as the insettext should not call the edit() call
>>>> for non highly editable insets.
>>>
>>| Yes, works for me.
>>>
>>| btw, the two-dialogs on right click bug is back *again* :(
>> 
>> Caused by this patch?
>
| I don't think so. InsetCollapsable::insetButtonRelease() is returning
| "false" when it should return the variable "ret", which tells us that
| a sub-inset already opened a "layout" dialog.
>
| I commited this fix:
>
| Index: src/insets/insetcollapsable.C
| ===================================================================
| RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/insets/insetcollapsable.C,v
| retrieving revision 1.105
| diff -u -p -r1.105 insetcollapsable.C
| --- src/insets/insetcollapsable.C       9 May 2002 13:43:39 -0000       1.105
| +++ src/insets/insetcollapsable.C       13 May 2002 08:46:53 -0000
| @@ -355,7 +355,7 @@ bool InsetCollapsable::insetButtonReleas
|         if ((button == 3) && !ret) {
|                 return showInsetDialog(bv);
|         }
| -       return false;
| +       return ret;
|  }

And that also helped the bug-reporters?

-- 
        Lgb

Reply via email to