Le 29/01/2016 01:30, Uwe Stöhr a écrit :
Attached is a patch to fix bug http://www.lyx.org/trac/ticket/8712.

OK, I'll try to have a go at it. The problem is that every time I look at this box code, my eyes bleed:
* type as a string instead of enum
* special length as string instead of properly extending the Length class
* articificial use_parbox, use_makebox, inner_box parameters that have to be set in a very precise manner in fragile code that I am not able to follow.

I know this has been done to introduce features that are "important to the user", but at some time, sanity is nice too. The insetbox parameters should describe what we think the box is, not how LaTeX output is going to happen.

With that out of my chest, here is an actual question:

But in this case a makebox must be used. This is already handled
in the box dialog. The bug is that it is not yet handled if the box
changing commands are executed via the context menu.

Can you point me to the relevant code in GuiBox.cpp?

Additional question: what is this
  else // if (for_box)
line about?

JMarc

Reply via email to