1. Асайнваш ивент хендлърите директно в html-a.
2. Ползваш прекалено много document.getElementById.
3. Това "javascript:" във ивент хендлърите е излишно и изглежда зле.
4. Това на 97-ми ред във major.js изобщо не изглежда добре. Помисли да го изолираш във функция.
5. Тия три while-a във startGame функцията има начин да не са три а само един.

Общо взето е това на пръв поглед. Не мисли, че се заяждам нарочно или от липса на работа. Мисля, че тези неща ще ти донесат повече полза, отколкото похвали.

Господ ще ми прости греховете. Това му е работата.