DomHelper: cannot create a label tag

[e.e.coli]

Captain, I canna

dh.append(myThang, {tag: 'label, for: 'anything'});

because 'for' is a reserved word, but is also a required attribute for a label tag.

Cheers.

[KuN]

'for':'anything'
doesnt work?

[e.e.coli]

Well, dip me in cornflour and deep fry me. It does work.

Excuse me, Jack, KuN, and all.

[e.e.coli]

'for':'anything'
doesnt work?

Ach, in fact it does not work... in IE 6 and in an older Firefox. "for" is indeed a reserved word, and is not permitted as a property identifier in an object.

Jack, if you're reading this, my current workaround is to patch DomHelper something like

            if(attr == 'cls'){
                b += ' class="' + o['cls'] + '"';
            }else if(attr == 'forfield'){
                b += ' for="' + o[attr] + '"';
            }else{
                b += ' ' + attr + '="' + o[attr] + '"';
            }

and

           if(attr=='cls'){
                el.className = o['cls'];
            }else if (attr=='forfield') {
            	el.htmlFor = o['forfield'];
            }else{
                if(useSet) el.setAttribute(attr, o[attr]);
                else el[attr] = o[attr];
            }

I'm not advocating for "forfield" as a "good" solution; I trust your aesthetics about that.
Cheers.

[e.e.coli]

Better yet, if you use

htmlFor

as the special property, you only have to patch the html gen side, not the dom side.

[jack.slocum]

I have never run into an error using 'for', but it doesn't surprise me that it could throw an error.

I used the htmlFor patch. Thanks for posting it.

            ...
            if(attr == 'cls'){
                b += ' class="' + o['cls'] + '"';
            }else if(attr == 'htmlFor'){
                b += ' for="' + o['htmlFor'] + '"';
            }else{
                b += ' ' + attr + '="' + o[attr] + '"';
            }
            ...

[e.e.coli]

I used the htmlFor patch. Thanks for posting it.

You are most welcome. It's a tiny drop in the ext bucket.