Changer couleur lien avec javascript

[sigma2008]

Bonjour, j'ai ce code qui ne fonctionne pas (le code change la couleur du lien en choisissant la couleur dans input):

<script>
function colorDiv1() {
    var selection = document.getElementsByTagName('lienmenu').value;
    document.getElementById("#menu li a").style.backgroundColor = selection;
}
</script>
<input value="#000000" type="color" id="lienmenu" name="lienmenu" onchange="colorDiv1()">

<div id="menu">
<ul>
<li>
<a href="">Bonjour tout le monde</a>
</li>
</ul>
</div>

J'utilise le même code pour changer la couleur d'un (#menudroit) avec ce code :

document.getElementById("menudroit").style.backgroundColor = selection;

ça marche parfaitement mais pour (#menu li a) ça ne marche pas, j'ai essayé querySelectorAll et querySelector ça ne marche pas aussi !

PS : j'insiste à utiliser #menu li a
Merci pour votre aide ^^

 

[Furtif]

Ce serait plus simple de faire ton hover sur un lien avec 4 lignes de CSS..

Autant eviter de javascript pour ca..

 

[niap]

Normal que ça ne fonctionne pas.
document.getElementById("#menu li a") ne te donnera aucun résultat, si tu veux faire un tel sélecteur il faut jQuery.
Et getElementsByTagName ne te renverra rien non plus dans ce cas, n'est pas approprié non plus, il faut sélectionner par id.

<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.10.2/jquery.min.js"></script>
<script>
function colorDiv1() {
    var selection = $('#lienmenu').val();
    $("#menu li a").css('background-color', selection);
}
</script>

 

[spout]

si tu veux faire un tel sélecteur il faut jQuery

Ou document.querySelectorAll

 

[UsagiYojimbo]

Ce serait plus simple de faire ton hover sur un lien avec 4 lignes de CSS..

Autant eviter de javascript pour ca..

+1.

 

[niap]

si tu veux faire un tel sélecteur il faut jQuery

Ou document.querySelectorAll

Oui aussi

<script>
function colorDiv1() {
	var selection = document.getElementById('lienmenu').value;
	var el = document.querySelectorAll("#menu li a");
	for(i = 0; i < el.length; i++){
		el[i].style.backgroundColor = selection;
	}
}
</script>

 

[sigma2008]

@niap ça marche parfaitement merci