Uwe StĂśhr wrote:
> commit 9d05ac5c9124608cdbc989161794fe56aa839e1d
> Author: Uwe StĂśhr <uwesto...@lyx.org>
> Date: Tue Sep 11 02:10:29 2012 +0200
>
> Fix UI for decimal table alignment (bug #8285)
>
> Fixes 2 issues:
> 1. LyX uses for a decimal alignment a multicolumn and having for a cell a
> multicolumn _and_ a multirow is invalid LaTeX.
> 2. It was impossible to unset a decimal alignment via the context menu or
> toolbar button.
>
> diff --git a/src/frontends/qt4/GuiTabular.cpp
> b/src/frontends/qt4/GuiTabular.cpp
> index 79706d4..ee4d1df 100644
> --- a/src/frontends/qt4/GuiTabular.cpp
> +++ b/src/frontends/qt4/GuiTabular.cpp
> @@ -209,6 +209,13 @@ void GuiTabular::checkEnabled()
> hAlignCO->setEnabled(!(multirowCB->isChecked()
> && !widgetsToLength(columnWidthED, columnWidthUnitLC).empty())
> && specialAlignmentED->text().isEmpty());
> + // decimal alignment is only possible for non-multicol and non-multirow
> cells
> + if ((multicolumnCB->isChecked() || multirowCB->isChecked())
> + && hAlignCO->findText(qt_("At Decimal Separator")))
> + hAlignCO->removeItem(3);
Uwe, can you comment on this hAlignCO->removeItem(3); line, it looks terrible.
What is the intention?
Pavel
