Commit Graph

143 Commits

Author SHA1 Message Date
rwldrn
5c3b9e0c24 jQuery.clone() check that destination child nodes are not null. Fixes #9587 2011-06-15 11:14:52 -04:00
timmywil
1d1cb582c0 elem.getElementsByTagName calls the function in IE6/7. Fixes #9370. 2011-05-25 19:35:52 -04:00
Scott Hughes
287156197f Landing pull request 389. Null elements in clone to avoid memory leak in IE. Fixes #9341.
More Details:
 - https://github.com/jquery/jquery/pull/389
 - http://bugs.jquery.com/ticket/9341
2011-05-25 15:18:13 -04:00
Rick Waldron
ec829431fe Landing pull request 365. jQuery.buildFragment, ensure doc is a document; Fixes #8950.
More Details:
 - https://github.com/jquery/jquery/pull/365
 - http://bugs.jquery.com/ticket/8950
2011-05-13 12:26:17 -04:00
jaubourg
391398cf23 Fixes #9221. Wraps openings of html comments and CDATA blocks found at the beginning of inserted script elements into a javascript block comment so that the new implementation of globalEval will not throw an exception in IE (execScript being less lenient than eval). Unit tests added. 2011-05-11 14:13:07 +02:00
John Resig
bfad45fe45 Fix problem with appending multiple string arguments in IE 6. Fixes #9072. 2011-05-03 15:02:26 -04:00
Rick Waldron
0c2d1aee54 jQuery.buildFragment, ensure doc is a document; Includes comments; Adds unit test. Fixes #8950 2011-04-30 10:42:36 -04:00
timmywil
d274b7b9f7 Landing pull request 332. Appending disconnected radio or checkbox inputs and keeping checked setting Fixes #8060, #8500.
More Details:
 - https://github.com/jquery/jquery/pull/332
 - http://bugs.jquery.com/ticket/8060
 - http://bugs.jquery.com/ticket/8500
2011-04-21 21:51:23 -04:00
timmywil
ca5bc202ac IE9's support for SVG elements unfortunately does not extend to clearAttributes. Checks for its presence before proceeding. 2011-04-13 23:45:58 -04:00
timmywil
73f9ab6705 JSLint failed due to function created within for loop 2011-04-12 18:30:21 -04:00
John Resig
3418f32387 Merge branch 'master' of https://github.com/rjgotten/jquery into rjgotten-master. Also added in unit tests covering the case. Fixes #6180.
Conflicts:
	src/manipulation.js
2011-04-12 16:39:30 -04:00
timmywil
5fc2281fcc - Added a hook to swap display none for width and height in browsers that do not sufficiently support get/setAttribute 2011-04-03 19:13:41 -04:00
timmywil
de79e8c7e0 Make the new attr/prop changes pass the test suite (in Webkit). There are still errors in IE.
+ Added hooks for selected, checked, readonly, disabled to removeAttr if set to falsey

+ Removed all attrs from attrFix, these aren't needed for setAttribute

+ If prop is used for class, do we want a propFix for class?

  - We could just assume the user should know to use className with prop.  I've done the latter for now.

+ Created tests for $.fn.prop and $.fn.removeProp

  - Actually all I did was change broken attr tests to prop where it made sense.
