Fonction Javascript pour bouton de changement de couleur

Quelqu'un peut me montrer quoi de mal avec cette:

<html>

    <script type = "text/javascript">
        function colourGreen()
        {
            document.getElementById("button1").style.bgColor = 0xFFFF00;
        }
    </script>
    <body>

        <form action="">
            <div id = "button1">
                <input type="button" value="Colour">
            </div id>
            <div id = "button2">
                <input type="button" value="Price up" onclick = "colourGreen()">
            </div id>
            <div id = "button3">
                <input type="button" value="Price down">
            </div id>
        </form>

    </body>
</html>
  • button1 est un div, pas un bouton de.
  • Vous avez également errants 'id' attributs dans votre fermeture des balises div.
  • Ainsi, vous pouvez garder les divs si vous le souhaitez, pour l'espacement. Mais ne prenez pas la peine de leur donner des ID. Mettre l'ID de l'intérieur de la <input> balises, comme ceci: <input type="button" id="button1" value="Couleur" />
  • Vous avez également besoin de fermer correctement votre <input> balises, soit par l'ajout </input> ou à l'aide de l'auto-fermeture <input bla bla />.
InformationsquelleAutor Tom | 2010-07-01
  1. 4
    document.getElementById("button1").style.backgroundColor = '#FFFF00';
    • #FFFF00 obtiendrez jaune. Essayez #00FF00 pour le vert.
    InformationsquelleAutor Darin Dimitrov
  2. 1

    Essayer:

    document.getElementById("button1").style.backgroundColor = '#00ff00';

    Il est backgroundColor pas bgColor

    • Merci beaucoup! je me souviens que vous m'avez aidé avant 🙂 Aussi à quelqu'un d'autre lecture, ne pas utiliser un div comme le fait que les couleurs de l'ensemble de la ligne et non pas le bouton!
    • #FF0000 vous obtiendrez rouge. Essayez #00FF00 pour le vert.
    InformationsquelleAutor Sarfraz
  3. 1

    Vous pouvez créer une règle css et changer le nom de la classe du bouton de lien à la règle. 

    <style type="text/css">
     .buttonColor{
          background-color: green;
     }
</style>

<script type ="text/javascript">
    function colourGreen() {
        document.getElementById("button1").className = "buttonColor";
    }
</script>

    De cette façon, si vous décidez de changer la couleur ou le fond, vous n'aurez pas à changer sur chaque page. Vous serez en mesure de changer qu'un seul fichier css.

    InformationsquelleAutor John
  4. 0

    Je voudrais essayer 

    .style.backgroundColor = 0xFFFF00;
    • Malheureusement cela ne fonctionne pas. 0xFFFF00 valeur non valide.
    • Oups, merci. Je m'étais focalisé sur le cas des OP incorrecte de l'attribut style.
    InformationsquelleAutor Larry K
  5. 0

    Je suppose que votre divs sont seulement aussi grand que vos boutons et sont donc cachés par les boutons eux-mêmes. Le changement de couleur sur le bouton vous même au lieu de la div?

    Un plus propres et plus fiables pour modifier le css des éléments à l'aide des sélecteurs jquery. $("#bouton1").css("background-color","#ff0000");
    Assurez-vous d'inclure jquery .js fichiers avant de les essayer ou vous aurez un objet d'erreur prévu.

    InformationsquelleAutor Bob