Help, lol. What's wrong with this JS. | SoloLearn: Learn to code for FREE!

+2

Help, lol. What's wrong with this JS.

https://code.sololearn.com/Wu3n1NR6WtTp/?ref=app

8/24/2017 9:03:55 AM

James Barrow

16 Answers

New Answer

+2

nevermind, i got it. https://code.sololearn.com/Wu3n1NR6WtTp/?ref=app

+6

window.onload = function(){ var button = document.getElementById("button"); var rainbow = ["red", "orange","yellow","green","blue","indigo ", "violet"]; function change(){ document.body.style.background = rainbow[Math.floor(7*Math.random())]; } button.addEventListener("click", change); } math>Math body.background? what is it? I only know body.style.background "change()" seriously? It is "change" bc "click" event will do "change()" when click Absolutely I can explain more correctly and truly of that. But I'm afraid You will require more confuse

+5

It still returns an error in the console why?

+4

oh then Your code wrong again should be document.body.style.background =document.getElementById('thanks').innerHTML = rainbow[Math.floor(7*Math.random())]; document.getElementById('thanks') for select element with include id "thanks"

+4

+3

Because your script is executed before your html is loaded. So it cant find the button... I suggest you to put your whole script inside window.onload = function () { //your code }

+3

+3

oh sorry My mistake vision :P tyvm.innerHTML=document.body.style.background=rainbow[7*Math.random()|0] no ".color"

+3

add "tyvm.style.color=" in front of the previous answer

+3

ah ha I just don't get what you want that's all

+2

Unfortunately that isn't working. It's popping a math error. Saying math is undefined.

+2

thanks man. i think i got now. thanks for the critique of my code. 👍

+2

i am still playing with the code. a Trying to get the color of the h1 tag to change also and that is what's throwing the error atm

+2

so there is no need to use the variable tyvm?

+2

Hey man, that fix changes what the h1 says. I want to change the color of the text.

+2

check it, it works.