Author: spitz
Date: Mon Jan  3 18:21:37 2011
New Revision: 37082
URL: http://www.lyx.org/trac/changeset/37082

Log:
Fix UI glitch in the prefs dialog (bug 7190).

Modified:
   lyx-devel/branches/BRANCH_1_6_X/src/frontends/qt4/GuiPrefs.cpp
   lyx-devel/branches/BRANCH_1_6_X/src/frontends/qt4/GuiPrefs.h
   lyx-devel/branches/BRANCH_1_6_X/status.16x

Modified: lyx-devel/branches/BRANCH_1_6_X/src/frontends/qt4/GuiPrefs.cpp
==============================================================================
--- lyx-devel/branches/BRANCH_1_6_X/src/frontends/qt4/GuiPrefs.cpp      Mon Jan 
 3 18:21:22 2011        (r37081)
+++ lyx-devel/branches/BRANCH_1_6_X/src/frontends/qt4/GuiPrefs.cpp      Mon Jan 
 3 18:21:37 2011        (r37082)
@@ -970,29 +970,35 @@
        connect(displayGraphicsCB, SIGNAL(toggled(bool)), this, 
SIGNAL(changed()));
        connect(instantPreviewCO, SIGNAL(activated(int)), this, 
SIGNAL(changed()));
        connect(previewSizeSB, SIGNAL(valueChanged(double)), this, 
SIGNAL(changed()));
-       if (instantPreviewCO->currentIndex() == 0)
-               previewSizeSB->setEnabled(false);
-       else
-               previewSizeSB->setEnabled(true);
        connect(paragraphMarkerCB, SIGNAL(toggled(bool)), this, 
SIGNAL(changed())); 
 }
 
 
 void PrefDisplay::on_instantPreviewCO_currentIndexChanged(int index)
 {
-       if (index == 0)
-               previewSizeSB->setEnabled(false);
-       else
-               previewSizeSB->setEnabled(true);
+       previewSizeSB->setEnabled(index != 0);
+}
+
+
+void PrefDisplay::on_displayGraphicsCB_toggled(bool on)
+{
+       instantPreviewCO->setEnabled(on);
+       previewSizeSB->setEnabled(on && instantPreviewCO->currentIndex() > 0);
 }
 
 
 void PrefDisplay::apply(LyXRC & rc) const
 {
        switch (instantPreviewCO->currentIndex()) {
-               case 0: rc.preview = LyXRC::PREVIEW_OFF; break;
-               case 1: rc.preview = LyXRC::PREVIEW_NO_MATH; break;
-               case 2: rc.preview = LyXRC::PREVIEW_ON; break;
+               case 0:
+                        rc.preview = LyXRC::PREVIEW_OFF;
+                        break;
+               case 1:
+                       rc.preview = LyXRC::PREVIEW_NO_MATH;
+                       break;
+               case 2:
+                       rc.preview = LyXRC::PREVIEW_ON;
+                       break;
        }
 
        rc.display_graphics = displayGraphicsCB->isChecked();
@@ -1027,6 +1033,9 @@
        instantPreviewCO->setEnabled(rc.display_graphics);
        previewSizeSB->setValue(rc.preview_scale_factor);
        paragraphMarkerCB->setChecked(rc.paragraph_markers);
+       previewSizeSB->setEnabled(
+               rc.display_graphics
+               && rc.preview != LyXRC::PREVIEW_OFF);
 }
 
 

Modified: lyx-devel/branches/BRANCH_1_6_X/src/frontends/qt4/GuiPrefs.h
==============================================================================
--- lyx-devel/branches/BRANCH_1_6_X/src/frontends/qt4/GuiPrefs.h        Mon Jan 
 3 18:21:22 2011        (r37081)
+++ lyx-devel/branches/BRANCH_1_6_X/src/frontends/qt4/GuiPrefs.h        Mon Jan 
 3 18:21:37 2011        (r37082)
@@ -275,6 +275,7 @@
 
 private Q_SLOTS:
        void on_instantPreviewCO_currentIndexChanged(int);
+       void on_displayGraphicsCB_toggled(bool);
 };
 
 

Modified: lyx-devel/branches/BRANCH_1_6_X/status.16x
==============================================================================
--- lyx-devel/branches/BRANCH_1_6_X/status.16x  Mon Jan  3 18:21:22 2011        
(r37081)
+++ lyx-devel/branches/BRANCH_1_6_X/status.16x  Mon Jan  3 18:21:37 2011        
(r37082)
@@ -154,6 +154,9 @@
 - Wake up from waiting state after encountering a file parsing error
   (bug 6950).
 
+- Fix enabling/disabling of Instant Preview widgets in the preferences
+  dialog (bug 7190).
+
 
 * DOCUMENTATION AND LOCALIZATION
 

Reply via email to