Tweak default symbol for overviews

qgis · Nov 7, 2017 · 332a69c · 332a69c
1 parent aa2b73d
commit 332a69c
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 2 deletions.
diff --git a/src/core/layout/qgslayoutitemmapoverview.cpp b/src/core/layout/qgslayoutitemmapoverview.cpp
@@ -38,11 +38,10 @@ QgsLayoutItemMapOverview::QgsLayoutItemMapOverview( const QString &name, QgsLayo
 void QgsLayoutItemMapOverview::createDefaultFrameSymbol()
 {
   QgsStringMap properties;
-  properties.insert( QStringLiteral( "color" ), QStringLiteral( "255,0,0,255" ) );
+  properties.insert( QStringLiteral( "color" ), QStringLiteral( "255,0,0,75" ) );
   properties.insert( QStringLiteral( "style" ), QStringLiteral( "solid" ) );
   properties.insert( QStringLiteral( "style_border" ), QStringLiteral( "no" ) );
   mFrameSymbol.reset( QgsFillSymbol::createSimple( properties ) );
-  mFrameSymbol->setOpacity( 0.3 );
 }
 
 void QgsLayoutItemMapOverview::draw( QPainter *painter )

diff --git a/tests/src/python/test_qgslayoutmap.py b/tests/src/python/test_qgslayoutmap.py
@@ -84,6 +84,7 @@ def testOverviewMap(self):
         overviewMap.setExtent(myRectangle2)
         overviewMap.overview().setFrameMap(self.map)
         checker = QgsLayoutChecker('composermap_overview', self.layout)
+        checker.setColorTolerance(6)
         checker.setControlPathPrefix("composer_mapoverview")
         myTestResult, myMessage = checker.testLayout()
         self.layout.removeLayoutItem(overviewMap)

diff --git a/...apoverview/expected_composermap_overview/expected_composermap_overview_mask.png b/...apoverview/expected_composermap_overview/expected_composermap_overview_mask.png
diff --git a/...d_composermap_overview_blending/expected_composermap_overview_blending_mask.png b/...d_composermap_overview_blending/expected_composermap_overview_blending_mask.png
diff --git a/...ected_composermap_overview_invert/expected_composermap_overview_invert_mask.png b/...ected_composermap_overview_invert/expected_composermap_overview_invert_mask.png