views:

6006

answers:

3

I have an element grabbed from document.getElementById('the_id'). How can I get its next sibling and hide it? I tried this but it didn't work:

elem.nextSibling.style.display = 'none';

Firebug error was elem.nextSibling.style is undefined.

+9  A: 

it's because Firefox considers the whitespace between element nodes to be text nodes (whereas IE does not) and therefore using .nextSibling on an element gets that text node in Firefox.

It's useful to have a function to use to get the next element node. Something like this

/* 
   Credit to John Resig for this function 
   taken from Pro JavaScript techniques 
*/
function next(elem) {
    do {
     elem = elem.nextSibling;
    } while (elem && elem.nodeType != 1);
    return elem;  
}

then you can do

var elem = document.getElementById('the_id');
var nextElem = next(elem); 

if (nextElem) 
    nextElem.style.display = 'none';
Russ Cam
But `elem` still might be null.
Gumbo
No it doesn’t. It just guaratees that `nextSibling` is not accessed when `elem` is “not true” or `elem.nodeType == 1`. But if there isn’t such an element, `elem` is just the last sibling node regardless of what type it is.
Gumbo
Here’s an example: var elem = document.createElement("div");elem.appendChild(document.createTextNode("foo"));elem.appendChild(document.createTextNode("bar"));alert(next(elem.firstChild) === null); // "true"
Gumbo
@Gumbo - apologies, you're right, will update now
Russ Cam
Thanks, this worked perfectly!
DisgruntledGoat
@Russ Cam: +1 great code, thanks!.
Marco Demajo
+2  A: 

Firebug error was elem.nextSibling.style is undefined.

because nextSibling can be a text-node or other node type

do {
   elem = elem.nextSibling;
} while(element && elem.nodeType !== 1); // 1 == Node.ELEMENT_NODE
if(elem) elem.style.display = 'none';
Rafael
+1 For checking if `elem` is not null before accessing the `style` attribute.
Gumbo
A: 

Try Looping through the children of this element using something like :

var i=0; (foreach child in elem) { if (i==0) { document.getElementByID(child.id).style.display='none'; } }

Please make appropriate corrections to the Syntax.