2011-04-03 19:13:38 -04:00
Jordan Boesch
124acbfbc5 removing parens 2011-03-14 14:17:02 -06:00
Jordan Boesch
c9ef09c800 bug 6158; fixing replaceWith from throwing errors on non existant elements 2011-03-05 09:46:12 -06:00
Jordan Boesch
5c2d70979c bug 6158; fixing replaceWith from throwing errors on empty elements 2011-03-05 09:30:29 -06:00
jeresig
292acd97a2 Make a new jQuery.support.noCloneChecked - splitting apart the previous feature detect relating to clone in IE, fixes the last remaining issue with IE 9 RC. Fixes #8365. 2011-02-23 13:18:44 -05:00
Anton M
78fc79fad4 Make sure .clone(true) correctly clones namespaced events. Fixes #4537. 2011-02-10 22:20:48 +01:00
Anton M
4490f4285c Fix some whitespace issues. Improve and correct an events test. 2011-02-10 22:20:47 +01:00
Mathias Bynens
462bb1f66a Fixes #8098. Use the fast document.head when available. Don't set unneeded "script.type = text/javascript". 2011-02-02 00:59:00 +01:00
rwldrn
fa4c90987f Fixes #8129. Fix cloning multiple selected options in IE8. 2011-02-02 00:09:51 +01:00
jeresig
375b787838 The default for .clone() is to not clone any events. Fixes #8123. 2011-02-01 08:57:18 -05:00
Colin Snover
0a0cff9d29 Use the original element/fragment as the last item to be appended to the document instead of the first in order to prevent missing elements when appending to multiple elements. Fixes #8070. 2011-01-28 10:55:39 -06:00
rwldrn
f1bd0cfc4d Merge branch '8017lint' of https://github.com/rwldrn/jquery into 8017lint 2011-01-22 00:41:42 +01:00
rwldrn
6458885881 Cleaned up; fixes per review 2011-01-20 15:25:04 -05:00
rwldrn
49cd9c66bf cloneCopyEvent; jQuery.clone() review 2011-01-20 14:27:02 -05:00
Colin Snover
e78d3a7e2d Merge in data_nocollide branch. Fixes #6968, improves unit testing framework checks for leaky stuff. 2011-01-17 15:31:43 -06:00
Colin Snover
b14f02899e Ensure that buildFragment clones elements properly in all browsers. Fixes #3879, #6655. Also improves form element clone tests and fixes bugs in $.fn.clone exposed by these new test cases related to the values of checkboxes and radio buttons in IE. 2011-01-09 18:38:44 -06:00
Dave Methvin
fcf623786a Ensure that the DOM element ref in an event handler is removed by cleanData to avoid an IE6/7/8 memory leak. Fixes #7054. 2011-01-09 16:25:44 -06:00
Colin Snover
885d06c8ef Fix domManip leaks the first element when appending elements to multiple other elements. 2011-01-09 15:56:40 -06:00
Colin Snover
8e59a99e0a Change the way jQuery.data works so that there is no longer a chance of collision between user data and internal data. Fixes #6968. 2011-01-09 15:52:33 -06:00
Dave Methvin
5fd21fc02b Don't cache non-html strings in buildFragment to avoid possible collision with the names of Object methods like toString. Also makes the unit tests 0.5% to 8% faster. Fixes #6779. 2010-12-27 13:43:52 -06:00
Colin Snover
82ac384b49 Don't use for-in loops on Arrays. Fixes #7817. Thanks to dmethvin.
Conflicts:
	src/manipulation.js
2010-12-22 18:32:33 -06:00
Colin Snover
445fdf720c Remove code for ticket #7717 which has been marked WONTFIX to match existing $.data functionality and to prevent infinite loops caused by circular references. 2010-12-22 14:43:17 -06:00
Colin Snover
5607bd8d53 Fix a potential error in the previous commit caused by the use of a separate index variable. Thanks to dmethvin for the review. 2010-12-22 09:14:53 -06:00
Dave Methvin
4424bda377 Use a for loop rather than for/in loop when copying events, so that code will work with an augmented Array.prototype. Fixes 7809. 2010-12-20 22:23:59 -05:00
Colin Snover
faefbb1ad0 Fix #7717 and #7165. Thanks to dmethvin and iliakan for their help fixing these issues. 2010-12-19 15:33:53 -06:00
Colin Snover
291b1edf44 Address some code style issues in the clone fix. 2010-12-16 01:29:06 -06:00
Colin Snover
c9c9057c4d More specific version numbers. 2010-12-12 20:01:47 -06:00
Colin Snover
1a3fd3329d Speed up & compatibility improvements for new clone mechanism in IE. 2010-12-12 19:23:22 -06:00
Colin Snover
4fae75d575 Merge branch 'bug5566' into csnover-bug5566. Fixes #4386, #5566, #6997.
Conflicts:
	src/manipulation.js
	test/unit/manipulation.js
2010-12-12 02:39:06 -06:00
Colin Snover
7481a3645a Fix the clone method to be a little less insane in IE, which fixes the new event-cloning clone() as well as probably a bunch of IE-related clone bugs. 2010-12-12 02:20:31 -06:00
jeresig
29616e60c8 Backing out cec68e2b00, was causing serialize tests to fail. Un-fixes #5566. 2010-12-09 13:27:52 -05:00
jeresig
cec68e2b00 Merge branch 'bug5566' of https://github.com/csnover/jquery into csnover-bug5566 2010-12-09 12:44:30 -05:00
Colin Snover
e4900df838 Clone fragments in domManip using jQuery.clone instead of DocumentFragment.cloneNode in order to carry over event data. Fixes #5566, #6997. 2010-12-06 19:37:16 -06:00
Colin Snover
c8be9461ee Clean up empty lines with whitespace in manipulation.js 2010-12-06 19:36:42 -06:00
Anton M
ab2a70e6cf Make sure IE clones body elements correctly. Fixes 4386. 2010-11-21 01:58:46 +01:00
John Resig
fb48ae8e6c Make sure that when multiple variables are being declared that assignments are each done on their own line. 2010-11-09 11:09:07 -05:00
John Resig
b5b3c73db3 We don't do end of line comments, move them above. 2010-11-09 10:34:32 -05:00
rjgotten
df7dfc2404 Check against the type attribute of script elements retrieved through getElementsByTagName() so that only those elements of type "text/javascript" are handled by 'clean'.
Fixes #6180: jQuery.clean should not touch script tags that are not of type text/javascript
2010-10-17 08:30:05 -07:00