search

MastersAcademy / js-course-2017

Репозиторій з курсу "JavaScript" проекту Masters Academy
https://www.facebook.com/cherkasy.masters/
MIT License
9 stars 114 forks source link

add homework_8 #343

Closed MZabolotnev closed 6 years ago

MZabolotnev commented 6 years ago

done

smithua commented 6 years ago

Привет. Спасибо за работу, в целом все хорошо, работает.

Есть несколько замечаний:

  1. в CSS используется отступ 2 пробела.
  2. у нас JS курс, где мы изучаем, в начале, сам язык. По этому все домашки пишем на чистом JS, не знаю почему ты решил написать на jquery.
  3. добавь "sound.currentTime = 0;" - что бы при каждом нажатии играл звук.
  4. добавь в HTML этот мета тег, заодно почитай зачем он https://www.w3schools.com/css/css_rwd_viewport.asp
  5. JS должен подключаться перед закрывающим , иначе блокируется отрисовка страницы пока не загрузится весь JS.
  6. так же не забывай что нужно закрывать свой код от выполнения в глобальном скоупе. Для этого используй IIFE обертку.
MZabolotnev commented 6 years ago

Привет! Переделаю)

15 января 2018 г., 2:45 пользователь Aleksandr Fesenko < notifications@github.com> написал:

Привет.

  1. в CSS используется отступ 2 пробела.
  2. у нас JS курс, где мы изучаем, в начале, сам язык. По этому все домашки пишем на чистом JS, не знаю почему ты решил написать на jquery.
  3. добавь "sound.currentTime = 0;" - что бы при каждом нажатии играл звук.
  4. добавь в HTML этот мета тег, заодно почитай зачем он https://www.w3schools.com/css/css_rwd_viewport.asp https://www.w3schools.com/css/css_rwd_viewport.asp
  5. JS должен подключаться перед закрывающим , иначе блокируется отрисовка страницы пока не загрузится весь JS.
  6. так же не забывай что нужно закрывать свой код от выполнения в глобальном скоупе. Для этого используй IIFE обертку.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/MastersAcademy/js-course-2017/pull/343#issuecomment-357553125, or mute the thread https://github.com/notifications/unsubscribe-auth/AYbjQZIx6OeqpQqENf5g0JwGBnEaSorzks5tKpGfgaJpZM4RdTa5 .