From: Scott Kostyshak <skostysh@lyx.org>
Date: Tue, 19 Mar 2013 18:48:20 +0000 (-0400)
Subject: GuiTabular: disabling is now done on textChanged()
X-Git-Tag: 2.1.0beta1~520
X-Git-Url: https://git.lyx.org/gitweb/?a=commitdiff_plain;h=ccd46cbe8ef5e285e00ab3819da0e22cecf84da1;p=features.git

GuiTabular: disabling is now done on textChanged()

Before, it was done on editingFinished(). The new behavior is more
intuitive because it is easier for the user to see how editing the box
is connected with enabling/disabling the other widgets. Before, the user
would not get instant feedback and would have to click away before the
connection is revealed. This new behavior is less efficient, however,
because checkEnabled() is called after every keystroke.
---

diff --git a/src/frontends/qt4/GuiTabular.cpp b/src/frontends/qt4/GuiTabular.cpp
index 3e2fafccf5..312b14fd87 100644
--- a/src/frontends/qt4/GuiTabular.cpp
+++ b/src/frontends/qt4/GuiTabular.cpp
@@ -127,7 +127,7 @@ GuiTabular::GuiTabular(QWidget * parent)
 		this, SLOT(checkEnabled()));
 	connect(captionStatusCB, SIGNAL(clicked()),
 		this, SLOT(checkEnabled()));
-	connect(specialAlignmentED, SIGNAL(editingFinished()),
+	connect(specialAlignmentED, SIGNAL(textEdited(const QString &)),
 		this, SLOT(checkEnabled()));
 	connect(columnWidthED, SIGNAL(textEdited(const QString &)),
 		this, SLOT(checkEnabled()));