Re: Give insets a full Row

2002-04-15 Thread Jean-Marc Lasgouttes
Juergen == Juergen Vigna [EMAIL PROTECTED] writes: Juergen You may be right but we overwirte it only if the depth 5, Juergen should I add code anyway to put it more to the right? OK, I think it is fine as it is. Let's wait for actual complaints before doing anything. JMarc

Re: Give insets a full Row

2002-04-15 Thread Jean-Marc Lasgouttes
> "Juergen" == Juergen Vigna <[EMAIL PROTECTED]> writes: Juergen> You may be right but we overwirte it only if the depth > 5, Juergen> should I add code anyway to put it more to the right? OK, I think it is fine as it is. Let's wait for actual complaints before doing anything. JMarc

RE: Give insets a full Row

2002-04-12 Thread Juergen Vigna
On 11-Apr-2002 Juergen Vigna wrote: The attached small patch changes LyX's behaviour for NeedFullRow insets in that it does give them ALWAYS the full row for drawing and does not care if there is some indent or depth. IMO this is correct and should be done, also this fixes a few problems we

Re: Give insets a full Row

2002-04-12 Thread Lars Gullik Bjønnes
Juergen Vigna [EMAIL PROTECTED] writes: | The attached small patch changes LyX's behaviour for NeedFullRow insets in | that it does give them ALWAYS the full row for drawing and does not care if | there is some indent or depth. IMO this is correct and should be done, also | this fixes a few

Re: Give insets a full Row

2002-04-12 Thread Lars Gullik Bjønnes
Juergen Vigna [EMAIL PROTECTED] writes: | On 11-Apr-2002 Juergen Vigna wrote: The attached small patch changes LyX's behaviour for NeedFullRow insets in that it does give them ALWAYS the full row for drawing and does not care if there is some indent or depth. IMO this is correct and should be

Re: Give insets a full Row

2002-04-12 Thread Andre Poenitz
On Fri, Apr 12, 2002 at 10:31:15AM +0200, Lars Gullik Bjønnes wrote: | + Inset * ins; | + if ((row-par()-getChar(row-pos()) == Paragraph::META_INSET) | + (ins=row-par()-getInset(row-pos())) | + (ins-needFullRow() || ins-display())) | + return

Re: Give insets a full Row

2002-04-12 Thread Jean-Marc Lasgouttes
Juergen == Juergen Vigna [EMAIL PROTECTED] writes: Juergen The attached small patch changes LyX's behaviour for Juergen NeedFullRow insets in that it does give them ALWAYS the full Juergen row for drawing and does not care if there is some indent or Juergen depth. IMO this is correct and should

Re: Give insets a full Row

2002-04-12 Thread Jean-Marc Lasgouttes
Juergen == Juergen Vigna [EMAIL PROTECTED] writes: Juergen On 11-Apr-2002 Juergen Vigna wrote: The attached small patch changes LyX's behaviour for NeedFullRow insets in that it does give them ALWAYS the full row for drawing and does not care if there is some indent or depth. IMO this is

Re: Give insets a full Row

2002-04-12 Thread Juergen Vigna
On 12-Apr-2002 Lars Gullik Bjønnes wrote: | + Inset * ins; | + if ((row-par()-getChar(row-pos()) == Paragraph::META_INSET) | + (ins=row-par()-getInset(row-pos())) | + (ins-needFullRow() || ins-display())) | + return LYX_PAPER_MARGIN; | +

Re: Give insets a full Row

2002-04-12 Thread Juergen Vigna
On 12-Apr-2002 Jean-Marc Lasgouttes wrote: Juergen == Juergen Vigna [EMAIL PROTECTED] writes: Juergen The attached small patch changes LyX's behaviour for Juergen NeedFullRow insets in that it does give them ALWAYS the full Juergen row for drawing and does not care if there is some indent

Re: Give insets a full Row

2002-04-12 Thread Juergen Vigna
On 12-Apr-2002 Jean-Marc Lasgouttes wrote: Juergen Hmm, I did not get any complaints so I will commit this patch Juergen later this morning. Huh? I just posted my own complaint one minute ago... Well I did not get it until now so! Anyway if you think this is wrong we can always revert it

Re: Give insets a full Row

