Jean-Marc Lasgouttes wrote:
I understand this. But if breakParagraph adjusts the cursor, it is not to set some DocIterator, it is to set the View cursor, which is and has to be a LCursor. It do not see why this part should become a Model action. You can see the bugs that this produces in the View.
Thinking a bit more about this. Your patch makes sense in the interim period were the MV separation is not complete in breakParagraph(). In the future, LyXText::setCursor(LCursor &, ...) should be transferred to InsetText. As I am not going to do a lot of cleanup now, I guess you can go ahead, sorry for the noise.
Abdel.
