We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
2.4.2
http://jsfiddle.net/jpkoskela/zLh2d6y7/
All browsers.
Add a 1x1 rectangle with 0.1 stroke width to canvas. Zoom in.
Border should be rendered for the 1x1 rectangle.
Rendering of border is optimized away.
rect.class.js
_render: function(ctx) { // optimize 1x1 case (used in spray brush) if (this.width === 1 && this.height === 1) { ctx.fillRect(-0.5, -0.5, 1, 1); return; } ```
The text was updated successfully, but these errors were encountered:
you are right. This optimization was for the spray brush. i think since we have caching we should remove it. Do you want to do it with a PR?
Sorry, something went wrong.
Successfully merging a pull request may close this issue.
Version
2.4.2
Test Case
http://jsfiddle.net/jpkoskela/zLh2d6y7/
Information about environment
All browsers.
Steps to reproduce
Add a 1x1 rectangle with 0.1 stroke width to canvas. Zoom in.
Expected Behavior
Border should be rendered for the 1x1 rectangle.
Actual Behavior
Rendering of border is optimized away.
Source of Problem
rect.class.js
The text was updated successfully, but these errors were encountered: