Merge pull request #4756 from TakumaKira/main

Allow drawing point with large scale and small strokeWeight.
processing · Mar 13, 2021 · 39b4fd7 · 39b4fd7
2 parents 2fdd4c9 + 2dec04a
commit 39b4fd7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/src/core/p5.Renderer2D.js b/src/core/p5.Renderer2D.js
@@ -568,17 +568,11 @@ p5.Renderer2D.prototype.point = function(x, y) {
   }
   const s = this._getStroke();
   const f = this._getFill();
-  x = Math.round(x);
-  y = Math.round(y);
   // swapping fill color to stroke and back after for correct point rendering
   this._setFill(s);
-  if (ctx.lineWidth > 1) {
-    ctx.beginPath();
-    ctx.arc(x, y, ctx.lineWidth / 2, 0, constants.TWO_PI, false);
-    ctx.fill();
-  } else {
-    ctx.fillRect(x, y, 1, 1);
-  }
+  ctx.beginPath();
+  ctx.arc(x, y, ctx.lineWidth / 2, 0, constants.TWO_PI, false);
+  ctx.fill();
   this._setFill(f);
 };
 

diff --git a/test/unit/core/structure.js b/test/unit/core/structure.js
@@ -221,7 +221,7 @@ suite('Structure', function() {
           myp5.background(0);
           myp5.stroke(255);
           myp5.point(10, 10);
-          if (myp5.get(10, 10)[0] !== 255) {
+          if (myp5.get(10, 10)[0] === 0) {
             reject(new Error("Drawing matrix doesn't appear to be reset"));
           }
           myp5.rotate(10);