Cambiar fondo de un Div

[Adler]

La idea es que cuando se pulse sobre la imagen cambie el fondo del div

Código:

var color = new Array('ff0000','000080','000000','808000','800080','ffff00','804000','ff00ff','cdb32c','00ffff')
function sacar_color(c) {
document.getElementById('Mcolor').style.backgroundColor = "#" + color[c];     }

<div id="bbcolor_mostrar" name="Mcolor"></div>

<td><img  name="color5" id="color5" src="./iconos/color5.gif" width="12px" height="12px" border="0" alt="Amarillo" title="Amarillo" onClick="sacar_color(5);" />

Gracias

[ZiTAL]

fijate en la ID de la capa y la ID que pones en el getElementById...

[netzky]

a nivel Post en html funciona la propiedad name, pero tu usas ahorita el metodo de Javascript de obtener un elemento por el ID no por el nombre, como anteriormente te dijeron... estas trayendo a un elemento que no existe, en ese momento por eso no veras ningun cambio...


Codigo Original:

Código HTML:

var color = new Array('ff0000','000080','000000','808000','800080','ffff00','804000','ff00ff','cdb32c','00ffff')
function sacar_color(c) {
document.getElementById('Mcolor').style.backgroundColor = "#" + color[c];     }

<div id="bbcolor_mostrar" name="Mcolor"></div>

<td><img  name="color5" id="color5" src="./iconos/color5.gif" width="12px" height="12px" border="0" alt="Amarillo" title="Amarillo" onClick="sacar_color(5);" /> 

Codigo Final:

Código HTML:

var color = new Array('ff0000','000080','000000','808000','800080','ffff00','804000','ff00ff','cdb32c','00ffff')
function sacar_color(c) {
document.getElementById('color5').style.backgroundColor = "#" + color[c];     }

<div id="bbcolor_mostrar" name="Mcolor"></div>

<td><img  name="color5" id="color5" src="./iconos/color5.gif" width="12px" height="12px" border="0" alt="Amarillo" title="Amarillo" onClick="sacar_color(5);" /> 

Te recomiendo crear una variable mas en tu funcion javascript, para asi poder cambiarle el color de fondo a cualquier elemento en vez de crear una funcion para cada elemento al cual quieras cambiarle el fondo

[Adler]

Hola, gracias por contestar.
Algo que se me olvidó comentar (las prisas ) es que trabajo con Firefox.

Creo que me he expresado mal. Lo que quiero es que el Div cambie su fondo, cuando pinche en la imagen.

Con respecto a la Id del Div teneis razón, pero ese no es problema, fue solamente que lo hice de memoría, asi que tanto la getElementById, Id, Name es el mismo.

¿Alguna otra idea?

Gracias

[Adler]

Bueno ya encontre la solución. Esta en CSS que no le había puesto padding

Gracias a todos aquellos que se interesaron por mi consulta