Skip to content

.load() documentation unclear about applying to multiple elements #1125

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

Closed
Chealer opened this issue Feb 6, 2019 · 2 comments
Closed

.load() documentation unclear about applying to multiple elements #1125

Chealer opened this issue Feb 6, 2019 · 2 comments

Comments

@Chealer
Copy link
Contributor

Chealer commented Feb 6, 2019

The Description in .load()'s documentation ("Load data from the server and place the returned HTML into the matched element") implies the jQuery object must contain a single element.

While none of the examples of the page contradict that, the callback's description implies multiple elements are possible:

The callback is fired once for each element in the jQuery collection, and this is set to each DOM element in turn.

@dmethvin
Copy link
Member

dmethvin commented Feb 8, 2019

Yes I agree that "element" should be "elements". I checked the code to be sure. Would you like to make a pull request for that?

@Chealer
Copy link
Contributor Author

Chealer commented Feb 9, 2019

Is that a request for a request? ;-)
Thank you for checking Dave. I believe a junior with the appropriate permission could have solved this with less overhead, but I still committed a fix.

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

No branches or pull requests

2 participants