Function working in IE, but not Firefox

[grande]

function edit(entity) {
	str=document.getElementById(entity).innerHTML;
	if (str.indexOf("<INPUT") < 0) {
		document.getElementById(entity).innerHTML="<INPUT type='text' name='edit_" + entity + "' value='" + str + "'>";
	}
}

This is supposed to change text into a textbox. It works fine in IE, but not in Firefox (I haven't tested in any other browsers because I don't have any other browsers on this PC)

Any ideas?

-------------------------
Call me barely Impressive Captain.

 

[cLFlaVA]

1) what does firefox's javascript console say? any errors reported?

2) consider creating an input element using createElement instead of just setting the innerHTML.



*cLFlaVA
----------------------------
[tt]( <P> <B>)¹³ * (<P> <.</B>)[/tt]

[URL unfurl="true"]http://www.coryarthus.com/[/url]

 

[grande]

By the way, I should say what's happening.

In Firefox, the textbox is created, but when I click on it again, it inserts the HTML to create a textbox into the textbox.

When I put an alert on indexOf("<INPUT"), it always comes back as -1 in Firefox.

-------------------------
Call me barely Impressive Captain.

 

[BillyRayPreachersSon]

Try using "input" in lowercase. Browsers take what you give them and parse it - most put tag names in lowercase when you ask for things back.

Hope this helps,
Dan



Coedit Limited - Delivering standards compliant, accessible web solutions

[tt]Dan's Page [blue]@[/blue] Code Couch

http://www.codecouch.com/dan/

[/tt]

 

[tsuji]

>if (str.indexOf("<INPUT") < 0) {
[tt]if (str.[blue]toUpperCase().[/blue]indexOf("<INPUT") < 0) {[/tt]

 

[tsdragon]

I agree with tsuji: never assume the case of anything you don't have control over. Convert it to upper (or lower) case and you'll always know for certain.

Tracy Dryden

Meddle not in the affairs of dragons,
For you are crunchy, and good with mustard.

 

[BillyRayPreachersSon]

grande,

Did you resolve your problem?

Dan



Coedit Limited - Delivering standards compliant, accessible web solutions

[tt]Dan's Page [blue]@[/blue] Code Couch

http://www.codecouch.com/dan/

[/tt]

 

[BillyRayPreachersSon]

grande,

Did you resolve your problem?

Dan



Coedit Limited - Delivering standards compliant, accessible web solutions

[tt]Dan's Page [blue]@[/blue] Code Couch

http://www.codecouch.com/dan/

[/tt]

 

[BillyRayPreachersSon]

grande,

Did you resolve your problem?

Dan




Coedit Limited - Delivering standards compliant, accessible web solutions

[tt]Dan's Page [blue]@[/blue] Code Couch

http://www.codecouch.com/dan/

[/tt]