Uwe Stöhr wrote:
> This patch adds (Broken) to the insert->float->text wrap float
> entry, so it becomes
>
> Insert->Float->Text wrap float (Broken)
>
> This because it _is_ broken. I don't know of any LaTeX version
> where this thing works well at all.

Veto!
We can't tell users that certain functions are broken, either we remove this function or fix it. when you think it is broken.
I see no problem telling users that we have some feature that
doesn't always work as intended. But they need a warning,
if we're going to provide floatflt.
But in the case of wrap floats, it works as described in the floatflt package manual, so it is not broken. The details how to handle this float type is explained in the EmbeddedObjects manual.
Nobody that want a wrapped figure will expect that it sometimes disappear from output, depending on where the page breaks happen. They insert such figures. If they fail immediately - fine! Then they go with a plain float.

If they work immediately but disappear later after a page 1 edit moved the
page breaks - well yuck. I consider it normal use to not read about every little piece of functionality in the docs, and the failure mode here is nasty. Information loss with no warning. Now, if LyX considered the loss of a wrap float fatal, then we might not need the warning in the menu itself. A message like "wrap float failure, see the userguide" would be fine too.
The warning was the easiest I could come up with.

It is known that floatflt make problems when the wrap floats are close to page breaks, it is also known that the flotflt package produces much better results. So to improve the situation we should try to fix bug 2931 instead:
http://bugzilla.lyx.org/show_bug.cgi?id=2931
Sure - if wrapfig is better then this is a much better solution.

Helge Hafting

Reply via email to