Ocena kodu [HTML,CSS,JS]

Question

Witam ;)

Przed chwilą skończyłem dopracowywanie gry(znana jako kółko i krzyżyk). Napisałem ją ze względu na to, że w JSie stawiam swoje pierwsze kroki i chciałem się sprawdzić.

Teraz chciałbym się dowiedzieć od Was, co sądzicie o jej kodzie - myślę, że nie jest on nawet dobry, ale to jak zawsze z początku ;). Wiem, że strona nie jest responsywna, myślę też że nie do końca dobrze zastosowałem tagi do umieszczania elementów. Nie jestem pewny także nazewnictwa klas z użyciem myślników.

Link do samej gierki: [KLIK]

Link do kodu: [KLIK]

Dziękuje za opinie z góry i pozdrawiam :).

Answer 1

Brakuje przycisku "Restart".

var tytul = document.getElementById("duzy").innerHTML;
tytul = tytul.toUpperCase();
document.getElementById("duzy").innerHTML=tytul;
//to było trudne XD

text-transform: uppercase w CSS.

Zamiast nadawania wszystkiemu [onclick] warto zastosować po prostu event delegation – wówczas wszystkimi zdarzeniami na planszy będziesz kierował z jednego miejsca.

No i przede wszystkim: gry nie mogą być w globalnym scope, bo to pozwala na oszukiwanie ;)

Comments

Wiem o takiej właściwości, chciałem po prostu zrobić to JSie. Dzięki za opinie.

Answer 2

jestem też takim początkujacym ale rzuca mi się w oczy jedna rzecz, mianowicie

<aside id="description" class="description-css">

po co używasz diva i klasy na jednym elemencie? Po co? nie lepiej napisać

<aside class="description">?

Comments

Myślę że chodzi Ci o id, a nie diva. Poprzez id odnoszę się do rzeczy w JSie, a w klasach styluję element. Przy klasach miałbym też utrudniony dostęp do rzeczy, mógłbym zapomnieć dopisać [0] i trudno byłoby się połapać czemu jest błąd.

---

dzięki,ogarnąłem mniej więcej po co to :P

Answer 3

Taka uwaga, że u mnie na Firefoksie, nie wyświetla się
http://pasteboard.co/1yHi59yr.png

Comments

Obstawiam, że nie masz czcionki Consolas ani Segoe UI - zaraz dodam coś od google'a. Dzięki opinię.

---

Możesz spróbować teraz?

---

Nadal to samo.

---

Poprawione - okazuje się, że Mozilla nie obsługuje innerText.