function setColor(color)

[woodier]

I'm attempting to use Javascript (function setColor(color)) with a popup color picker. I need to set the color for at least 2 page attributes. My first attempt is cross browser compatible, but limited to 1 field.

<script type="text/javascript">
function setColor(color){
if (opener && !opener.closed){
opener.document.theForm.theField.value = color;
opener.focus();
}
window.close();
}
</script>

My second attempt works for Firefox, but not for Internet Explorer.

<script type="text/javascript">
function setColor(color)
{
if (opener && !opener.closed && opener.setTargetField)
{
opener.setTargetField(targetField, color.toUpperCase());

}
window.close();
}
</script>

Any help will be greatly appreciated.

[acoder]

Welcome to TSDN!

Can you post the parent window code or at least setTarget.

[woodier]

Greetings!

Maybe it would be easier to start off with DOM, window.opener.document.getElementById, but I'm not familiar with it.

<script type="text/javascript">

function openColorPicker(targetField){

var w = window.open('WriteTable.html','color_popup','width =400,height=450,scrollbars=0');
w.targetField = targetField; //create target field variable in popup window with the passed targetField as value
w.focus();
return false;
}

//callback function
function setTargetField(targetField, color){
if (targetField){
targetField.value = color;
}
window.focus();
}
</script>

<form name="theForm" action="javascript:setColors()" style="margin:0;">

Background color:
<br><input name="bgcolorbox" type="text" class="colorbox" maxlength="6" onChange="setColors()" readonly="readonly"><br \>
<a href="" onclick="return openColorPicker(document.theForm.bgcolorbox)">chan ge color</a><br />
<br>&nbsp;
<br>Font color:
<br><input name="fontcolorbox" type="text" class="colorbox" maxlength="6" onChange="setColors()" readonly="readonly"><br \>
<a href="" onclick="return openColorPicker(document.theForm.fontcolorbox)">ch ange color</a><br />

<br><input type="submit" value="Set Colors" class="subtn">

<br></form>

[acoder]

Do you get an error in IE? What you could try doing instead is to pass a string (the id of the form field). This means you will have to set the ID for each form element. Then to set the color, you could use:

document.getElementById(targetFieldID).value = color;

[woodier]

acoder,

You agree with my earlier statement. I stated earlier that I have not ever used the required javascript, so I don't know how to return the value to the parent page or how to write the callback function. Please show the full script and not just a snippet. This is an example of my popup cells.

<td id="cell" 0_0="" onclick='javascript:setColor("#FFFFFF");' bgcolor="#ffffff" height="25" width="30"></td>

Thanks in advance.

[acoder]

So does IE give an error?

Try adding a var targetField at the beginning of the popup.

[woodier]

I'm sorry that I forgot to inform you that it is errorless.