Permalink
Browse files

Docs: Update support comments related to IE

All support comments were checked for Edge applicability.
  • Loading branch information...
1 parent 622db29 commit 693f1b537b0a19cda8b7e8f5379bffa5351b8a6e @mgol mgol committed Mar 23, 2016
Showing with 12 additions and 11 deletions.
  1. +4 −3 src/ajax.js
  2. +2 −2 src/attributes/support.js
  3. +1 −1 src/attributes/val.js
  4. +1 −1 src/core.js
  5. +1 −1 src/data.js
  6. +1 −1 src/effects.js
  7. +1 −1 src/manipulation.js
  8. +1 −1 src/offset.js
View
@@ -535,12 +535,13 @@ jQuery.extend( {
if ( s.crossDomain == null ) {
urlAnchor = document.createElement( "a" );
- // Support: IE8-11+
- // IE throws exception if url is malformed, e.g. http://example.com:80x/
+ // Support: IE <=8 - 11, Edge 12 - 13
+ // IE throws exception on accessing the href property if url is malformed,
+ // e.g. http://example.com:80x/
try {
urlAnchor.href = s.url;
- // Support: IE8-11+
+ // Support: IE <=8 - 11 only
// Anchor's host property isn't correctly set when s.url is relative
urlAnchor.href = urlAnchor.href;
@jasonsturges
jasonsturges Jun 8, 2016

Is this self assignment of urlAnchor.href = urlAnchor.href intended?

@mgol
mgol Jun 8, 2016 edited Member

It is. As weirdly as it sounds, this actually changes various anchor properties in IE.

@mgol
mgol Jun 8, 2016 Member

See #1880 (comment) for more info about what happens. Fortunately Edge fixes these bugs but since even IE 11 has them this code is not going to be removed in the foreseeable future.

@jasonsturges
jasonsturges Jun 8, 2016

That makes sense - thanks for the details.

s.crossDomain = originAnchor.protocol + "//" + originAnchor.host !==
@@ -14,11 +14,11 @@ define( [
// Default value for a checkbox should be "on"
support.checkOn = input.value !== "";
- // Support: IE<=11+
+ // Support: IE <=11 only
// Must access selectedIndex to make default options select
support.optSelected = opt.selected;
- // Support: IE<=11+
+ // Support: IE <=11 only
// An input loses its value after becoming a radio
input = document.createElement( "input" );
input.value = "t";
@@ -85,7 +85,7 @@ jQuery.extend( {
return val != null ?
val :
- // Support: IE10-11+
+ // Support: IE <=10 - 11 only
// option.text throws exceptions (#14686, #14858)
// Strip and collapse whitespace
// https://html.spec.whatwg.org/#strip-and-collapse-whitespace
View
@@ -274,7 +274,7 @@ jQuery.extend( {
},
// Convert dashed to camelCase; used by the css and data modules
- // Support: IE9-11+
+ // Support: IE <=9 - 11, Edge 12 - 13
// Microsoft forgot to hump their vendor prefix (#9572)
camelCase: function( string ) {
return string.replace( rmsPrefix, "ms-" ).replace( rdashAlpha, fcamelCase );
View
@@ -87,7 +87,7 @@ jQuery.fn.extend( {
i = attrs.length;
while ( i-- ) {
- // Support: IE11+
+ // Support: IE 11 only
// The attrs elements can be null (#14894)
if ( attrs[ i ] ) {
name = attrs[ i ].name;
View
@@ -142,7 +142,7 @@ function defaultPrefilter( elem, props, opts ) {
// Restrict "overflow" and "display" styles during box animations
if ( isBox && elem.nodeType === 1 ) {
- // Support: IE 9 - 11
+ // Support: IE <=9 - 11, Edge 12 - 13
// Record all 3 overflow attributes because IE does not infer the shorthand
// from identically-valued overflowX and overflowY
opts.overflow = [ style.overflow, style.overflowX, style.overflowY ];
View
@@ -29,7 +29,7 @@ define( [
var
rxhtmlTag = /<(?!area|br|col|embed|hr|img|input|link|meta|param)(([a-z][^\/\0>\x20\t\r\n\f]*)[^>]*)\/>/gi,
- // Support: IE 10-11, Edge 10240+
+ // Support: IE <=10 - 11, Edge 12 - 13
// In IE/Edge using regex groups here causes severe slowdowns.
// See https://connect.microsoft.com/IE/feedback/details/1736512/
rnoInnerhtml = /<script|<style|<link/i,
View
@@ -91,7 +91,7 @@ jQuery.fn.extend( {
return;
}
- // Support: IE<=11+
+ // Support: IE <=11 only
// Running getBoundingClientRect on a
// disconnected node in IE throws an error
if ( !elem.getClientRects().length ) {

0 comments on commit 693f1b5

Please sign in to comment.