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

change logic of eraser #580

Merged
merged 1 commit into from
Feb 2, 2024
Merged

change logic of eraser #580

merged 1 commit into from
Feb 2, 2024

Conversation

MrGumpIT
Copy link
Contributor

@MrGumpIT MrGumpIT commented Feb 1, 2024

The main difference is the change in how it's detected if a stroke should be erased. Now, a stroke is erased only if the square of the distance between the current pointer position and the last erased position is greater than or equal to the square of the eraser's stroke width. Moreover, if a point of the stroke is too close to the pointer position, that point is omitted, a new path is started, and the stroke is marked as "broken". If a stroke has been broken, then it is always modified, even if it has only one path.
It would be interesting to set the default size of the eraser equal to the size of the pen and see how it works, I have not been able to do it.
solve #461

Copy link

vercel bot commented Feb 1, 2024

@MrGumpIT is attempting to deploy a commit to the Linwood Team on Vercel.

A member of the Team first needs to authorize it.

@CodeDoctorDE
Copy link
Member

Thanks for fixing this. I will test it out tomorrow.
If you want you can join the discord (https://linwood.dev/discord) or matrix server (https://linwood.dev/matrix) and we can talk about the app further.

From the first view it looks good.

Copy link

vercel bot commented Feb 1, 2024

Deployment failed with the following error:

Too many requests - try again in 2 minutes (more than 60, code: "api-deployments-flood").

Copy link

vercel bot commented Feb 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
butterfly ✅ Ready (Inspect) Visit Preview Feb 1, 2024 9:35pm

@vercel vercel bot temporarily deployed to Preview – butterfly February 1, 2024 21:35 Inactive
@vercel vercel bot temporarily deployed to Preview – butterfly-docs February 1, 2024 21:36 Inactive
@CodeDoctorDE
Copy link
Member

Hi, I tried it and sometimes it leaves small points. I don't know if this is an issue here but it would be nice if we could fix this somehow.
Btw: If you write Closes: #461 instead of solve #461 it automatically closes the issue when merging it. It would be nice if you can join the matrix or discord server to talk

@MrGumpIT
Copy link
Contributor Author

MrGumpIT commented Feb 2, 2024

I tried to solve it in many ways but without success, it happens when you try to break a line, this is because we don't delete the point directly but we overwrite it. For now I think it's an acceptable compromise: you get fewer points than before and they can still be cancelled. I'll keep trying anyway. I joined the discord you should be able to see me

@CodeDoctorDE
Copy link
Member

Yeah,
I'm the same opinion, I will merge it.
What's your name on discord, I can't find your name on the Linwood discord

@CodeDoctorDE CodeDoctorDE merged commit 9c72e6b into LinwoodDev:develop Feb 2, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants