Jean-Marc Lasgouttes wrote:
The following patch cleans up the insetcaption code wrt allowing inset
insertion:
- allow graphics inset, since it has been working for some time (but I
forgot to tell)
What is this about?
- use insetAllowed to disallow some insets (as should be)
I just followed Georg's advice to reimplement getStatus(). If you voiced
your opinion at that time this would have saved me some work and yours.
The is still the input inset that has been forbidden, and I am not
sure why. I there a _really_ _good_ reason for that? I can think of
valid uses (hint: a file can contain a single word, or a time stamp).
Indeed, I did not think of that. Enable it than.
This has been my first occasion to test the insetcaption, and I have
to say that I still do not know why it is useful...
It is very useful IMO and I want to do more with it.
I do not like much the addition of empty lines above and below "just
in case" since I do not know many people that would need both...
Again, I asked opinion about that but nobody answered so I just did what
seems logical to me. Besides, 1) DEPM will take care of that if it is
not used and 2) LateX does not care about the additional lines.
Instead of disabling break_paragraph, it may have been better to
change it to handle the break-before/after caption correctly.
Be my guest. Even if I think that this break-before/after concept is not
easy to grasp for a newbie.
Abdel.
