Change background to random colors on click - Javascript/jQuery

Question

I'm trying to change my body's background color to a random color every time a nav-bar link is clicked.

I got it so that a random color gets changed the first time a link is clicked - but when a different link is clicked, nothing changes.

Any tips?

HTML:

    <div class="nav">
      <ul id="show">
        <li id="link1"><a href="#portfolio">Portfolio</li></a>
        <li id="link2"><a href="#about">About</li></a>
        <li id="link3"><a href="#email">Contact</li></a>
        <hr/>
      </ul>
    </div>

JS:

$(function() {

     var colors = ['red', 'blue', 'green', 'grey'];

     var randColor = colors[Math.floor(Math.random()*colors.length)];

     $('.nav a').click(function() {
         $('body').css('background-color', randColor);
     });
});

Your HTML structure is invalid. It’s supposed to be <li><a>…</a></li> and not <li><a>…</li></a>. And remove the <hr/> inside the <ul>. <ul>s and <ol>s can only have <li>s as child elements. — Sebastian Simon
– Sebastian Simon, Commented Sep 30, 2015 at 1:00
Based on your function I made this codepen.io/eirenaios/pen/EmYBBP — Suresh Karia
– Suresh Karia, Commented Apr 12, 2017 at 7:43

epascarello · Accepted Answer · 2015-09-30 00:59:07Z

3

Because you only generate one random number and keep using it, move it inside of the click method.

answered Sep 30, 2015 at 0:59

epascarello

208k20 gold badges206 silver badges246 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Arun P Johny · Accepted Answer · 2015-09-30 01:02:33Z

In each click you need to find a new random color, in your case the randColor variable is updated only once on the dom ready, in the click handle the same value is used all the time.

$(function() {

  var colors = ['red', 'blue', 'green', 'grey'],
    color;

  $('.nav a').click(function() {
    var randColor;
    do {
      randColor = colors[Math.floor(Math.random() * colors.length)];
    } while (color == randColor);
    $('body').css('background-color', randColor);
    color = randColor;
  });
});

<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script>
<div class="nav">
  <ul id="show">
    <li id="link1"><a href="#portfolio">Portfolio</a></li>
    <li id="link2"><a href="#about">About</a></li>
    <li id="link3"><a href="#email">Contact</a></li>
  </ul>
  <hr/>
</div>

Collectives™ on Stack Overflow

Change background to random colors on click - Javascript/jQuery

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related