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

Question

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

Answer 1

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

Answer 2

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

Answer 3

It still returns an error in the console why?

Answer 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"

Answer 5

Yay nice

Answer 6

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 }

Answer 7

....Maybe na?

Answer 8

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

Answer 9

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

Answer 10

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

Answer 11

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

Answer 12

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

Answer 13

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

Answer 14

so there is no need to use the variable tyvm?

Answer 15

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

Answer 16

check it, it works.