Bug fix: Handling of alpha pixels when committing selection

Updated FinishSelection in Document.cs to use DrawWithOperator to force the use of Operator.Source when committing the selection layer. Added overload of DrawWithOperator that supplies ImageSurface implicitly. (#840)
PintaProject · Jun 14, 2024 · 2a1868d · 2a1868d
1 parent 84d5518
commit 2a1868d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/Pinta.Core/Classes/Document.cs b/Pinta.Core/Classes/Document.cs
@@ -214,7 +214,7 @@ public void FinishSelection ()
 
 		Context g = new (Layers.CurrentUserLayer.Surface);
 		selection.Clip (g);
-		layer.Draw (g);
+		layer.DrawWithOperator (g, Operator.Source, opacity: 1.0, transform: true);
 
 		Layers.DestroySelectionLayer ();
 		Workspace.Invalidate ();

diff --git a/Pinta.Core/Classes/Layer.cs b/Pinta.Core/Classes/Layer.cs
@@ -146,6 +146,11 @@ public void Draw (
 		ctx.Restore ();
 	}
 
+	public void DrawWithOperator (Context ctx, Operator op, double opacity = 1.0, bool transform = true)
+	{
+		DrawWithOperator (ctx, Surface, op, opacity, transform);
+	}
+
 	public void DrawWithOperator (
 		Context ctx,
 		ImageSurface surface,