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

feat(fabric.util): Add a function to work with path measurements #6525

Merged
merged 8 commits into from
Aug 23, 2020

Conversation

asturur
Copy link
Member

@asturur asturur commented Aug 16, 2020

This function ( currently untested ) should provide us a way to measure paths.

src/util/path.js Outdated
return (1 - t) * (1 - t) * (1 - t);
}

function getPointOnCubicBezierIterator(P1x, P1y, P2x, P2y, P3x, P3y, P4x, P4y) {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to make those lowercase.

src/util/path.js Outdated
@@ -396,39 +396,192 @@
return destinationPath;
};

/**
* Calc lenght from point x1,y1 to x2,y2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Length please

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i spelled it good almost everywhere, after various checks and corrections 😂

@asturur asturur merged commit 0fbf7ad into master Aug 23, 2020
@asturur asturur mentioned this pull request Aug 23, 2020
shanicerae pushed a commit to shanicerae/fabric.js that referenced this pull request Jan 16, 2021
@asturur asturur deleted the measure-path branch January 16, 2021 23:03
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 this pull request may close these issues.

2 participants