cursorup/down

2003-11-18 Thread Alfredo Braunstein
- shouldn't these be handled by the BufferView? As opposed to right/left, they (should/used to) ignore the document structure. Right now they enter in insets ignoring it, but exit insets losing the x coordinate (by some dispatchresult flag I presume). Alfredo

Re: [patch] towards simplified left/right/up/down

2003-11-18 Thread Alfredo Braunstein
Andre Poenitz wrote: > +cursorRight(bv()); note that cursorRight takes a bool argument. (Dunno what's for, though ;-) Alfredo

Re: [patch] remove InsetText::cx/xy

2003-11-18 Thread Andre Poenitz
On Mon, Nov 17, 2003 at 07:16:15PM +0100, Alfredo Braunstein wrote: > Andre Poenitz wrote: > > > not really needed. > > > > What's the current state of coordinate handling btw? > > My selection2.diff patch makes passing mouse events to the main LyXText in > coordinates relatives to the LyXText i

Re: some current bugs

2003-11-18 Thread Andre Poenitz
On Mon, Nov 17, 2003 at 07:21:22PM +0100, Alfredo Braunstein wrote: > Andre Poenitz wrote: > > > Alfredo is still trying to be nice. We have to work on that... > > So I can become an angry caveman too? ;-) Would be a step forward on the UIU trail ("user interface unification") Andre'

Re: tex2lyx and optional arguments

2003-11-18 Thread Andre Poenitz
On Mon, Nov 17, 2003 at 08:58:36PM +0100, Georg Baum wrote: > Am Montag, 17. November 2003 09:52 schrieb Andre Poenitz: > > Looks good. Is this ready for 'commit'? > > Yes. Maybe with the size() -> !empty() change suggested by Angus. BTW, this > construct is used at a few other places in tex2lyx,

Re: cursorup/down

2003-11-18 Thread Andre Poenitz
On Tue, Nov 18, 2003 at 09:19:16AM +0100, Alfredo Braunstein wrote: > - shouldn't these be handled by the BufferView? Maybe. I am not sure, though. > As opposed to right/left, they (should/used to) ignore the document > structure. Getting up/down (sort of) right was _the_ problem when glueing th

Re: [patch] towards simplified left/right/up/down

