Permalink
Browse files

Correct oldIE-related comments, revert some workarounds. Close gh-1207.

  • Loading branch information...
1 parent ba16ba2 commit 7049877530882c2f8ee09f71f7fc736ca9570736 @mgol mgol committed with dmethvin Mar 20, 2013
Showing with 10 additions and 17 deletions.
  1. +4 −2 CONTRIBUTING.md
  2. +1 −1 src/ajax.js
  3. +3 −6 src/attributes.js
  4. +1 −7 src/core.js
  5. +1 −1 src/effects.js
View
@@ -207,10 +207,12 @@ This will only run the "css" module tests. This will significantly speed up your
**ALWAYS RUN THE FULL SUITE BEFORE COMMITTING AND PUSHING A PATCH!**
-### jQuery supports the following browsers:
+### jQuery 2.x supports the following browsers:
* Chrome Current-1
* Safari Current-1
* Firefox Current-1
-* IE 6+
+* IE 9+
* Opera Current-1
+
+jQuery 1.x additionally supports IE6+.
View
@@ -8,7 +8,7 @@ var
ajax_rquery = /\?/,
rhash = /#.*$/,
rts = /([?&])_=[^&]*/,
- rheaders = /^(.*?):[ \t]*([^\r\n]*)\r?$/mg, // IE leaves an \r character at EOL
+ rheaders = /^(.*?):[ \t]*([^\r\n]*)$/mg,
// #7653, #8125, #8152: local protocol detection
rlocalProtocol = /^(?:about|app|app-storage|.+-extension|file|res|widget):$/,
rnoContent = /^(?:GET|HEAD)$/,
View
@@ -23,11 +23,8 @@ jQuery.fn.extend({
removeProp: function( name ) {
name = jQuery.propFix[ name ] || name;
return this.each(function() {
- // try/catch handles cases where IE balks (such as removing a property on window)
- try {
- this[ name ] = undefined;
- delete this[ name ];
- } catch( e ) {}
+ this[ name ] = undefined;
+ delete this[ name ];
});
},
@@ -249,7 +246,7 @@ jQuery.extend({
for ( ; i < max; i++ ) {
option = options[ i ];
- // oldIE doesn't update selected after form reset (#2551)
+ // IE6-9 doesn't update selected after form reset (#2551)
if ( ( option.selected || i === index ) &&
// Don't return options that are disabled or in a disabled optgroup
( jQuery.support.optDisabled ? !option.disabled : option.getAttribute("disabled") === null ) &&
View
@@ -133,13 +133,7 @@ jQuery.fn = jQuery.prototype = {
// Check parentNode to catch when Blackberry 4.6 returns
// nodes that are no longer in the document #6963
if ( elem && elem.parentNode ) {
- // Handle the case where IE and Opera return items
- // by name instead of ID
- if ( elem.id !== match[2] ) {
- return rootjQuery.find( selector );
- }
-
- // Otherwise, we inject the element directly into the jQuery object
+ // Inject the element directly into the jQuery object
this.length = 1;
this[0] = elem;
}
View
@@ -279,7 +279,7 @@ function defaultPrefilter( elem, props, opts ) {
// height/width overflow pass
if ( elem.nodeType === 1 && ( "height" in props || "width" in props ) ) {
// Make sure that nothing sneaks out
- // Record all 3 overflow attributes because IE does not
+ // Record all 3 overflow attributes because IE9-10 do not
// change the overflow attribute when overflowX and
// overflowY are set to the same value
opts.overflow = [ style.overflow, style.overflowX, style.overflowY ];

0 comments on commit 7049877

Please sign in to comment.