Only draw each rectangle outline pixel once

python-pillow · Jan 5, 2021 · e97c26e · e97c26e
1 parent d374015
commit e97c26e
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 2 deletions.
diff --git a/Tests/images/imagedraw_rectangle_translucent_outline.png b/Tests/images/imagedraw_rectangle_translucent_outline.png
diff --git a/Tests/test_imagedraw.py b/Tests/test_imagedraw.py
@@ -699,6 +699,21 @@ def test_rectangle_I16():
     )
 
 
+def test_rectangle_translucent_outline():
+    # Arrange
+    im = Image.new("RGB", (W, H))
+    draw = ImageDraw.Draw(im, "RGBA")
+
+    # Act
+    draw.rectangle(BBOX1, fill="black", outline=(0, 255, 0, 127), width=5)
+
+    # Assert
+    with Image.open(
+        "Tests/images/imagedraw_rectangle_translucent_outline.png"
+    ) as expected:
+        assert_image_equal(im, expected)
+
+
 def test_floodfill():
     red = ImageColor.getrgb("red")
 

diff --git a/src/libImaging/Draw.c b/src/libImaging/Draw.c
@@ -724,8 +724,8 @@ ImagingDrawRectangle(
         for (i = 0; i < width; i++) {
             draw->hline(im, x0, y0 + i, x1, ink);
             draw->hline(im, x0, y1 - i, x1, ink);
-            draw->line(im, x1 - i, y0, x1 - i, y1, ink);
-            draw->line(im, x0 + i, y1, x0 + i, y0, ink);
+            draw->line(im, x1 - i, y0 + width, x1 - i, y1 - width + 1, ink);
+            draw->line(im, x0 + i, y0 + width, x0 + i, y1 - width + 1, ink);
         }
     }