2003-11-18 Thread Andre Poenitz
On Tue, Nov 18, 2003 at 09:24:45AM +0100, Alfredo Braunstein wrote: > Andre Poenitz wrote: > > > +cursorRight(bv()); > > note that cursorRight takes a bool argument. (Dunno what's for, though ;-) funny. I copied this from somewhere else. So the argument doesn't look terribly important, d

Re: [patch] remove InsetText::cx/xy

2003-11-18 Thread Alfredo Braunstein
Andre Poenitz wrote: > On Mon, Nov 17, 2003 at 07:16:15PM +0100, Alfredo Braunstein wrote: >> Andre Poenitz wrote: >> >> > not really needed. >> > >> > What's the current state of coordinate handling btw? >> >> My selection2.diff patch makes passing mouse events to the main LyXText >> in coordi

Re: [patch] towards simplified left/right/up/down

2003-11-18 Thread Alfredo Braunstein
Andre Poenitz wrote: > On Tue, Nov 18, 2003 at 09:24:45AM +0100, Alfredo Braunstein wrote: >> Andre Poenitz wrote: >> >> > +cursorRight(bv()); >> >> note that cursorRight takes a bool argument. (Dunno what's for, though >> ;-) > > funny. I copied this from somewhere else. So the argumen

Re: [patch] remove InsetText::cx/xy

2003-11-18 Thread Andre Poenitz
On Tue, Nov 18, 2003 at 09:50:32AM +0100, Alfredo Braunstein wrote: > > I noticed some regression that might be related to the > > CursorItem::cached_y_ change. Simply move through the attached doc > > with to see what I mean. The interesting point comes when > > leaving the inner inset. > > I se

Re: [Patch] XML sanitation part II

2003-11-18 Thread Martin Vermeer
On Mon, Nov 17, 2003 at 11:16:57PM +0200, Martin Vermeer spake thusly: > My primary objective with this is to parametrize the docbook list code > by moving stuff from LyX to the .layout files. (Yes, this makes them a > little messier looking right now. Doesn't bother the end user nor the > LaTeX

Re: cursorup/down

2003-11-18 Thread Alfredo Braunstein
Andre Poenitz wrote: > On Tue, Nov 18, 2003 at 09:19:16AM +0100, Alfredo Braunstein wrote: >> - shouldn't these be handled by the BufferView? > > Maybe. I am not sure, though. > >> As opposed to right/left, they (should/used to) ignore the document >> structure. > > Getting up/down (sort of) ri

Re: [Patch] XML sanitation part II

2003-11-18 Thread Lars Gullik Bjønnes
Martin Vermeer <[EMAIL PROTECTED]> writes: | On Mon, Nov 17, 2003 at 11:16:57PM +0200, Martin Vermeer spake thusly: | >> My primary objective with this is to parametrize the docbook list code >> by moving stuff from LyX to the .layout files. (Yes, this makes them a >> little messier looking righ

Re: cursorup/down

2003-11-18 Thread Andre Poenitz
On Tue, Nov 18, 2003 at 10:01:35AM +0100, Alfredo Braunstein wrote: > Andre Poenitz wrote: > > > On Tue, Nov 18, 2003 at 09:19:16AM +0100, Alfredo Braunstein wrote: > >> - shouldn't these be handled by the BufferView? > > > > Maybe. I am not sure, though. > > > >> As opposed to right/left, they

Re: cursorup/down

2003-11-18 Thread Alfredo Braunstein
Alfredo Braunstein wrote: > Mmm... but we don't use x_target_ at all IIUC. We simply return with > DISPATCHED_RIGHT/LEFT when exiting and inset. ugh: FINISHED_UP/DOWN Alfredo

Re: [patch] remove InsetText::cx/xy

2003-11-18 Thread Alfredo Braunstein
Andre Poenitz wrote: >> If I understand correctly, the problem is that we don't 'clear' the >> text->cursor information when we pop or push the LCursor. >> For instance: select something inside the inset, then click outside, then >> enter the inset by or , and you will see the old selection. > >

Re: [patch] remove InsetText::cx/xy

2003-11-18 Thread Andre Poenitz
On Tue, Nov 18, 2003 at 10:54:06AM +0100, Alfredo Braunstein wrote: > Andre Poenitz wrote: > > >> If I understand correctly, the problem is that we don't 'clear' the > >> text->cursor information when we pop or push the LCursor. > >> For instance: select something inside the inset, then click outs

was: [patch] remove InsetText::cx/xy. is: up/down in insets

2003-11-18 Thread Alfredo Braunstein
Andre Poenitz wrote: >> > In the light of IU and simplicity I lean towards the first option, but >> > I am not sure. Behaviour would probably change then (i.e. multi-cell >> > selection in the mathed way, not the current tabular way. I don't see >> > that as disadvantage, but then I am not imparti

Re: was: [patch] remove InsetText::cx/xy. is: up/down in insets

2003-11-18 Thread Angus Leeming
Alfredo Braunstein wrote: > If the only possible drawbacks are selections, then I'd say go for > it... > > (but of course, I'm not the only one here ;-) Don't forget that the main motivation for _all_ these changes is a desire for clean and understandable code. Mathed certainly fits the bill th

Re: was: [patch] remove InsetText::cx/xy. is: up/down in insets

2003-11-18 Thread Andre Poenitz
On Tue, Nov 18, 2003 at 11:16:30AM +0100, Alfredo Braunstein wrote: > Andre Poenitz wrote: > > >> > In the light of IU and simplicity I lean towards the first option, but > >> > I am not sure. Behaviour would probably change then (i.e. multi-cell > >> > selection in the mathed way, not the current

[patch 13x] autogen.sh accepts autoconf 2.58

2003-11-18 Thread Angus Leeming
Jean-Marc, Debian unstable now uses autoconf 2.58 by default. The attached patch tells autogen.sh to allow its use. All works perfectly. Ok to apply to the 1.3.x series? Angus ps, I don't have the 1.4.x tree checked out here. Could someone apply the patch to that for me please?Index: autogen.

Re: [patch 13x] autogen.sh accepts autoconf 2.58

2003-11-18 Thread Andre Poenitz
On Tue, Nov 18, 2003 at 10:37:20AM +, Angus Leeming wrote: > ps, I don't have the 1.4.x tree checked out here. Could someone apply > the patch to that for me please? Done.

Re: [Patch] XML sanitation part II

2003-11-18 Thread Martin Vermeer
On Tue, Nov 18, 2003 at 10:12:31AM +0100, Lars Gullik Bjønnes spake thusly: > > Martin Vermeer <[EMAIL PROTECTED]> writes: > > | On Mon, Nov 17, 2003 at 11:16:57PM +0200, Martin Vermeer spake thusly: > | > >> My primary objective with this is to parametrize the docbook list code > >> by moving

Re: [patch 13x] autogen.sh accepts autoconf 2.58

2003-11-18 Thread Jean-Marc Lasgouttes
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: Angus> Jean-Marc, Debian unstable now uses autoconf 2.58 by default. Angus> The attached patch tells autogen.sh to allow its use. All works Angus> perfectly. Angus> Ok to apply to the 1.3.x series? Yes. JMarc

Re: [patch 13x] autogen.sh accepts autoconf 2.58

2003-11-18 Thread Angus Leeming
Andre Poenitz wrote: > On Tue, Nov 18, 2003 at 10:37:20AM +, Angus Leeming wrote: >> ps, I don't have the 1.4.x tree checked out here. Could someone >> apply the patch to that for me please? > > Done. Vielen Dank. -- Angus

Re: [patch 13x] autogen.sh accepts autoconf 2.58

2003-11-18 Thread Angus Leeming
Jean-Marc Lasgouttes wrote: > Angus> Jean-Marc, Debian unstable now uses autoconf 2.58 by default. > Angus> The attached patch tells autogen.sh to allow its use. All > works Angus> perfectly. > Angus> Ok to apply to the 1.3.x series? > Yes. Merci beaucoup! -- Angus

Re: cursorup/down

2003-11-18 Thread Alfredo Braunstein
Andre Poenitz wrote: >> [I think that the problem with target_x_ is that it is updated *too >> [much*, >> and thus it is completely in-sync with the cursor (and thus useless). >> I'll have a look at it.] > > Might be true... It was. This patch solves this, + some bv() -> true (except in moveUp/

Re: cursorup/down

2003-11-18 Thread Andre Poenitz
On Tue, Nov 18, 2003 at 12:22:38PM +0100, Alfredo Braunstein wrote: > > Might be true... > > It was. > > This patch solves this, + some bv() -> true (except in moveUp/Down, where it > was set to false to allow entering insets) Good. > [another problem: we should set x_target in absolute coordin

Re: cursorup/down

2003-11-18 Thread Alfredo Braunstein
Andre Poenitz wrote: >> [another problem: we should set x_target in absolute coordinates, but >> we do it within LyXText, and LyXText doesn't kown it's own absolute >> x coords. Maybe it should?] > > Absolute document coordinates? Would be feasible. bv->text gets 0,0, the > rest could be compute

Re: [Patch] XML sanitation part II

2003-11-18 Thread Martin Vermeer
> Generically: > > \begin{LatexName} > > item text for itemize/enum >label_textitem text for description >... > > \end{LatexName} No, that would be \begin{LatexName} item text for itemize/enum ... label_textitem text for description ... \end{LatexName}

Re: [patch] towards simplified left/right/up/down

2003-11-18 Thread John Levon
On Tue, Nov 18, 2003 at 09:46:46AM +0100, Andre Poenitz wrote: > > > +cursorRight(bv()); > > > > note that cursorRight takes a bool argument. (Dunno what's for, though ;-) > > funny. I copied this from somewhere else. So the argument doesn't look > terribly important, does it? It was, a

Re: [patch] towards simplified left/right/up/down

2003-11-18 Thread Braunstein Alfredo
John Levon wrote: > It was, at least. The bug above is from when the bv argument was removed > from these methods; somebody wasn't quite careful enough in removing > the bv()'s. Somebody strikes again! Alfredo

Bug ??

2003-11-18 Thread Josef Carel
Hello: Im not sure it is a bug but in any case I need some help. Im just in the begining of using Lyx, version 1.3.3 I can't centered images inserted through float system and I can't find how to move it from the left side. It is a bug or something wrong with my procedure ? Thanks a lot Josef Care

Re: Bug ??

2003-11-18 Thread Martin Vermeer
On Tue, Nov 18, 2003 at 09:26:51PM +0200, Josef Carel spake thusly: > Hello: > Im not sure it is a bug but in any case I need some help. > Im just in the begining of using Lyx, version 1.3.3 > > I can't centered images inserted through float system and > I can't find how to move it from the left

Re: tex2lyx and optional arguments

2003-11-18 Thread Georg Baum
Am Dienstag, 18. November 2003 09:37 schrieb Andre Poenitz: > On Mon, Nov 17, 2003 at 08:58:36PM +0100, Georg Baum wrote: > > Am Montag, 17. November 2003 09:52 schrieb Andre Poenitz: > > > Looks good. Is this ready for 'commit'? > > > > Yes. Maybe with the size() -> !empty() change suggested by An

Re: scrolling (three questions)

2003-11-18 Thread Angus Leeming
On Thursday 30 October 2003 2:00 pm, Ronald Florence wrote: > Angus Leeming <[EMAIL PROTECTED]> writes: > > The fix is trivial (for Qt too) and requires that > > LyX responds only to a subset of mouse events from the underlying > > X11, MacOS or Win32 graphics libraries. In the xforms frontend we u

[patch] intro of absolute LyXText coords + some cursor fixing

2003-11-18 Thread Alfredo Braunstein
This patch introduces document absolute coordinates in LyXText, and use them to solve the target_x problem (almost all operation are still done as before, but now we can possibly simplify/correct code using the absolute coordinates, in small steps). I've also tried to fix cursorNext/Previous, and