* Add comment that clarifies toggled() vs clicked()

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@17061 a592a061-630c-0410-9148-cb99ea01b6c8

diff --git a/src/frontends/qt4/QGraphicsDialog.C b/src/frontends/qt4/QGraphicsDialog.C
index 7d72aa1728dc23a429c5622a4904975081d3fa60..74f4571b26d98ee5d9ca860826696b68d3440f7e 100644 (file)
--- a/src/frontends/qt4/QGraphicsDialog.C
+++ b/src/frontends/qt4/QGraphicsDialog.C
@@ -53,14 +53,18 @@ QGraphicsDialog::QGraphicsDialog(QGraphics * form)
                this, SLOT( change_adaptor() ) );
        connect(subcaption, SIGNAL( textChanged(const QString&) ),
                this, SLOT( change_adaptor() ) );
+
+       // FIXME: we should connect to clicked() when we move to Qt 4.2 because
+       // the toggled(bool) signal is also trigged when we update the widgets
        connect(subfigure, SIGNAL( toggled(bool) ),
                this, SLOT( change_adaptor() ) );
+       connect(displayGB, SIGNAL( toggled(bool) ),
+               this, SLOT( change_adaptor() ) );
+
        connect(latexoptions, SIGNAL( textChanged(const QString&) ),
                this, SLOT( change_adaptor() ) );
        connect(clip, SIGNAL( stateChanged(int) ),
                this, SLOT( change_adaptor() ) );
-       connect(displayGB, SIGNAL( toggled(bool) ),
-               this, SLOT( change_adaptor() ) );
        connect(showCB, SIGNAL( currentIndexChanged(int) ),
                this, SLOT( change_adaptor() ) );
        connect(displayscale, SIGNAL( textChanged(const QString&) ),