2002-04-12 Thread Jean-Marc Lasgouttes
Juergen == Juergen Vigna [EMAIL PROTECTED] writes: Juergen On 12-Apr-2002 Jean-Marc Lasgouttes wrote: Juergen Hmm, I did not get any complaints so I will commit this patch Juergen later this morning. Huh? I just posted my own complaint one minute ago... Juergen Well I did not get it until

RE: Give insets a full Row

2002-04-12 Thread Juergen Vigna
On 11-Apr-2002 Juergen Vigna wrote: > The attached small patch changes LyX's behaviour for NeedFullRow insets in > that it does give them ALWAYS the full row for drawing and does not care if > there is some indent or depth. IMO this is correct and should be done, also > this fixes a few problems

Re: Give insets a full Row

2002-04-12 Thread Lars Gullik Bjønnes
Juergen Vigna <[EMAIL PROTECTED]> writes: | The attached small patch changes LyX's behaviour for NeedFullRow insets in | that it does give them ALWAYS the full row for drawing and does not care if | there is some indent or depth. IMO this is correct and should be done, also | this fixes a few

Re: Give insets a full Row

2002-04-12 Thread Lars Gullik Bjønnes
Juergen Vigna <[EMAIL PROTECTED]> writes: | On 11-Apr-2002 Juergen Vigna wrote: >> The attached small patch changes LyX's behaviour for NeedFullRow insets in >> that it does give them ALWAYS the full row for drawing and does not care if >> there is some indent or depth. IMO this is correct and

Re: Give insets a full Row

2002-04-12 Thread Andre Poenitz
On Fri, Apr 12, 2002 at 10:31:15AM +0200, Lars Gullik Bjønnes wrote: > | + Inset * ins; > | + if ((row->par()->getChar(row->pos()) == Paragraph::META_INSET) && > | + (ins=row->par()->getInset(row->pos())) && > | + (ins->needFullRow() || ins->display())) > | +

Re: Give insets a full Row

2002-04-12 Thread Jean-Marc Lasgouttes
> "Juergen" == Juergen Vigna <[EMAIL PROTECTED]> writes: Juergen> The attached small patch changes LyX's behaviour for Juergen> NeedFullRow insets in that it does give them ALWAYS the full Juergen> row for drawing and does not care if there is some indent or Juergen> depth. IMO this is

Re: Give insets a full Row

2002-04-12 Thread Jean-Marc Lasgouttes
> "Juergen" == Juergen Vigna <[EMAIL PROTECTED]> writes: Juergen> On 11-Apr-2002 Juergen Vigna wrote: >> The attached small patch changes LyX's behaviour for NeedFullRow >> insets in that it does give them ALWAYS the full row for drawing >> and does not care if there is some indent or depth.

Re: Give insets a full Row

2002-04-12 Thread Juergen Vigna
On 12-Apr-2002 Lars Gullik Bjønnes wrote: > | + Inset * ins; > | + if ((row->par()->getChar(row->pos()) == Paragraph::META_INSET) && > | + (ins=row->par()->getInset(row->pos())) && > | + (ins->needFullRow() || ins->display())) > | + return

Re: Give insets a full Row

2002-04-12 Thread Juergen Vigna
On 12-Apr-2002 Jean-Marc Lasgouttes wrote: >> "Juergen" == Juergen Vigna <[EMAIL PROTECTED]> writes: > > Juergen> The attached small patch changes LyX's behaviour for > Juergen> NeedFullRow insets in that it does give them ALWAYS the full > Juergen> row for drawing and does not care if

Re: Give insets a full Row

2002-04-12 Thread Juergen Vigna
On 12-Apr-2002 Jean-Marc Lasgouttes wrote: > Juergen> Hmm, I did not get any complaints so I will commit this patch > Juergen> later this morning. > > Huh? I just posted my own complaint one minute ago... Well I did not get it until now so! Anyway if you think this is wrong we can always

Re: Give insets a full Row

2002-04-12 Thread Jean-Marc Lasgouttes
> "Juergen" == Juergen Vigna <[EMAIL PROTECTED]> writes: Juergen> On 12-Apr-2002 Jean-Marc Lasgouttes wrote: Juergen> Hmm, I did not get any complaints so I will commit this patch Juergen> later this morning. >> Huh? I just posted my own complaint one minute ago... Juergen> Well I did not