Georg Baum wrote:
Abdelrazak Younes wrote:
My problem is that setXY() needs a BufferView with my change. So I just
commented out this call which, if I'm right is not useful because
drawT() is for text export and has nothing to do with painting on
screen. I guess this call to setXY was just copy&paste from the draw()
method.
I don't think so, see below.
Georg, you worked recently for the text export feature. Are you able to
confirm my findings?
AFAIK the text painter was used to export math formulas as ASCII art.
Therefore the setXY() call makes sense. Imagine that the text painter is
like a real painter, but operating on a very coarse grid of character cells
where each cell can be filled with an ASCII character.
I don't know why it is currently disabled. I do know that we have a bugzilla
request for reenabling it. I believe only Andre can tell whether that is
doable or whether the whole drawT machinery should be removed.
OK, then, adding a BufferView to the TextPainter construction would make
sense. The problem is then that I'll have to add a BufferView argument
to InsetBase::plaintext() and that's a _lot_ of work. And I am not sure
it is right anyway.
So, for now, I'll just put a FIXME in there explaining the situation and
your explanation. We'll update the code when Andre comes back to the list.
Abdel.
