Jean-Marc Lasgouttes wrote:
"Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:

In the new getPos, you pass 3 parameters that all should com from
one cursor. This is wrong, especially since the method os useful to
find the position of a cursor.

Abdelrazak> IMHO, what is wrong really is this cursorPos() method.
Abdelrazak> IIUC an inset is a model of some part of the LyX document.
Abdelrazak> As such it should be independent of the cursor position.
Abdelrazak> It is fine if the inset returns its metrics but that's it.
Abdelrazak> All the rest should covered externally to the inset.

I agree that these things should be moved out of insets.

I've done enough cleanups for 1.5 ;-)

Abdel.

Reply via email to