Alfredo Braunstein wrote: >> below is a horrible mix of the two... > > But it's absolutely not fault of the patch. The absolute coordinates > were needed, but absent. So it's all the rest that is at fault. We > will have this sort of stuff until all have absolute coords.
Sorry, I should have made myself clear. I think your idea is a good one. I was merely highlighting that 'intermediate code' will be hard for the unenlightened (me!) to follow unless it evolves to 'absolute coords everywhere' code. > Basically, I cannot do (nor would be wise) all in one step Agreed. > Why this particular piece is ugly: Oh, I just got bored reading the patch and scrolled to the end ;-) > in inset::draw we have screen coordinates x,y translated to absolute > doc coords (i.e. y += top_y) > in inset::edit we translate screen coordinates to lyxtext > coordinates (x -= txt.x , y += top_y - txt.y) > (see my coordinates status on the first mail) Yes, I understand. I've been reading your mails ;-) -- Angus
