JavaScript classList.toggle() for multiple divs classlist toggle multiple classes,classlist toggle multiple,javascript classlist toggle

I have 3 divs as colors to choose from and 3 blank divs. I want to let the user be able to:

(1) click a colored div and then a blank div, then the blank div is colored as the color the user choose. And the code seems to work.

(2) I want the user to be able to click the colored blank div again and it becomes white. And the code seems to work.

The problem is, if the blank div is colored and the user choose another color and click the colored blank div again, a newer color class will be added to the div, and things become unpredictable. You can open the console and track the messy change of the class of the blank div.

How can I solve this problem? I only want the blank divs to toggle between two classes.

var chosenColor;
function pickColor(arg){
  chosenColor=arg.id;
}

function draw(id){
  document.getElementById(id).classList.toggle("white");
  document.getElementById(id).classList.toggle(chosenColor);
}

.box{
  width: 100px;
  height: 100px;
  border: 1px solid black;
  display: inline-block;
  
}

.red{background: red}
.blue{background: blue;}
.yellow{background: yellow;}
.white{background: white;}

<html>
  <body>
    <div class="box red" id="red" onclick="pickColor(this)">1</div>
    <div class="box blue" id="blue" onclick="pickColor(this)">2</div>
    <div class="box yellow" id="yellow" onclick="pickColor(this)">3</div>
    <br><br>
    <div class="box white" id="4" onclick="draw(4)">4</div>
    <div class="box white" id="5" onclick="draw(5)">5</div>
    <div class="box white" id="6" onclick="draw(6)">6</div>
  </body>
</html>

Answer:1

Check to see if the element's classname is white. If not, set its class name to white - else, set it to the chosen color. You can put the boxes in a container and use .container > div selector, removing the need to give the boxes the .box class. Also, in a listener, this will refer to the clicked element - there's no need to use getElementById when you already have a reference to the element.

var chosenColor;

function pickColor(arg) {
  chosenColor = arg.id;
}

function draw(element, id) {
  if (element.className !== 'white') element.className = 'white';
  else element.className = chosenColor;
}

.container > div {
  width: 100px;
  height: 100px;
  border: 1px solid black;
  display: inline-block;
}

.red {
  background: red
}

.blue {
  background: blue;
}

.yellow {
  background: yellow;
}

.white {
  background: white;
}

<div class="container">
  <div class="red" id="red" onclick="pickColor(this)">1</div>
  <div class="blue" id="blue" onclick="pickColor(this)">2</div>
  <div class="yellow" id="yellow" onclick="pickColor(this)">3</div>
  <br><br>
  <div class="white" id="4" onclick="draw(this, 4)">4</div>
  <div class="white" id="5" onclick="draw(this, 5)">5</div>
  <div class="white" id="6" onclick="draw(this, 6)">6</div>
</div>

Answer:2

Instead of using classes and running into the issue of assigning multiple nested classes or having to use complicated white logic...

I'd use data-* attribute:

var chosenColor;

function pick(el) {
  chosenColor = el.dataset.color;
}

function draw(el) {
  el.dataset.color = el.dataset.color ? "" : chosenColor;
}

body { background: #eee; }

.box {
  width: 100px;
  height: 100px;
  border: 1px solid black;
  display: inline-block;
  background: white;         /* BY DEFAULT !!! */
}

[data-color=red]    { background: red; }
[data-color=blue]   { background: blue; }
[data-color=yellow] { background: yellow; }

<div class="box" onclick="pick(this)" data-color="red">1</div>
<div class="box" onclick="pick(this)" data-color="blue">2</div>
<div class="box" onclick="pick(this)" data-color="yellow">3</div>
<br><br>
<div class="box" onclick="draw(this)">4</div>
<div class="box" onclick="draw(this)">5</div>
<div class="box" onclick="draw(this)">6</div>

Answer:3

Why am I getting this error for JavaScript Date object?

I wrote the following code in JS var d = new Date(2019, 9, 14); var currentTime = d.getTime(); var daysToAdd = 3; var secondsInDay = 86400; var d = new Date(currentTime + daysToAdd*...

getting error this site can’t be reached
getting error in pom.xml
getting error
getting error while installing angular cli
getting error message on instagram
getting error in pom.xml first line
getting error while opening eclipse
getting error while installing testng in eclipse
getting error in paytm kyc
getting error in application android
getting error while setting jio tune
getting #error in ssrs
getting error not registered on network
getting error uan is not active
getting error while installing eclipse
getting error javac is not recognized
getting error while launching eclipse
getting error while transferring pf
getting error message on netflix
getting error in application

JavaScript classList.toggle() for multiple divs classlist toggle multiple classes,classlist toggle multiple,javascript classlist toggle

Why am I getting this error for JavaScript Date object?

How to show date correctly in react? [duplicate]

AngularJS+UI-Router common CRUD configuration

how can I show confirmation of what text was entered as input?