Z rzeczy technicznych rzucających się w oczy
- Na stronie nie powinno być więcej niż 1. element <h1>. Jak chodzi o wielkość to dostosuj ją CSSem a nie headingami
- Lepiej stylować klasami niż id
- Style inline trochę bolą w oczy
<h3>
-
<br><br>
</h3>
- Nie mam pojęcia co miałeś na myśli przez tę abominację, ale takie połączenie jest co najmniej dziwne
- Z takich mniejszych rzeczy brakuje `>` w zamykającym body
</div>
</body
</html>
- Zamiast divów lepiej sprawdziłyby się znaczniki semantyczne (znajdziesz je kawałek poniżej nagłówka)
- Charset jest chyba UTF-8 a nie
<meta charset="utf8">
Pora na CSS'a
- Brak responsywności. Poczytaj sobie czym są media queries i ogólnie o responsywności
I JavaScript
var rect = canvas.getBoundingClientRect();
var clickX = event.clientX - rect.left;
var clickY = event.clientY - rect.top;
Zamiast var lepsze byłoby tutaj `let` albo nawet `const` (Bo się nie zmienia)
canvas.addEventListener('click', function(event)
To jest ogólnie okej, możesz sobie sprawdzić funkcje strzałkowe, które idealnie się do takich callbacków nadają
A z rzeczy mniej technicznych. W nazewnictwie trzymaj się jednego języka i fajnie jak nie popełnia się błędów ortograficznych
Wielkie litery też istnieją, Po kropce takowe dajemy.
Staraj się nie powtarzać słów w zdaniach po sobie. Między tym samym słowem 1-2 zdania przerwy to takie minimum
Dodaj jednak trochę tych polskich znaków.
Nazwy języków programowania też można odmieniać
I to tak w sumie tyle z szybkiego zerknięcia
