>>>>> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> Jean-Marc Lasgouttes wrote: >> I finally managed to find the real reason for bug 3143. The problem >> was a seemingly gratuitous change in >> http://www.lyx.org/trac/changeset/16433 (the model/view separation >> of paragraph/lyxtext): Abdelrazak> It was not gratuitous. The objective is to completely free Abdelrazak> LyXText from LCursor. Remember, LyXText is about the Abdelrazak> 'Model', LCursor brings here the 'View'. Hmm, I thought Paragraph was the model and LyXText the view. >> I propose to apply the following patch, even it the difference is >> not huge (at least we understand where the bug comes from instead >> of adding setCursor here and there). Abdelrazak> Please no. OK, OK. But I really do not understand the tenuous difference between setting cur or cur.top(). >> There are other such changes in the patch, but it is difficult to >> see whether they make a difference. Of course a rationalization of >> these setCursor methods would make sense... Abdelrazak> It would be better to modify setFont() to take a Abdelrazak> DocIterator instead of a LCursor and call that from inside Abdelrazak> the setCursor() where it is missing. You mean setCurrentFont? But current_font is clearly a view thing in your scheme, isn't it? You lost me. JMarc
