diff --git a/python/core/symbology-ng/qgsgraduatedsymbolrenderer.sip b/python/core/symbology-ng/qgsgraduatedsymbolrenderer.sip index 4f99ab69130..93c1c88fb36 100644 --- a/python/core/symbology-ng/qgsgraduatedsymbolrenderer.sip +++ b/python/core/symbology-ng/qgsgraduatedsymbolrenderer.sip @@ -197,6 +197,16 @@ class QgsGraduatedSymbolRenderer : QgsFeatureRenderer //! @note Added in 2.6 void calculateLabelPrecision( bool updateRanges = true ); + /** Creates a new graduated renderer. + * @param vlayer vector layer + * @param attrName attribute to classify + * @param classes number of classes + * @param mode classification mode + * @param symbol base symbol + * @param ramp color ramp for classes + * @param legendFormat + * @returns new QgsGraduatedSymbolRenderer object + */ static QgsGraduatedSymbolRenderer* createRenderer( QgsVectorLayer* vlayer, const QString& attrName, @@ -204,7 +214,6 @@ class QgsGraduatedSymbolRenderer : QgsFeatureRenderer Mode mode, QgsSymbol* symbol /Transfer/, QgsColorRamp* ramp /Transfer/, - bool inverted = false, const QgsRendererRangeLabelFormat& legendFormat = QgsRendererRangeLabelFormat() ); @@ -252,16 +261,11 @@ class QgsGraduatedSymbolRenderer : QgsFeatureRenderer */ void setSourceColorRamp( QgsColorRamp* ramp /Transfer/ ); - //! @note added in 2.1 - bool invertedColorRamp(); - void setInvertedColorRamp( bool inverted ); - /** Update the color ramp used. Also updates all symbols colors. * Doesn't alter current breaks. * @param ramp color ramp. Ownership is transferred to the renderer - * @param inverted set to true to invert ramp colors */ - void updateColorRamp( QgsColorRamp* ramp /Transfer/ = 0, bool inverted = false ); + void updateColorRamp( QgsColorRamp* ramp /Transfer/ = 0 ); /** Update all the symbols but leave breaks and colors. This method also sets the source * symbol for the renderer. diff --git a/src/core/symbology-ng/qgsgraduatedsymbolrenderer.cpp b/src/core/symbology-ng/qgsgraduatedsymbolrenderer.cpp index 33e9b023d22..bbf3b41782d 100644 --- a/src/core/symbology-ng/qgsgraduatedsymbolrenderer.cpp +++ b/src/core/symbology-ng/qgsgraduatedsymbolrenderer.cpp @@ -288,7 +288,6 @@ QgsGraduatedSymbolRenderer::QgsGraduatedSymbolRenderer( const QString& attrName, : QgsFeatureRenderer( QStringLiteral( "graduatedSymbol" ) ) , mAttrName( attrName ) , mMode( Custom ) - , mInvertedColorRamp( false ) , mGraduatedMethod( GraduatedColor ) , mAttrNum( -1 ) , mCounting( false ) @@ -501,7 +500,6 @@ QgsGraduatedSymbolRenderer* QgsGraduatedSymbolRenderer::clone() const if ( mSourceColorRamp.data() ) { r->setSourceColorRamp( mSourceColorRamp->clone() ); - r->setInvertedColorRamp( mInvertedColorRamp ); } r->setUsingSymbolLevels( usingSymbolLevels() ); r->setLabelFormat( labelFormat() ); @@ -790,7 +788,6 @@ QgsGraduatedSymbolRenderer* QgsGraduatedSymbolRenderer::createRenderer( Mode mode, QgsSymbol* symbol, QgsColorRamp* ramp, - bool inverted, const QgsRendererRangeLabelFormat& labelFormat ) { @@ -798,7 +795,6 @@ QgsGraduatedSymbolRenderer* QgsGraduatedSymbolRenderer::createRenderer( QgsGraduatedSymbolRenderer* r = new QgsGraduatedSymbolRenderer( attrName, ranges ); r->setSourceSymbol( symbol->clone() ); r->setSourceColorRamp( ramp->clone() ); - r->setInvertedColorRamp( inverted ); r->setMode( mode ); r->setLabelFormat( labelFormat ); r->updateClasses( vlayer, mode, classes ); @@ -916,7 +912,7 @@ void QgsGraduatedSymbolRenderer::updateClasses( QgsVectorLayer* vlayer, Mode mod QgsSymbol* newSymbol = mSourceSymbol ? mSourceSymbol->clone() : QgsSymbol::defaultSymbol( vlayer->geometryType() ); addClass( QgsRendererRange( lower, upper, newSymbol, label ) ); } - updateColorRamp( nullptr, mInvertedColorRamp ); + updateColorRamp( nullptr ); } QgsFeatureRenderer* QgsGraduatedSymbolRenderer::create( QDomElement& element ) @@ -985,9 +981,6 @@ QgsFeatureRenderer* QgsGraduatedSymbolRenderer::create( QDomElement& element ) if ( !sourceColorRampElem.isNull() && sourceColorRampElem.attribute( QStringLiteral( "name" ) ) == QLatin1String( "[source]" ) ) { r->setSourceColorRamp( QgsSymbolLayerUtils::loadColorRamp( sourceColorRampElem ) ); - QDomElement invertedColorRampElem = element.firstChildElement( QStringLiteral( "invertedcolorramp" ) ); - if ( !invertedColorRampElem.isNull() ) - r->setInvertedColorRamp( invertedColorRampElem.attribute( QStringLiteral( "value" ) ) == QLatin1String( "1" ) ); } // try to load mode @@ -1098,9 +1091,6 @@ QDomElement QgsGraduatedSymbolRenderer::save( QDomDocument& doc ) { QDomElement colorRampElem = QgsSymbolLayerUtils::saveColorRamp( QStringLiteral( "[source]" ), mSourceColorRamp.data(), doc ); rendererElem.appendChild( colorRampElem ); - QDomElement invertedElem = doc.createElement( QStringLiteral( "invertedcolorramp" ) ); - invertedElem.setAttribute( QStringLiteral( "value" ), mInvertedColorRamp ); - rendererElem.appendChild( invertedElem ); } // save mode @@ -1306,13 +1296,12 @@ void QgsGraduatedSymbolRenderer::setSymbolSizes( double minSize, double maxSize } } -void QgsGraduatedSymbolRenderer::updateColorRamp( QgsColorRamp *ramp, bool inverted ) +void QgsGraduatedSymbolRenderer::updateColorRamp( QgsColorRamp *ramp ) { int i = 0; if ( ramp ) { setSourceColorRamp( ramp ); - setInvertedColorRamp( inverted ); } if ( mSourceColorRamp ) @@ -1323,10 +1312,7 @@ void QgsGraduatedSymbolRenderer::updateColorRamp( QgsColorRamp *ramp, bool inver if ( symbol ) { double colorValue; - if ( inverted ) - colorValue = ( mRanges.count() > 1 ? static_cast< double >( mRanges.count() - i - 1 ) / ( mRanges.count() - 1 ) : 0 ); - else - colorValue = ( mRanges.count() > 1 ? static_cast< double >( i ) / ( mRanges.count() - 1 ) : 0 ); + colorValue = ( mRanges.count() > 1 ? static_cast< double >( i ) / ( mRanges.count() - 1 ) : 0 ); symbol->setColor( mSourceColorRamp->color( colorValue ) ); } updateRangeSymbol( i, symbol ); @@ -1441,7 +1427,7 @@ void QgsGraduatedSymbolRenderer::addBreak( double breakValue, bool updateSymbols switch ( mGraduatedMethod ) { case GraduatedColor: - updateColorRamp( mSourceColorRamp.data(), mInvertedColorRamp ); + updateColorRamp( mSourceColorRamp.data() ); break; case GraduatedSize: setSymbolSizes( minSymbolSize(), maxSymbolSize() ); diff --git a/src/core/symbology-ng/qgsgraduatedsymbolrenderer.h b/src/core/symbology-ng/qgsgraduatedsymbolrenderer.h index 66caf437d9c..37f9c36d72d 100644 --- a/src/core/symbology-ng/qgsgraduatedsymbolrenderer.h +++ b/src/core/symbology-ng/qgsgraduatedsymbolrenderer.h @@ -235,7 +235,6 @@ class CORE_EXPORT QgsGraduatedSymbolRenderer : public QgsFeatureRenderer * @param mode classification mode * @param symbol base symbol * @param ramp color ramp for classes - * @param inverted set to true to invert color ramp * @param legendFormat * @returns new QgsGraduatedSymbolRenderer object */ @@ -246,7 +245,6 @@ class CORE_EXPORT QgsGraduatedSymbolRenderer : public QgsFeatureRenderer Mode mode, QgsSymbol* symbol, QgsColorRamp* ramp, - bool inverted = false, const QgsRendererRangeLabelFormat& legendFormat = QgsRendererRangeLabelFormat() ); @@ -285,16 +283,11 @@ class CORE_EXPORT QgsGraduatedSymbolRenderer : public QgsFeatureRenderer */ void setSourceColorRamp( QgsColorRamp* ramp ); - //! @note added in 2.1 - bool invertedColorRamp() { return mInvertedColorRamp; } - void setInvertedColorRamp( bool inverted ) { mInvertedColorRamp = inverted; } - /** Update the color ramp used. Also updates all symbols colors. * Doesn't alter current breaks. * @param ramp color ramp. Ownership is transferred to the renderer - * @param inverted set to true to invert ramp colors */ - void updateColorRamp( QgsColorRamp* ramp = nullptr, bool inverted = false ); + void updateColorRamp( QgsColorRamp* ramp = nullptr ); /** Update all the symbols but leave breaks and colors. This method also sets the source * symbol for the renderer. @@ -343,7 +336,6 @@ class CORE_EXPORT QgsGraduatedSymbolRenderer : public QgsFeatureRenderer Mode mMode; QScopedPointer mSourceSymbol; QScopedPointer mSourceColorRamp; - bool mInvertedColorRamp; QgsRendererRangeLabelFormat mLabelFormat; QScopedPointer mExpression; diff --git a/src/gui/qgscolorrampbutton.cpp b/src/gui/qgscolorrampbutton.cpp index edbea2e9773..384965bdecc 100644 --- a/src/gui/qgscolorrampbutton.cpp +++ b/src/gui/qgscolorrampbutton.cpp @@ -226,7 +226,7 @@ QPixmap QgsColorRampButton::createMenuIcon( QgsColorRamp* colorramp ) void QgsColorRampButton::buttonClicked() { - if ( !isRandomColorRamp() ) + if ( !isRandomColorRamp() && !isNull() ) { showColorRampDialog(); } @@ -241,7 +241,7 @@ void QgsColorRampButton::prepareMenu() mMenu->clear(); QAction* invertAction = new QAction( tr( "Invert color ramp" ), this ); - invertAction->setEnabled( !isRandomColorRamp() ); + invertAction->setEnabled( !isNull() && !isRandomColorRamp() ); mMenu->addAction( invertAction ); connect( invertAction, &QAction::triggered, this, &QgsColorRampButton::invertColorRamp ); @@ -317,6 +317,7 @@ void QgsColorRampButton::prepareMenu() mMenu->addAction( newColorRampAction ); QAction* editColorRampAction = new QAction( tr( "Edit color ramp..." ), this ); + editColorRampAction->setEnabled( !isNull() && !isRandomColorRamp() ); connect( editColorRampAction, &QAction::triggered, this, &QgsColorRampButton::showColorRampDialog ); mMenu->addAction( editColorRampAction ); } diff --git a/src/gui/symbology-ng/qgscategorizedsymbolrendererwidget.cpp b/src/gui/symbology-ng/qgscategorizedsymbolrendererwidget.cpp index e544ee29d1d..e274286c513 100644 --- a/src/gui/symbology-ng/qgscategorizedsymbolrendererwidget.cpp +++ b/src/gui/symbology-ng/qgscategorizedsymbolrendererwidget.cpp @@ -421,6 +421,12 @@ QgsCategorizedSymbolRendererWidget::QgsCategorizedSymbolRendererWidget( QgsVecto { btnColorRamp->setColorRampFromName( defaultColorRamp ); } + else + { + QgsColorRamp* ramp = new QgsGradientColorRamp( QColor( 255, 255, 255 ), QColor( 255, 0, 0 ) ); + btnColorRamp->setColorRamp( ramp ); + delete ramp; + } mCategorizedSymbol = QgsSymbol::defaultSymbol( mLayer->geometryType() ); diff --git a/src/gui/symbology-ng/qgsgraduatedsymbolrendererwidget.cpp b/src/gui/symbology-ng/qgsgraduatedsymbolrendererwidget.cpp index 4ee3926bd35..c8248956a1e 100644 --- a/src/gui/symbology-ng/qgsgraduatedsymbolrendererwidget.cpp +++ b/src/gui/symbology-ng/qgsgraduatedsymbolrendererwidget.cpp @@ -18,6 +18,7 @@ #include "qgssymbol.h" #include "qgssymbollayerutils.h" #include "qgscolorramp.h" +#include "qgscolorrampbutton.h" #include "qgsstyle.h" #include "qgsvectorlayer.h" @@ -446,18 +447,22 @@ QgsGraduatedSymbolRendererWidget::QgsGraduatedSymbolRendererWidget( QgsVectorLay mSizeUnitWidget->setUnits( QgsUnitTypes::RenderUnitList() << QgsUnitTypes::RenderMillimeters << QgsUnitTypes::RenderMapUnits << QgsUnitTypes::RenderPixels ); - cboGraduatedColorRamp->populate( mStyle ); - spinPrecision->setMinimum( QgsRendererRangeLabelFormat::MinPrecision ); spinPrecision->setMaximum( QgsRendererRangeLabelFormat::MaxPrecision ); + btnColorRamp->setShowRandomColorRamp( true ); + // set project default color ramp QString defaultColorRamp = QgsProject::instance()->readEntry( QStringLiteral( "DefaultStyles" ), QStringLiteral( "/ColorRamp" ), QLatin1String( "" ) ); if ( defaultColorRamp != QLatin1String( "" ) ) { - int index = cboGraduatedColorRamp->findText( defaultColorRamp, Qt::MatchCaseSensitive ); - if ( index >= 0 ) - cboGraduatedColorRamp->setCurrentIndex( index ); + btnColorRamp->setColorRampFromName( defaultColorRamp ); + } + else + { + QgsColorRamp* ramp = new QgsGradientColorRamp( QColor( 255, 255, 255 ), QColor( 255, 0, 0 ) ); + btnColorRamp->setColorRamp( ramp ); + delete ramp; } @@ -543,10 +548,7 @@ void QgsGraduatedSymbolRendererWidget::connectUpdateHandlers() { connect( spinGraduatedClasses, SIGNAL( valueChanged( int ) ), this, SLOT( classifyGraduated() ) ); connect( cboGraduatedMode, SIGNAL( currentIndexChanged( int ) ), this, SLOT( classifyGraduated() ) ); - connect( cboGraduatedColorRamp, SIGNAL( currentIndexChanged( int ) ), this, SLOT( reapplyColorRamp() ) ); - connect( cboGraduatedColorRamp, SIGNAL( sourceRampEdited() ), this, SLOT( reapplyColorRamp() ) ); - connect( mButtonEditRamp, SIGNAL( clicked() ), cboGraduatedColorRamp, SLOT( editSourceRamp() ) ); - connect( cbxInvertedColorRamp, SIGNAL( toggled( bool ) ), this, SLOT( reapplyColorRamp() ) ); + connect( btnColorRamp, &QgsColorRampButton::colorRampChanged, this, &QgsGraduatedSymbolRendererWidget::reapplyColorRamp ); connect( spinPrecision, SIGNAL( valueChanged( int ) ), this, SLOT( labelFormatChanged() ) ); connect( cbxTrimTrailingZeroes, SIGNAL( toggled( bool ) ), this, SLOT( labelFormatChanged() ) ); connect( txtLegendFormat, SIGNAL( textChanged( QString ) ), this, SLOT( labelFormatChanged() ) ); @@ -563,10 +565,7 @@ void QgsGraduatedSymbolRendererWidget::disconnectUpdateHandlers() { disconnect( spinGraduatedClasses, SIGNAL( valueChanged( int ) ), this, SLOT( classifyGraduated() ) ); disconnect( cboGraduatedMode, SIGNAL( currentIndexChanged( int ) ), this, SLOT( classifyGraduated() ) ); - disconnect( cboGraduatedColorRamp, SIGNAL( currentIndexChanged( int ) ), this, SLOT( reapplyColorRamp() ) ); - disconnect( cboGraduatedColorRamp, SIGNAL( sourceRampEdited() ), this, SLOT( reapplyColorRamp() ) ); - disconnect( mButtonEditRamp, SIGNAL( clicked() ), cboGraduatedColorRamp, SLOT( editSourceRamp() ) ); - disconnect( cbxInvertedColorRamp, SIGNAL( toggled( bool ) ), this, SLOT( reapplyColorRamp() ) ); + disconnect( btnColorRamp, &QgsColorRampButton::colorRampChanged, this, &QgsGraduatedSymbolRendererWidget::reapplyColorRamp ); disconnect( spinPrecision, SIGNAL( valueChanged( int ) ), this, SLOT( labelFormatChanged() ) ); disconnect( cbxTrimTrailingZeroes, SIGNAL( toggled( bool ) ), this, SLOT( labelFormatChanged() ) ); disconnect( txtLegendFormat, SIGNAL( textChanged( QString ) ), this, SLOT( labelFormatChanged() ) ); @@ -622,8 +621,9 @@ void QgsGraduatedSymbolRendererWidget::updateUiFromRenderer( bool updateCount ) { methodComboBox->setCurrentIndex( 0 ); if ( mRenderer->sourceColorRamp() ) - cboGraduatedColorRamp->setSourceColorRamp( mRenderer->sourceColorRamp() ); - cbxInvertedColorRamp->setChecked( mRenderer->invertedColorRamp() ); + { + btnColorRamp->setColorRamp( mRenderer->sourceColorRamp() ); + } } else { @@ -634,7 +634,7 @@ void QgsGraduatedSymbolRendererWidget::updateUiFromRenderer( bool updateCount ) maxSizeSpinBox->setValue( mRenderer->maxSymbolSize() ); } } - mMethodStackedWidget->setCurrentIndex( methodComboBox->currentIndex() ); + toggleMethodWidgets( methodComboBox->currentIndex() ); methodComboBox->blockSignals( false ); QgsRendererRangeLabelFormat labelFormat = mRenderer->labelFormat(); @@ -659,18 +659,15 @@ void QgsGraduatedSymbolRendererWidget::graduatedColumnChanged( const QString& fi void QgsGraduatedSymbolRendererWidget::on_methodComboBox_currentIndexChanged( int idx ) { - mMethodStackedWidget->setCurrentIndex( idx ); + toggleMethodWidgets( idx ); if ( idx == 0 ) { mRenderer->setGraduatedMethod( QgsGraduatedSymbolRenderer::GraduatedColor ); - QgsColorRamp* ramp = cboGraduatedColorRamp->currentColorRamp(); + QgsColorRamp* ramp = btnColorRamp->colorRamp(); if ( !ramp ) { - if ( cboGraduatedColorRamp->count() == 0 ) - QMessageBox::critical( this, tr( "Error" ), tr( "There are no available color ramps. You can add them in Style Manager." ) ); - else - QMessageBox::critical( this, tr( "Error" ), tr( "The selected color ramp is not available." ) ); + QMessageBox::critical( this, tr( "Error" ), tr( "No color ramp defined." ) ); return; } mRenderer->setSourceColorRamp( ramp ); @@ -678,11 +675,43 @@ void QgsGraduatedSymbolRendererWidget::on_methodComboBox_currentIndexChanged( in } else { + lblColorRamp->setVisible( false ); + btnColorRamp->setVisible( false ); + lblSize->setVisible( true ); + minSizeSpinBox->setVisible( true ); + lblSize->setVisible( true ); + maxSizeSpinBox->setVisible( true ); + mSizeUnitWidget->setVisible( true ); + mRenderer->setGraduatedMethod( QgsGraduatedSymbolRenderer::GraduatedSize ); reapplySizes(); } } +void QgsGraduatedSymbolRendererWidget::toggleMethodWidgets( int idx ) +{ + if ( idx == 0 ) + { + lblColorRamp->setVisible( true ); + btnColorRamp->setVisible( true ); + lblSize->setVisible( false ); + minSizeSpinBox->setVisible( false ); + lblSizeTo->setVisible( false ); + maxSizeSpinBox->setVisible( false ); + mSizeUnitWidget->setVisible( false ); + } + else + { + lblColorRamp->setVisible( false ); + btnColorRamp->setVisible( false ); + lblSize->setVisible( true ); + minSizeSpinBox->setVisible( true ); + lblSizeTo->setVisible( true ); + maxSizeSpinBox->setVisible( true ); + mSizeUnitWidget->setVisible( true ); + } +} + void QgsGraduatedSymbolRendererWidget::refreshRanges( bool reset ) { if ( !mModel ) @@ -748,13 +777,10 @@ void QgsGraduatedSymbolRendererWidget::classifyGraduated() int nclasses = spinGraduatedClasses->value(); - QScopedPointer ramp( cboGraduatedColorRamp->currentColorRamp() ); + QScopedPointer ramp( btnColorRamp->colorRamp() ); if ( !ramp ) { - if ( cboGraduatedColorRamp->count() == 0 ) - QMessageBox::critical( this, tr( "Error" ), tr( "There are no available color ramps. You can add them in Style Manager." ) ); - else - QMessageBox::critical( this, tr( "Error" ), tr( "The selected color ramp is not available." ) ); + QMessageBox::critical( this, tr( "Error" ), tr( "No color ramp defined." ) ); return; } @@ -787,10 +813,7 @@ void QgsGraduatedSymbolRendererWidget::classifyGraduated() { if ( !ramp ) { - if ( cboGraduatedColorRamp->count() == 0 ) - QMessageBox::critical( this, tr( "Error" ), tr( "There are no available color ramps. You can add them in Style Manager." ) ); - else - QMessageBox::critical( this, tr( "Error" ), tr( "The selected color ramp is not available." ) ); + QMessageBox::critical( this, tr( "Error" ), tr( "No color ramp defined." ) ); return; } mRenderer->setSourceColorRamp( ramp.take() ); @@ -815,11 +838,11 @@ void QgsGraduatedSymbolRendererWidget::classifyGraduated() void QgsGraduatedSymbolRendererWidget::reapplyColorRamp() { - QScopedPointer< QgsColorRamp > ramp( cboGraduatedColorRamp->currentColorRamp() ); + QScopedPointer< QgsColorRamp > ramp( btnColorRamp->colorRamp() ); if ( !ramp ) return; - mRenderer->updateColorRamp( ramp.take(), cbxInvertedColorRamp->isChecked() ); + mRenderer->updateColorRamp( ramp.take() ); mRenderer->updateSymbols( mGraduatedSymbol ); refreshSymbolView(); } diff --git a/src/gui/symbology-ng/qgsgraduatedsymbolrendererwidget.h b/src/gui/symbology-ng/qgsgraduatedsymbolrendererwidget.h index e2f95eafb7d..ab6ef8db9d8 100644 --- a/src/gui/symbology-ng/qgsgraduatedsymbolrendererwidget.h +++ b/src/gui/symbology-ng/qgsgraduatedsymbolrendererwidget.h @@ -122,6 +122,7 @@ class GUI_EXPORT QgsGraduatedSymbolRendererWidget : public QgsRendererWidget, pr private slots: void cleanUpSymbolSelector( QgsPanelWidget* container ); void updateSymbolsFromWidget(); + void toggleMethodWidgets( int idx ); protected: void updateUiFromRenderer( bool updateCount = true ); diff --git a/src/ui/qgsgraduatedsymbolrendererv2widget.ui b/src/ui/qgsgraduatedsymbolrendererv2widget.ui index cc613902609..6171c004ec0 100644 --- a/src/ui/qgsgraduatedsymbolrendererv2widget.ui +++ b/src/ui/qgsgraduatedsymbolrendererv2widget.ui @@ -148,137 +148,106 @@ Negative rounds to powers of 10 - - - - - 0 - 0 - - - - - 16777215 - 50 - - - - 0 - - - - - 0 - - - + + + Color ramp - cboGraduatedColorRamp + btnColorRamp - - - - - - - Edit - - - - - - - Invert - - - - - - - Qt::Horizontal - - - - 40 - 20 - - - - - - - - - - 0 - - - - - 6 - - - 999999999.000000000000000 - - - 0.200000000000000 - - - 1.000000000000000 - - - false - - - - - - - Size from - - - cboGraduatedColorRamp - - - - - - - 6 - - - 999999999.000000000000000 - - - 0.200000000000000 - - - 10.000000000000000 - - - false - - - - - + + - + 0 0 - - to + + + 120 + 0 + + + + + 16777215 + 16777215 + - + + + + + Size from + + + minSizeSpinBox + + + + + + + + + 6 + + + 999999999.000000000000000 + + + 0.200000000000000 + + + 1.000000000000000 + + + false + + + + + + + + 0 + 0 + + + + to + + + + + + + 6 + + + 999999999.000000000000000 + + + 0.200000000000000 + + + 10.000000000000000 + + + false + + + + + + @@ -291,11 +260,9 @@ Negative rounds to powers of 10 - - - - - + + + 0 @@ -522,9 +489,10 @@ Negative rounds to powers of 10 1 - QgsColorRampComboBox - QComboBox -
qgscolorrampcombobox.h
+ QgsColorRampButton + QToolButton +
qgscolorrampbutton.h
+ 1
QgsDoubleSpinBox @@ -545,9 +513,7 @@ Negative rounds to powers of 10 - cboGraduatedColorRamp - mButtonEditRamp - cbxInvertedColorRamp + btnColorRamp cboGraduatedMode spinGraduatedClasses btnGraduatedClassify diff --git a/tests/src/python/test_qgsgraduatedsymbolrenderer.py b/tests/src/python/test_qgsgraduatedsymbolrenderer.py index bbc97b8c5cf..9e5f3646069 100644 --- a/tests/src/python/test_qgsgraduatedsymbolrenderer.py +++ b/tests/src/python/test_qgsgraduatedsymbolrenderer.py @@ -142,7 +142,6 @@ def dumpGraduatedRenderer(r): else: rstr = rstr + symbol.dump() + ':' rstr = rstr + dumpColorRamp(r.sourceColorRamp()) - rstr = rstr + str(r.invertedColorRamp()) + ':' rstr = rstr + dumpRangeList(r.ranges()) return rstr @@ -305,13 +304,6 @@ class TestQgsGraduatedSymbolRenderer(unittest.TestCase): dumpColorRamp(renderer.sourceColorRamp()), "Get/set renderer color ramp") - renderer.setInvertedColorRamp(True) - self.assertTrue(renderer.invertedColorRamp(), - "Get/set renderer inverted color ramp") - renderer.setInvertedColorRamp(False) - self.assertFalse(renderer.invertedColorRamp(), - "Get/set renderer inverted color ramp") - renderer.setSourceColorRamp(ramp) self.assertEqual( dumpColorRamp(ramp),