Restore some previous rendering behaviour

qgis · Nov 7, 2021 · ac6bbd6 · ac6bbd6
1 parent ce50674
commit ac6bbd6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/core/raster/qgsrasterlayerrenderer.cpp b/src/core/raster/qgsrasterlayerrenderer.cpp
@@ -337,6 +337,9 @@ bool QgsRasterLayerRenderer::render()
     projector->setCrs( mRasterViewPort->mSrcCRS, mRasterViewPort->mDestCRS, mRasterViewPort->mTransformContext );
   }
 
+  // important -- disable SmoothPixmapTransform for raster layer renders. We want individual pixels to be clearly defined!
+  renderContext()->painter()->setRenderHint( QPainter::SmoothPixmapTransform, false );
+
   // Drawer to pipe?
   QgsRasterIterator iterator( mPipe->last() );
   QgsRasterDrawer drawer( &iterator, renderContext()->dpiTarget() );

diff --git a/src/core/textrenderer/qgstextrenderer.cpp b/src/core/textrenderer/qgstextrenderer.cpp
@@ -1233,8 +1233,10 @@ void QgsTextRenderer::drawShadow( QgsRenderContext &context, const QgsTextRender
                    -offsetDist * std::sin( angleRad + M_PI_2 ) );
 
   p->save();
-  p->setRenderHint( QPainter::SmoothPixmapTransform );
   context.setPainterFlagsUsingContext( p );
+  // this was historically ALWAYS set for text renderer. We may want to consider getting it to respect the
+  // corresponding flag in the render context instead...
+  p->setRenderHint( QPainter::SmoothPixmapTransform );
   if ( context.useAdvancedEffects() )
   {
     p->setCompositionMode( shadow.blendMode() );