If I may be so bold, perhaps you'd be better off using iterators and avoid
nesting while loops based on the same numeric condition.
For exemple, let's imagine I'd like to round up all h3 nodes and their
immediate following siblings in neat <div>s here's what I'd do :
$(".modAccordion > h3").each(function() {
var switch = true; // will be used in the select iteration to stop the
selection when reaching the next h3 element
$(this).nextAll() // selects all the next siblings
.select(function() { return switch && (switch = ! $(this).is('h3')); }) //
stops the selection when the next h3 is reached
.wrapAll('<div class="new"></div>'); // wrap all remaining (the immediate
next siblings before encountering the next h3) in a div with class="new"
});
This should be less messy than nested whiles, while doing the same thing if
I read your method right.
Michel Belleville
2009/11/19 hagbardceline <hagbardcelin...@gmail.com>
> I have the following script (see below), which dynamically adds div
> between closing and opening h3 tags.
> The script works fine in Firefox but IE is infinite looping the first
> while.
>
> Does anyone have an idea how to fix the script for IE?
>
> Thx!
> hagbard
>
>
> <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
> "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd";>
> <html xmlns="http://www.w3.org/1999/xhtml"; xml:lang="en-us">
> <head>
>
> <meta http-equiv="content-type" content="text/html; charset=utf-8" />
> <script type="text/javascript"
> src="http://tablesorter.com/jquery-latest.js";></script>
> <script type="text/javascript">
>
> jQuery(function(){
>
> var nodes = new Array();
> nodes = $(".modAccordion").children();
> $(".modAccordion").children().remove();
>
> var i = 0;
> var headerTest = /.(Head)./;
>
> while(i < nodes.length)
> {
> var testNode = nodes[i].toString();
>
> if((testNode.match(headerTest))){
> $(".modAccordion").append(nodes[i]);
> i++;
>
> var newDiv = document.createElement("div");
> newDiv.className="new";
>
> while(i < nodes.length && !((nodes[i].toString()).match
> (headerTest)))
> {
> newDiv.appendChild(nodes[i]);
> i++;
> }
> $(".modAccordion").append(newDiv);
> }
>
> }
>
> });
>
> </script>
> <style>
> div.new
> {
> background: #00ff00;
>
> }
>
> .modAccordion
> {
> background: #ff0000;
>
> }
>
> </style>
> <title>TableSorter</title>
> </head>
> <body>
> <div class="modAccordion">
> <h3>header 1</h3>
> <p>some text</p>
> <p>some other text</p>
> <h3>header 2</h3>
> <p>some text</p>
> <ul>
> <li>list element 1</li>
> <li>list element 2</li>
> </ul>
> <h3>header 3</h3>
> <p>some text</p>
> <p>some other text</p>
> <p>some more text</p>
> <p>some different text</p>
> </div>
> </body>
> </html>
>
