@Kangax
I am not aware of any issues with insertBefore and table elements. I
know that IE has issues with innerHTML and table/select elements.
@EMoreth
if nextSibling is null insertBefore will act as appendChild so it
should still work out.
http://developer.mozilla.org/en/DOM/element.insertBefor
Using native IE method (swapNode) might help for weird issues in table.
http://msdn.microsoft.com/en-us/library/ms536774(VS.85).aspx
if (element.swapNode) return element.swapNode(secondElement);
2008/9/23 EMoreth <[EMAIL PROTECTED]>
>
> Shouldnt it use the parent node of both elements ??
>
> Ca
Shouldnt it use the parent node of both elements ??
Cause if i decide to swap 'div1' and 'div2' here ill have no
nextSibling (assuming that no textNodes would be returned as
siblings)...
EMoreth
On Sep 23, 5:04 pm, kangax <[EMAIL PROTECTED]> wrote:
> On Sep 23,
On Sep 23, 11:25 am, John-David Dalton <[EMAIL PROTECTED]>
wrote:
> @Tag can you provide a case where this would be a real issue ?
> I can't see someone swapping nodes with the document.documentElement
> or anything.
I think `Element.replace` is a good way to ensure no cross-browser
bugs crawl in
@Tag can you provide a case where this would be a real issue ?
I can't see someone swapping nodes with the document.documentElement
or anything.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups
"Prototype: Core" group
Perhaps you intend it, but you're assuming both nodes exist in a DOM
fragment (and are not the root node of the fragment).
I realize this scenario may be best handled by the developer's use of
Element.replace instead, but perhaps some error checking will help
avoid confusion.
TAG
On Sep
> I didn't hijacked anyone's thread.
That's a matter of opinion. In *my* opinion, not hijacking the thread
would have looked like this: "I'd really like to see a fix for bug
#1234 (error using update() on 'object' elements) in 1.6.0.3, it's a
serious problem." And, er, yes, that *would* have i
Awesome use of nextSibling and parentNode!
I have added my version which checks for the existence of the
element.swapNode method (in IE).
You can find it posted in the ticket.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google G
Hi all,
I was reading in some LightHouse comments about the new swap method on
elements and was wondering if the following is interesting for that
matter. I had it written this way in my simple code:
// nodes to swap (t)arget node, (s)econdary node
swapNodes:
function(t, s) {
var n =