-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
Normalized Set format and clarified set vs array #3342
Conversation
I was reading the docs on this today and was very confused by how and why the docs were doing things. After spending some time testing the examples, I realized that they simply were not consistent in how they were written and were type casting in someplace for no reason. These changes settle on representing the `Set` according to the standard unless explicitly showing how to convert from a set to an array. This should help devs not already familiar with Set to better understand it and its uniqueness at first glance and reduce confusion like I had by utilizing the standards for Set when applicable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much @RIAEvangelist
Please see the review comments
files/en-us/web/javascript/reference/global_objects/set/index.html
Outdated
Show resolved
Hide resolved
files/en-us/web/javascript/reference/global_objects/set/index.html
Outdated
Show resolved
Hide resolved
files/en-us/web/javascript/reference/global_objects/set/index.html
Outdated
Show resolved
Hide resolved
files/en-us/web/javascript/reference/global_objects/set/index.html
Outdated
Show resolved
Hide resolved
Apologies, I had to go to Florida for a funeral. Will review the suggested edits. |
…html Co-authored-by: Ruth John <Rumyra@users.noreply.github.com>
…html Co-authored-by: Ruth John <Rumyra@users.noreply.github.com>
…html Co-authored-by: Ruth John <Rumyra@users.noreply.github.com>
…html added 'Set' to the returns comment for the Set object to increase accuracy. Co-authored-by: Ruth John <Rumyra@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merged all suggestions in, I see why they are relevant and useful. Thanks!
Thank you so much @RIAEvangelist 🎉 |
I was reading the docs on this today and was very confused by how and why the docs were doing things. After spending some time testing the examples, I realized that they simply were not consistent in how they were written and were type casting in someplace for no reason. These changes settle on representing the
Set
according to the standard unless explicitly showing how to convert from a set to an array.This should help devs not already familiar with Set to better understand it and its uniqueness at first glance and reduce confusion like I had by utilizing the standards for Set when applicable
Checklist — To help your pull request get merged faster, please do the following: