Beck, Andrew Thomas - BECAT001 wrote:
Ok, I see now. Unfortunately your patch is not correct IMHO: If I click on
argument 3 I expect the cursor to appear in argument 3, not in the first
one as your patch does. In perinciple it is ok to call
Of course you tested your original patch to ensure
Beck, Andrew Thomas - BECAT001 wrote:
>
>>Ok, I see now. Unfortunately your patch is not correct IMHO: If I click on
>>argument 3 I expect the cursor to appear in argument 3, not in the first
>>one as your patch does. In perinciple it is ok to call
>
> Of course you tested your original patch
Am Mittwoch, 1. Februar 2006 23:20 schrieb Beck, Andrew Thomas - BECAT001:
Here is a patch to fix bug 2060 - again.
It looks sensible. Are you sure that nothing needs to be done in
cursorPos
and drawSelection?
Nope, I have no idea. I looked at your original patch which mentioned
I just noticed this bug was targeted to 1.4.0. Is
the intention to get this new patch in?
andrew
Am Donnerstag, 2. Februar 2006 22:31 schrieb Beck, Andrew Thomas -
BECAT001:
I just noticed this bug was targeted to 1.4.0. Is
the intention to get this new patch in?
It was not now targetted to 1.4.0, it was targetted to 1.4.0 a long time
ago. I only reopened it today, because you found out
Ok, I see now. Unfortunately your patch is not correct IMHO: If I click on
argument 3 I expect the cursor to appear in argument 3, not in the first
one as your patch does. In perinciple it is ok to call
Of course you tested your original patch to ensure it met this requirement?
I suspect lyx
Am Mittwoch, 1. Februar 2006 23:20 schrieb Beck, Andrew Thomas - BECAT001:
> >
> >> Here is a patch to fix bug 2060 - again.
> >
> >It looks sensible. Are you sure that nothing needs to be done in
cursorPos
> >and drawSelection?
>
> Nope, I have no idea. I looked at your original patch which
I just noticed this bug was targeted to 1.4.0. Is
the intention to get this new patch in?
andrew
Am Donnerstag, 2. Februar 2006 22:31 schrieb Beck, Andrew Thomas -
BECAT001:
> I just noticed this bug was targeted to 1.4.0. Is
> the intention to get this new patch in?
It was not now targetted to 1.4.0, it was targetted to 1.4.0 a long time
ago. I only reopened it today, because you found
>Ok, I see now. Unfortunately your patch is not correct IMHO: If I click on
>argument 3 I expect the cursor to appear in argument 3, not in the first
>one as your patch does. In perinciple it is ok to call
Of course you tested your original patch to ensure it met this requirement?
I suspect
I'm working on a fix which should be available soon.
Excellent!
I just wanted to clarify the intention with this code. I tried
to follow the thread,
http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg82007.html
but am none the wiser.
Is the intention of the current code to somehow work
Title: Re: crash: click on macro instantiation - bug 2060
Here is a patch to fix bug 2060 - again.
I've tested this patch in the following ways (lyx file attached)
create a macro with 0 arguments
create a macro with 1 argument
create a macro with 3 argument
use each of the above macros
Beck, Andrew Thomas - BECAT001 wrote:
Here is a patch to fix bug 2060 - again.
It looks sensible. Are you sure that nothing needs to be done in cursorPos
and drawSelection?
- those with arguments open for editing with first selected
with a macro open, click elsewhere within the inset
What
Here is a patch to fix bug 2060 - again.
It looks sensible. Are you sure that nothing needs to be done in cursorPos
and drawSelection?
Nope, I have no idea. I looked at your original patch which mentioned
fixing a potential problem, but can't see how that would eventuate.
If you can point out
>
>> I'm working on a fix which should be available soon.
>
>Excellent!
I just wanted to clarify the intention with this code. I tried
to follow the thread,
http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg82007.html
but am none the wiser.
Is the intention of the current code to somehow
Title: Re: crash: click on macro instantiation - bug 2060
Here is a patch to fix bug 2060 - again.
I've tested this patch in the following ways (lyx file attached)
create a macro with 0 arguments
create a macro with 1 argument
create a macro with 3 argument
use each of the above macros
Beck, Andrew Thomas - BECAT001 wrote:
> Here is a patch to fix bug 2060 - again.
It looks sensible. Are you sure that nothing needs to be done in cursorPos
and drawSelection?
> - those with arguments open for editing with first selected
> with a macro open, click elsewhere within the inset
>
>> Here is a patch to fix bug 2060 - again.
>
>It looks sensible. Are you sure that nothing needs to be done in cursorPos
>and drawSelection?
Nope, I have no idea. I looked at your original patch which mentioned
fixing a potential problem, but can't see how that would eventuate.
If you can
18 matches
Mail list logo
