]> git.lyx.org Git - features.git/commitdiff
GuiTabular: disabling is now done on textChanged()
authorScott Kostyshak <skostysh@lyx.org>
Tue, 19 Mar 2013 18:48:20 +0000 (14:48 -0400)
committerScott Kostyshak <skostysh@lyx.org>
Tue, 19 Mar 2013 18:48:20 +0000 (14:48 -0400)
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.

src/frontends/qt4/GuiTabular.cpp

index 3e2fafccf58b60d9e17b49b1b6fa49c188b9e8cc..312b14fd874b0639e06be08b0158e5871db3de55 100644 (file)
@@ -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()));