Added note for Global Ajax Event Handlers#797
Conversation
There was a problem hiding this comment.
Is it 1.9 or 1.8? Descriptions currently say 1.8.
There was a problem hiding this comment.
In 1.8 you should, in 1.9 you must. Is that clear enough?
There was a problem hiding this comment.
Ah, I see. I think what you have is fine then. We typically use "As of jQuery x.x" instead of "Starting with x.x." Would you mind changing that phrase?
There was a problem hiding this comment.
Updated based on your suggestion :)
|
Thanks @AurelioDeRosa. Are you going to also remove the notes that currently appear in the |
|
I think your comment is related to #798. Can you check that issue I raised? |
|
Yeah, that's what I saw. What I'm suggesting is, if you're adding the "global" note to each method, you should also remove the "local" note. |
|
In this case, should I highlight that from 1.8 is recommended, but from 1.9 is mandatory? |
|
Never mind. I (finally) see what you're getting at. Maybe just change the global note from "must" to "must" (using If you do that, then it LGTM, so feel free to merge it.. |
|
@kswedberg updated based on your comment :) |
|
Excellent. Thanks, @AurelioDeRosa ! Wanna merge it in? |
|
@kswedberg Done! |
|
Thanks, @AurelioDeRosa ! |
Fixes gh-479