Re: wrong latex output with decimally aligned columns

2011-05-09 Thread Edwin Leuven 
Richard Heck rgh...@comcast.net wrote:
 Anyway, I'll have a look at the patch later. For now, it looks good, and
 of course we'll also need it for branch.

i applied this

http://www.lyx.org/trac/changeset/38655

ok for branch?

edwin

Re: wrong latex output with decimally aligned columns

2011-05-09 Thread Richard Heck 
On 05/09/2011 08:34 AM, Edwin Leuven wrote:
 Richard Heck rgh...@comcast.net wrote:
 Anyway, I'll have a look at the patch later. For now, it looks good, and
 of course we'll also need it for branch.
 i applied this

 http://www.lyx.org/trac/changeset/38655

 ok for branch?

Yes, please.

Richard

Re: wrong latex output with decimally aligned columns

2011-05-09 Thread Edwin Leuven 
Richard Heck rgh...@bobjweil.com wrote:
 Edwin Leuven wrote:
 ok for branch?

 Yes, please.

done:

http://www.lyx.org/trac/changeset/38658

Re: wrong latex output with decimally aligned columns

2011-05-09 Thread Richard Heck 
On 05/09/2011 11:22 AM, Edwin Leuven wrote:
 Richard Heck rgh...@bobjweil.com wrote:
 Edwin Leuven wrote:
 ok for branch?

 Yes, please.
 done:

 http://www.lyx.org/trac/changeset/38658

Thanks.

rh

Re: wrong latex output with decimally aligned columns

2011-05-09 Thread Edwin Leuven 
Richard Heck  wrote:
> Anyway, I'll have a look at the patch later. For now, it looks good, and
> of course we'll also need it for branch.

i applied this

http://www.lyx.org/trac/changeset/38655

ok for branch?

edwin

Re: wrong latex output with decimally aligned columns

2011-05-09 Thread Richard Heck 
On 05/09/2011 08:34 AM, Edwin Leuven wrote:
> Richard Heck  wrote:
>> Anyway, I'll have a look at the patch later. For now, it looks good, and
>> of course we'll also need it for branch.
> i applied this
>
> http://www.lyx.org/trac/changeset/38655
>
> ok for branch?
>
Yes, please.

Richard

Re: wrong latex output with decimally aligned columns

2011-05-09 Thread Edwin Leuven 
Richard Heck  wrote:
> Edwin Leuven wrote:
>> ok for branch?
>>
> Yes, please.

done:

http://www.lyx.org/trac/changeset/38658

Re: wrong latex output with decimally aligned columns

2011-05-09 Thread Richard Heck 
On 05/09/2011 11:22 AM, Edwin Leuven wrote:
> Richard Heck  wrote:
>> Edwin Leuven wrote:
>>> ok for branch?
>>>
>> Yes, please.
> done:
>
> http://www.lyx.org/trac/changeset/38658
>
Thanks.

rh

Re: wrong latex output with decimally aligned columns

2011-05-08 Thread Richard Heck 
On 05/08/2011 09:38 AM, Edwin Leuven wrote:
 hi guys,

 the latex output for decimally aligned columns depends on cached
 metrics info (the member decimal_width)

 this means that we only get the right output if these members are up-to-date

 the attached patch corrects this

 in the meantime to be sure to get the proper output one needs to 1)
 open all insets, 2) scroll from the beginning to the end, 3) generate
 output ... ouch! :(

 i don't know why i never was not bitten by this bug before

The answer, I expect, is that it was less frequent before export in
threads, since, if you had happened to have gone through the document,
you wouldn't have seen the bug. But it would still have been there, much
like #7525. It might have been only one table that was wrong, though,
and maybe you wouldn't even have noticed.

Anyway, I'll have a look at the patch later. For now, it looks good, and
of course we'll also need it for branch.

Richard

Re: wrong latex output with decimally aligned columns

2011-05-08 Thread Richard Heck 
On 05/08/2011 09:38 AM, Edwin Leuven wrote:
> hi guys,
>
> the latex output for decimally aligned columns depends on cached
> metrics info (the member decimal_width)
>
> this means that we only get the right output if these members are up-to-date
>
> the attached patch corrects this
>
> in the meantime to be sure to get the proper output one needs to 1)
> open all insets, 2) scroll from the beginning to the end, 3) generate
> output ... ouch! :(
>
> i don't know why i never was not bitten by this bug before
>
The answer, I expect, is that it was less frequent before export in
threads, since, if you had happened to have gone through the document,
you wouldn't have seen the bug. But it would still have been there, much
like #7525. It might have been only one table that was wrong, though,
and maybe you wouldn't even have noticed.

Anyway, I'll have a look at the patch later. For now, it looks good, and
of course we'll also need it for branch.

Richard