JavaScript не будет отображаться в HTML - PullRequest
Новая
       5

JavaScript не будет отображаться в HTML

0 голосов
/ 23 декабря 2018

Я уже несколько дней пытаюсь сделать гравюру наброском.Я не могу найти, что попробовать дальше, так как я новичок в веб-разработке.Я поместил консольные журналы и не нашел никаких ошибок в инструментах разработки.

Я пытался разбивать другие файлы, чтобы запускать только одну функцию за раз, и почему-то мои addEventListeners не работают какони должны быть.Небольшое движение в правильном направлении очень поможет.

<!DOCTYPE html>
<html>
<head>
//<link rel="stylesheet" type="text/css" href="etchASketch.css">
 <style>
* {
    background-color: aqua;
    font-size: 16px;
}

#headers {
    height: 25%;
    position:relative;
  
}

h1 {
    font-size: 2em;
    font-family: cursive;
    color: blue;
    text-align: center;
    margin: auto;
    padding-bottom: 10px;
   
}

#instructions {
    text-align: center;
    font-size: 1.5em;
    font-family: cursive;
    margin: auto;
    padding-bottom: 15px;
    
}
#moreInstructions {
    text-align: center;
    font-family: cursive;
    font-size: 1.25em;
    
    width: 60%;
    margin: auto;
    padding-top: 15px;
}

#colorOptions {
    position: absolute;
    height: 45px;
    width: 125px;
    color: blue;
    text-align: center;
    font-family: cursive;
    font-size: 1.25em;
    cursor: pointer;
    border-radius: 20%;
    left: 25px;
    background-color: pink;
}

#btns {
   float: right;
   
    height: 200px;
    display:flex;
    flex-direction: column;
    justify-content: space-between;
    margin-right: 30px;
    
    
}

.radio {
    height: 25px;
    width: 50px;
    border-radius: 15%;
    margin: 15px;
    padding: 10px;
    display: inline-block;
    float: right;
   font-family: cursive;
   font-size:1em;
    
}

#sliderContainer {
    margin: auto;
    
    height: 75px;
    width: 400px;
    font-family: cursive;
    text-align: center;
    color: blue;
    background-color: turquoise;
}

#mySlider {
    margin: auto;
  
}

.grid {
    bottom: 0;
    height: 60%;
    width: 70%;
    margin: auto;
   border: 2px solid black;
    background-color: white;
}

.radio :active {
    background-color: grey;
}
     </style>
</head>
 <body>
    <div id="headers">
<h1 id="heading">Etch-A-Sketch</h1>
<h3 id="instructions">Choose a color and set the slider on the size of grid you prefer</h3>
<p id="moreInstructions">Move cursor over the grid to draw your masterpiece. If you make a mistake, just hit "Erase"
then move cursor over unwanted squares. Click "Draw" to continue drawing. Click "Clear Grid" to clear grid
entirely and begin a new drawing.</p>
</div>
<select id="colorOptions">
  <option value"colorChoices" selected>Colors</option>
  <option value"red">Red</option>
  <option value"pink">Pink</option>
  <option value"blue">Blue</option>
  <option value"green">Green</option>
  <option value"purple">Purple</option>
  <option value"black">Black</option>
</select>



<div id="btns">
<label class="radio">Draw</label>
<input type="radio" id="draw" class="radio" checked>

<label class="radio">Erase</label>
<input type="radio" id="erase" class="radio">

<label class="radio">Clear Grid</label>
<input type="radio" id="clearGrid" class="radio">
</div>

<div class="size">Size = <span class="out"> </span>
<input type="range" class="slider" id="myRange" min="1" max="64" value="16">

<div class="grid"></div>




<script>
    
let size = 16;
let color = 'black';

var slider = document.getElementById('myRange');
var output = document.querySelector('.out');
var clearGrid = document.getElementById('clearGrid');
let drawButton = document.getElementById('draw');
let eraseButton = document.getElementById('erase');
let select = document.getElementById('colorOptions');
let colorChoice = document.getElementById('colorOptions').value;


select.onchange = function() {
let color = this.value;
} 


drawButton.onclick = function() {
  if (this.checked == true && eraseButton.checked == false) color = color;
}

eraseButton.onclick = function() {
  if (this.checked == true) color = 'white';
  else if (this.checked == false)
  color = color;
}
 

function hoverFunc(e) {
  if (eraseButton.checked == true) {
    this.style.backgroundColor = 'white';
  } else if (drawButton.checked == true) {
   this.style.backgroundColor = color;
  }
} 
  

  
  slider.onchange = function() {
  output.innerHTML = this.value;
  size = this.value;
  buildBoard(this.value);
  }
  


function buildBoard(size) {
let grid = document.querySelector(".grid");
  

grid.style.gridTemplateRows = `repeat (${size}, 1fr)`;
grid.style.gridTemplateColumns = `repeat (${size}, 1fr)`;
for (let i = 0; i < size; i++) {
   for (let j = 0; j < size; j++) {
     let square = document.createElement('div');
     square.classList.add('cell');
     square.addEventListener('mouseover', hoverFunc);
     grid.appendChild(square);
   }
}
}



  
clearGrid.addEventListener('click', clearBoard);
function clearBoard() {
let cell = document.querySelectorAll('.cell');
cell.forEach(x => x.remove());
}








function hoverFunc(e) {
  if (eraseButton.checked == true) {
    this.style.backgroundColor = 'white';
  } else if (drawButton.checked == true) {
   this.style.backgroundColor = select.value;
  }
}


buildBoard(size);

    </script>

</body>
</html>

1 Ответ

0 голосов
/ 23 декабря 2018

Задача 1: 

<select id="colorOptions">
<option value"colorChoices" selected>Colors</option>
<option value"red">Red</option>
  <option value"pink">Pink</option>
  <option value"blue">Blue</option>
  <option value"green">Green</option>
  <option value"purple">Purple</option>
  <option value"black">Black</option>
</select>

Решение: 

    <select id="colorOptions">
  <option value="colorChoices" selected>Colors</option>
  <option value="red">Red</option>
  <option value="pink">Pink</option>
  <option value="blue">Blue</option>
  <option value="green">Green</option>
  <option value="purple">Purple</option>
  <option value="black">Black</option>
</select>

Задача 2: 

    <div id="btns">
<label class="radio">Draw</label>
<input type="radio" id="draw" class="radio" checked>

<label class="radio">Erase</label>
<input type="radio" id="erase" class="radio">

<label class="radio">Clear Grid</label>
<input type="radio" id="clearGrid" class="radio">
</div>

Решение: 

    <div id="btns">
<label class="radio">Draw</label>
<input type="radio" name="xyz" id="draw" class="radio" checked>

<label class="radio">Erase</label>
<input type="radio" name="xyz" id="erase" class="radio">

<label class="radio">Clear Grid</label>
<input type="radio" name="xyz" id="clearGrid" class="radio">
</div>

Присвойте атрибуты имени всем переключателям, и их значения должны быть одинаковыми

Сетка не имеет какой-либо высоты.Измените его с помощью встроенного css.

Для всех переменных сразу после тега script измените все let на var

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.

Нет похожих вопросов

...