Am 11.09.2012 11:55, schrieb Pavel Sanda:

+       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?

It deletes the decimal alignment from the combobox item list because this alignment is not possible in multicolumns/rows. You are right that hardcoding its number is error prone in case a further item is added. I changed this now:
http://www.lyx.org/trac/changeset/b64d17e9dcaeb84a94095622c93a051e4d93eab7/lyxgit
(the change from ! to == -1 is hereby necessary)

thanks and regards
Uwe

Reply via email to