Abdelrazak Younes wrote:
Georg Baum wrote:
Abdelrazak Younes wrote:
Nobody knows what this resetPos() is for?
I had a look at the code, and I believe that it is for horizontal
"scrolling": If you have a table that is wider than the screen and you go
to a cell that is at the right edge then, if you are lucky, the table is
shifted to the left so that you can edit the cell and even go further
right. This does however not work reliably.
If I am right then the variable name scx_ would make some sense: sc for
scrolling. The ChangeLog comments also seem to point in this
direction. And
most of them were done by Jürgen Vigna who is AFAIK in Denmark.
I see. And I see also now that this feature doesn't work any more when
you are navigating in a table bigger than the screen. I would be nice if
we manage to call resetPos() only once whatever code path inside tabular
but after the metrics have been updated.
The feature is back with this patch. I guess draw() is called every
time, isn't it? If so, then all other calls to resetPos() can be deleted.
Abdel.
