On Wed, Feb 16, 2011 at 11:57:22PM +0100, Vincent van Ravesteijn wrote: > Op 16-2-2011 23:19, for...@lyx.org schreef: > >Author: forenr > >Date: Wed Feb 16 23:19:49 2011 > >New Revision: 37704 > >URL: http://www.lyx.org/trac/changeset/37704 > > > >Log: > >Don't disable apply button if one (or more) of vertical alignment, rotation, > >or long table settings are changed. Fixes bug #7308. > > > >Modified: > > lyx-devel/trunk/src/insets/InsetTabular.cpp > > > >Modified: lyx-devel/trunk/src/insets/InsetTabular.cpp > >============================================================================== > >--- lyx-devel/trunk/src/insets/InsetTabular.cpp Wed Feb 16 22:08:18 > >2011 (r37703) > >+++ lyx-devel/trunk/src/insets/InsetTabular.cpp Wed Feb 16 23:19:49 > >2011 (r37704) > >@@ -4323,8 +4323,10 @@ > > return true; > > > > case Tabular::SET_TABULAR_WIDTH: > >- status.setEnabled(!tabular.rotate&& > >!tabular.is_long_tabular > >- && tabular.tabular_valignment == > >Tabular::LYX_VALIGN_MIDDLE); > >+ status.setEnabled(tabular.tabular_width.zero() > >+ || (!tabular.rotate&& !tabular.is_long_tabular > >+ && tabular.tabular_valignment == > >+ Tabular::LYX_VALIGN_MIDDLE)); > > break; > > > > case Tabular::SET_DECIMAL_POINT: > > This is not correct.
I'm sorry, you're wrong. > This is basically saying that SET_TABULAR_WIDTH should be allowed > for longtables, because tabular_width is zero by definition for > longtables. It works just fine. -- Enrico
