[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/work/kst/portto4/kst/src/widgets
From:       Barth Netterfield <netterfield () astro ! utoronto ! ca>
Date:       2013-01-31 17:56:28
Message-ID: 20130131175628.81DD3AC86C () svn ! kde ! org
[Download RAW message or body]

SVN commit 1336619 by netterfield:

BUG: 313847

Respect dialog defaults and curve properties for point density.



 M  +2 -11     curveappearance.cpp  


--- branches/work/kst/portto4/kst/src/widgets/curveappearance.cpp #1336618:1336619
@@ -130,35 +130,26 @@
   bool enable;
 
   // Use an intermediate boolean to avoid having to use setEnabled(true) and \
                setEnabled(false)
-  // Leave text labels enabled for now, it should be enough to disable the widgets
+  enable = showLines() || showBars();
 
   // Line appearance options common to lines and bars
-  enable = showLines() || showBars();
-  // _color->setEnabled(enable); Leave color enabled, even if using only points \
since they also have a color  _comboLineStyle->setEnabled(enable);
-  //  _textLabelWeight->setEnabled(enable);
-  // _spinBoxLineWidth->setEnabled(enable); Leave lineWidth enabled, even if using \
only points since they also use this setting +
   // Option specific to bars
   enable = showBars();
   _barFillColor->setEnabled(enable);
-  //  _textLabelBarStyle->setEnabled(enable);
 
   // Now point options
   enable = showPoints();
   _comboPointSymbol->setEnabled(enable);
-  //  _textLabelPointStyle->setEnabled(enable);
-  // Set point density to all by default
-  _comboPointDensity->setCurrentIndex(0); // 0 is "All"
   // and disable widget if not using lines, as using only points and not plotting \
all of them sounds weird  enable = enable && showLines();
   _comboPointDensity->setEnabled(enable);
-  //  _textLabelPointDensity->setEnabled(enable);
 
   // Heads
   enable = showHead();
   _comboHeadSymbol->setEnabled(enable);
   _headColor->setEnabled(enable);
-  //  _textLabelHeadStyle->setEnabled(enable);
 
 }
 


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic