search for: _counted

...e immediate parent -- the A tag -- which is incorrect. The script just removes the link (and image), instead of the entire row. It looks like IE6 (I haven''t tested IE7, but FF and Safari don''t do this) is messing up in the unmark() function on line 2295 (v.1.5.1.1) by setting the _counted property on the IMG and A elements to the string "undefined", as opposed to nothing. I assume that unmark() was called by _rows[0].down() in the addLineItem() function. So the line (#2595) in matchElements has "if (element._counted)", which returns true when it shouldn''...

...matches[2]](nodeValue, matches[5] || matches[6]); } }, @@ -3029,14 +3074,15 @@ Object.extend(Selector, { // marks an array of nodes for counting mark: function(nodes) { + var _true = Prototype.emptyFunction; for (var i = 0, node; node = nodes[i]; i++) - node._counted = true; + node._countedByPrototype = _true; return nodes; }, unmark: function(nodes) { for (var i = 0, node; node = nodes[i]; i++) - node._counted = undefined; + node._countedByPrototype = undefined; return nodes; }, @@ -3044,15 +3090,15...

Note that one of the 8 patches (#6) will be sent separately in reply to this email, as some of the replaced lines are too long, so git won't let me send the email. However, there is nothing wrong with that patch, and it should be applied in the sequence listed below. Note also that I assume this will be tested on a clean f11 install, rather than an upgrade of an existing ovirt server