-
Notifications
You must be signed in to change notification settings - Fork 138
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
Missing documentation for fabric.util.object.clone #309
Comments
hi @Serene47 you can look at the fabricjs repo and a jsdoc there if missing |
hi @asturur, Yeah I am fine with js doc, But my point is people who are new with fabric js, assume Some posts justifying my point
I also ran into that kind of problem and had to put some effort to find the repo and look into it. I assume if we provide documentation on the official page, people are more likely to read that. |
I have nothing against an explanation of that jsdoc on the website. But how would you frame it? This is what we say in the fabricJS docs for extend:
and this is what we say for clone:
Now the main issue is that the clone jsdoc needs to be updated with the content of the extend one. |
Hi, thanks for your suggestion I have added pull requests in fabricjs/fabric.js and fabricjs/fabricjs.com |
Please add documentation for the method
clone(object,deep)
in file/docs/fabric.util.object.html
.The source code for the function is in
https://github.com/fabricjs/fabric.js/blob/master/src/util/lang_object.js
The absence of the documentation misleads people to think that this function is for cloning fabric objects.
Feel free to assign me the task, if you are busy with other tasks.
The text was updated successfully, but these errors were encountered: