Skip to content
New issue

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

Brush smoothing #28

Closed
lonelymaw opened this issue Jun 2, 2020 · 1 comment · Fixed by #35
Closed

Brush smoothing #28

lonelymaw opened this issue Jun 2, 2020 · 1 comment · Fixed by #35

Comments

@lonelymaw
Copy link

Current behavior

Currently, the circular brush used for both painting and blurring seems to be disjointed, making fast strokes across the canvas discontinuous and leaving gaps that may leave sensitive information visible.

Suggested behavior

If possible, smoothing out the brush by repeating it more frequently as the user draws could help mitigate this.

@bMacSwigg
Copy link
Contributor

I can work on an improvement to this. My proposal is:

  • currently, on every mousemove event we draw a circle at the new mouse position
  • instead, we should record the previous position, and interpolate between the previous position and the new position

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants