And this didn't happen in previous revisions (Rev: 452, 451)? I'll see
if I can't find some time to run some tests later today.
--
Brandon Aaron
On 10/25/06, Klaus Hartl <[EMAIL PROTECTED]> wrote:
> Hi all,
>
> with the latest Revision the clean function lets Safari crash under
> certain circumstances (I had to fix that for Plazes):
>
> There is an <ul> to which some <li> elements should be appended. These
> <li>s are loaded via XHR as text/html and are appended in the success
> function like this:
>
> success: function(html) {
> $('#photos').append(html);
> }
>
> The server returns an HTML snippet like the following:
>
> <li>...</li>
> <li>...</li>
>
> Important: there is a line break after the closing li tag. In the clean
> function, on line 1568 there is the following code:
>
> arg = div.childNodes
>
> which returns a node list containing the children (first level) of the
> temporary div, of the form:
>
> [<li>, Text \n , <li>, Text \n, ... ]
>
> The line-break text nodes are causing Safari to crash completely!
>
> I fixed that by changing line 1568 to this:
> arg = $('>*', div).get();
>
> Not sure if that is the best solution. Maybe the trim function does not
> work correct?
>
> If solution is OK, I'd commit that...
>
>
> -- Klaus
>
> _______________________________________________
> jQuery mailing list
> [email protected]
> http://jquery.com/discuss/
>
_______________________________________________
jQuery mailing list
[email protected]
http://jquery.com/discuss/
