On Sun, Jul 03, 2005 at 11:03:23AM +0200, Georg Baum wrote: > Am Samstag, 2. Juli 2005 20:43 schrieb Andre Poenitz: > > > For grid-like structures it is sometimes convienient to extract row > > and col from idx. So there are some helper functions. And sometimes > > it is useful to give the user the impression that row and col are > > primary because that's what the user expects if he e.g. works in > > grids. In this situation we just adjust the idx. You found places > > where this adjsutment was not or not properly done. But that means > > just that: The adjustment is wrong, not the scheme. > > I agree. But in Martins defense I have to say that the scheme was not > very clear because of lacking documentation.
I wasn't arguing that it wasn't my fault. > The attached patch adds some documentation, removes col_ and row_ > again, fixes wrong index calculation for "delete-column" and even > fixes one further case of cursor misadjustment in insettabular. I am > going to apply this soon unless there are still objections. Looks fine. > BTW André, do you have any idea why undoing an added column in mathed > does not work? If it does (and I can't check right now) possibly because of this special idx fixing. An in-scheme-workaround for this probem would be to go one nesting level up in the hierarchy and use recordUndo there. Wastes a few bytes on the undo stack but should be bearablem, especially considering the frequency of such operations. Andre'
