Jürgen Spitzmüller wrote:
Abdelrazak Younes wrote:
Here is a radical solution. I should have thought about this earlier!
The dialog will be updated in any case by the LyXFunc::dispatch() method
so the updating done in InsetTabular::dispatch() is superfluous.

This patch
- doesn't compile (is_deleted_ is still used in the destructor)

Yes, I was about to send an updated patch when I noticed this:

- breaks the tabular dialog (try to check something in the dialog, e.g. "multicolumn"; it's not possible).

This is because the controller/dialog/dispatch is so brain damaged that I wonder how it worked up until now.

So we need to investigate further.

I am going to try to put the updateDialog() method in the Controller instead and see how it goes.

Abdel.

Reply via email to