Javascript DOM help

MySchizoBuddy

I want this label to be invisible
<label class="Radio" for="Radio_FCKeditor">Toolbar</label>
this should work. but it isn't
var elems = document.getElementsByTagName("label") ; for (var i = 0; i < elems.length; i++) { if (elems[i].attributes["for"].value == "Radio_FCKeditor") { elems[i].style.display = "none"; } }

How do I add the !important string next to none, because its being owerwritten

Max_B

It"d be MUCH more efficient to give your targeted label an ID and then just reference that ID.
document.getElementById('your_id').style.display = 'none';

MySchizoBuddy

yeah but the label tag is generated by Vanilla. not by me.
I used visibility:hidden instead

Dinoboff

It should be:if (elems[i].attributes["htmlFor"].value == "Radio_FCKeditor") {'for', 'style' and 'class' attributes have a special name in the html DOM: 'htmlFor', 'cssText' and 'className'.

MySchizoBuddy

thanks dino
do u know of a way to make the style.display = none take precedence. Now now its being overwritten by vanilla css. So i used visibility :hidden instead.

Dinoboff

Using style.display should be equivalent of using inline style so, it shouldn't be overwritten - I think.

Here is what I use to hide elements:/** * Hide on element. * @param {DOMelement} elem Element to hide */ hide = function(elem){ if(elem.style && typeof(elem.style.display) !== 'undefined'){ elem.$oldDisplay = elem.style.display; // Saved the display for later use. John Resig, 2006. elem.style.display = 'none'; } }; /** * Show a hidden element. * @param {DOMelement} elem Element to show */ show = function(elem){ if(elem.style && typeof(elem.style.display) !== 'undefined'){ elem.style.display = elem.$oldDisplay || ''; } };