From a6e1b9eed2bc3ff938c9072ee621ac77d227b3fc Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Wed, 20 Jul 2022 22:02:56 +0200 Subject: [PATCH] LibWeb: Snap transformed stacking contexts to pixel grid Make sure that we're painting them at an integer pixel position, to avoid smearing. --- Userland/Libraries/LibWeb/Painting/StackingContext.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/Userland/Libraries/LibWeb/Painting/StackingContext.cpp b/Userland/Libraries/LibWeb/Painting/StackingContext.cpp index ab7adf366a..dbdb091b4b 100644 --- a/Userland/Libraries/LibWeb/Painting/StackingContext.cpp +++ b/Userland/Libraries/LibWeb/Painting/StackingContext.cpp @@ -295,7 +295,12 @@ void StackingContext::paint(PaintContext& context) const auto transformed_destination_rect = affine_transform.map(source_rect).translated(transform_origin); source_rect.translate_by(transform_origin); - context.painter().draw_scaled_bitmap(transformed_destination_rect.to_rounded(), *bitmap, source_rect, opacity, Gfx::Painter::ScalingMode::BilinearBlend); + + // NOTE: If the destination and source rects are the same size, we round the source rect to ensure that it's pixel-aligned. + if (transformed_destination_rect.size() == source_rect.size()) + context.painter().draw_scaled_bitmap(transformed_destination_rect.to_rounded(), *bitmap, source_rect.to_rounded(), opacity); + else + context.painter().draw_scaled_bitmap(transformed_destination_rect.to_rounded(), *bitmap, source_rect, opacity, Gfx::Painter::ScalingMode::BilinearBlend); } else { paint_internal(context); }