Update key-concepts with XDomainRequest limitation #572
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added this update in order clarify the understanding implied in the sentence 'This limitation does not apply to scripts that are loaded via jQuery's Ajax methods'. This will help the beginner to understand $.ajax does not work seamlessly with every browser, but only with help of some plugins.
|
Looks like CLA is okay now. Somebody may remove the label. |
|
Great. Let me know if its good for merge. |
|
@dmethvin would you mind doing a quick review? |
|
It seems a bit vague for the learn site. Something more straightforward would be "Versions of Internet Explorer less than 10 do not support cross-domain AJAX requests." XDR is not really a very good substitute but you could reference it. |
|
Hey-hey @moo3, still interested in taking this PR? It needs some updating (see comment above) |
|
ping ping @moo3 |
Removed direct references to XDR and also the link to jQuery ticket, which is now a dead link.
|
LGTM now. |
|
Alright. I'll merge this one when I get home. |
Krinkle
pushed a commit
that referenced
this pull request
Apr 9, 2024
* Update key-concepts with XDomainRequest limitation * Update IE's limited support info
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added this update in order to clarify the understanding implied in the sentence 'This limitation does not apply to scripts that are loaded via jQuery's Ajax methods'. This will help the beginner to understand $.ajax does not work seamlessly with every browser, but only with help of some plugins.