Jean-Marc Lasgouttes wrote:
>>>>>> "Georg" == Georg Baum
>>>>>> <[EMAIL PROTECTED]>
>>>>>> writes:
>
> Georg> IIRC we have that in bugzilla.
>
> Part of this one?
> http://bugzilla.lyx.org/show_bug.cgi?id=2582
I think so, but am not sure.
> I did not know we has so many bugs about scriptinset.
We have too many bugs for many components of LyX :-(
> Georg> We will however get complaints if we export \sum_{} to LaTeX.
>
> Stylistic complaints, or LaTeX errors?
The former. "\sum_{}xxx" works fine and gives the same output as "\sum xxx",
but I would not be surprised if that would be different with other bases
than \sum.
> All my patch does is remove empty cells when notifyCursorLeaves is
> called. If I leave the cell by clicking somewhere else, the method is
> not invoked and the empty subscript remains. But in this case, there
> is a blue rectangle showing the empty superscript, so people know
> about it.
True, but the inset should nevertheless disappear (but it behaves like this
in 1.3, too).
> Previously, recordUndoInset was equivalent to running recordUndo in
> the enclosing inset (one level down in the cursor stack). This has the
> effect of saving the whole enclosing inset/paragraph, which is what we
> want. However, this means that, on undo, the cursor is set in this
> enclosing inset, and not inside the original inset.
>
> What the patch does is just run doRecordUndo manually and use the real
> cursor as 5th argument.
And it was that simple? Great. That is exactly the behaviour Martin and me
wanted to have some time ago, but did not know how to implement it. Thanks
for the explanation.
Georg
