Jürgen Spitzmüller wrote:
Abdelrazak Younes wrote:
One general comment: I know that the qt4 frontend is not as clean as it
should be but we should try to not add additional dependency to the core
(src/*.h). So, please move the map to ControlDocument instead and
fill-in the cursor position with a helper function.
OK.
Thanks.
Also I am puzzled that we need to memorize the scrollbar position. If we
move the internal cursor, won't the scrollbar automatically follow?
we can use ensureCursorVisible(), but then the window content will not be at
the same position. The impression is "jumping" content, which looks ugly.
Not if you do that _before_ showing the contents, will it?
I see that QTextEdit::moveCursor() is introduced only in 4.2 but that
seem exactly what we need.
I think moveCursor moves the cursor in the widget, which is not what we need
(setCursor seems more appropriate).
OK.
Abdel.
