On 3/9/20 9:25 PM, Scott Kostyshak wrote: > Is there a reason to do the following for outline-up and outline-down? > > setCursor(cur, cur.pit(), 0); > > It does not seem to be necessary in my testing, but I'm wondering if > there's a case I'm missing. Or perhaps it is not necessary but is > desired behavior for some reason? > > It has been this way for a long time, at least since 71f356c3. > > Attached is a patch to remove it so that the cursor position does not > change when executing those actions.
Is there a concern that these actions might invalidate the cursor? Maybe that is taken care of along the way, but I've just fixed a bug involving accepting changes in the outliner https://www.lyx.org/trac/ticket/10316 that led to a crash for this reason. Riki -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel