> I am going to work a bit on the bookmark handling if you don't mind (see > patch). The goal is to extend it to support in-inset bookmarks. > > Bo, is that OK for you?
If I understand your patch correctly, you move cursor inside moveToPosition, instead of returning a new position. There is a dirty trick used in the handling of bookmarks: bookmark id needs to be converted to pit because only the latter can be saved in session. Therefore, moveToPosition is called when a buffer is closed and switched, and we DO not want to move the cursor in this cases, especially for the buffer switch one. I guess I am talking about this bug: http://bugzilla.lyx.org/show_bug.cgi?id=3171 Cheers, Bo