Fix documentation for .val()

[aried3r]

See:
#828
jquery/jquery#2562
timmywil/jquery@4319c27

Fixes #828.

[jquerybot]

Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA).

📝 Please visit http://contribute.jquery.org/CLA/ to sign.

After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know.

---

If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check.

[aried3r]

Should I leave the trailing whitespace as is or is it fine?

[AurelioDeRosa]

Whitespaces should be removed.

[aried3r]

Done

[mgol]

LGTM.

[mgol]

Sorry, not LGTM. We don't have versioned docs, every page describes the API in all jQuery versions so we need to have all documented. See e.g. how the data page describes 3.0 behavior changes.

Could you try doing sth like that here?

[aried3r]

Could you try doing sth like that here?

I didn't know how to handle the example properly because I couldn't find an example. Hope it's fine like this.

[AurelioDeRosa]

I'd suggest to update the PR to have something like we have for delegate():

// jQuery 1.4.3+
$( elements ).delegate( selector, events, data, handler );
// jQuery 1.7+
$( elements ).on( events, selector, data, handler );

[aried3r]

How about

// jQuery <3.0
var multipleValues = $( "#multiple" ).val() || [];
// jQuery 3.0+
var multipleValues = $( "#multiple" ).val()

jQuery 1.0+ isn't quite correct, since that would include 3 as well. What do you think?

[AurelioDeRosa]

Yeah, it looks good to me.

[kswedberg]

Closed by 5689f1a