Re: [patch] getRowNearY (was: anchor_row_)

2003-08-25 Thread Andre Poenitz
On Sat, Aug 23, 2003 at 02:20:53AM +0200, Alfredo Braunstein wrote: > Andre Poenitz wrote: > > > On Sat, Aug 16, 2003 at 12:51:20AM +0200, Alfredo Braunstein wrote: > >> Alfredo Braunstein wrote: > >> > >> Here's a patch with both changes + some minor cosmetic stuff > > > > Now we have a similar

Re: [patch] getRowNearY (was: anchor_row_)

2003-08-23 Thread Alfredo Braunstein
Alfredo Braunstein wrote: > Andre Poenitz wrote: >> Now we have a similar problem when "going up" (hitting the ceiling >> brings you back to the bottom, but this is less bad than the other way >> round... > > Humm... this means getRowNearY is being called with y < 0... In any case, > we are doing

Re: [patch] getRowNearY (was: anchor_row_)

2003-08-22 Thread Alfredo Braunstein
Andre Poenitz wrote: > On Sat, Aug 16, 2003 at 12:51:20AM +0200, Alfredo Braunstein wrote: >> Alfredo Braunstein wrote: >> >> Here's a patch with both changes + some minor cosmetic stuff > > Now we have a similar problem when "going up" (hitting the ceiling > brings you back to the bottom, but t

Re: [patch] getRowNearY (was: anchor_row_)

2003-08-18 Thread Andre Poenitz
On Sat, Aug 16, 2003 at 12:51:20AM +0200, Alfredo Braunstein wrote: > Alfredo Braunstein wrote: > > Here's a patch with both changes + some minor cosmetic stuff Now we have a similar problem when "going up" (hitting the ceiling brings you back to the bottom, but this is less bad than the other wa

Re: [patch] getRowNearY (was: anchor_row_)

2003-08-16 Thread John Levon
On Sat, Aug 16, 2003 at 08:46:19AM +0200, Alfredo Braunstein wrote: > Arghh... cannot get emacs to align with only spaces (.emacs help welcomed). Wish I could help you ... > But am I right in that this should be aligned with ~40 spaces? Yes. "Spaces for alignment, tabs for indentation" :) rega

Re: [patch] getRowNearY (was: anchor_row_)

2003-08-15 Thread Alfredo Braunstein
Alfredo Braunstein wrote: > John Levon wrote: > >> On Sat, Aug 16, 2003 at 12:51:20AM +0200, Alfredo Braunstein wrote: >> >>> RowList::iterator LyXText::getRowNearY(int & y, >>> - ParagraphList::iterator & pit) const >>> + ParagraphList::iterator & pit

Re: [patch] getRowNearY (was: anchor_row_)

2003-08-15 Thread Alfredo Braunstein
John Levon wrote: > On Sat, Aug 16, 2003 at 12:51:20AM +0200, Alfredo Braunstein wrote: > >> RowList::iterator LyXText::getRowNearY(int & y, >> - ParagraphList::iterator & pit) const >> + ParagraphList::iterator & pit) >> const > > You're uglifying the

Re: [patch] getRowNearY (was: anchor_row_)

2003-08-15 Thread John Levon
On Sat, Aug 16, 2003 at 12:51:20AM +0200, Alfredo Braunstein wrote: > RowList::iterator LyXText::getRowNearY(int & y, > - ParagraphList::iterator & pit) const > + ParagraphList::iterator & pit) const You're uglifying the source. I guess you use a small

Re: [patch] getRowNearY (was: anchor_row_)

2003-08-15 Thread Alfredo Braunstein
Alfredo Braunstein wrote: Here's a patch with both changes + some minor cosmetic stuff Index: text.C === RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/text.C,v retrieving revision 1.421 diff -u -p -u -r1.421 text.C --- text.C 1

[patch] getRowNearY (was: anchor_row_)

2003-08-15 Thread Alfredo Braunstein
On Friday 15 August 2003 17:19, André wrote: > > > Ok. What about paragraph count from beginning plus position within? > > > > Fine... and you can even drop the position I guess. > > > > I wonder if this anchor_par_ shouldn't be stored on the bufferview > > instead of the lyxtext. > > Good idea a