Open spirithntr opened 4 years ago
Добрый день! К сожалению, только сегодня увидела письмо, как понимаю, смысла что-то исправлять уже нет. Но я посмотрю все замечания, постараюсь устранить. Только с репозиторием что-то не то. Последний итоговый вариант работы у меня находится в репозитории https://github.com/HelgaAnt/codejam-virtual-keyboard , там и PR есть, а у Вас в проверке фигурирует другой, где не последняя версия, поэтому там нет PR. Похоже, я где-то ссылки перепутала(.
Воскресенье, 19 апреля 2020, 11:42 +04:00 от Lola Bola notifications@github.com: PR-а нет. Критерий оценки: Максимальный балл за задание: 110 минимальный набор:
- реализована генерация DOM-элементов и body в index.html пустой изначально (может находится только тег script): +20
- нажатие на кнопку на физической клавиатуре подсвечивает кнопку на виртуальной (проверять следует нажатие цифр, букв, знаков препинания, backspace, del (если она присутствует), enter, shift, alt, ctrl, tab, caps lock, space, стрелки вниз-вверх-влево-вправо): +10 стандартный набор:
- есть переключение между русским и английским языком (сочетание клавиш для переключения языка должно быть указано на странице с клавиатурой), а также сохранение выбранного языка: +15
- клики мышкой по кнопкам на виртуальной клавиатуре или нажатие на кнопки физической клавиатуры, выводят символы в инпут (textarea): +15 дополнительный набор:
- реализована анимация нажатия на кнопку: +15 технические требования:
- использование в коде фишек стандарта ES6 и выше (classes, деструктуризация и т.д.): +15
- использование eslint: +10
- требования к репозиторию, коммитам и PR выполнены: +10 PR-а вообще нет штрафы:
- ошибки в консоли, связанные с исполняемым кодом (ошибки типа favicon.ico: Failed to load resource: the server responded with a status of 404 не учитываются) или предупреждения eslint-config-airbnb-base: -15 Code quality
- https://github.com/HelgaAnt/codejam-virtual-keyboard_1/blob/8c9d12f3fce459d80f37e1bc259505689f7d3dcb/script.js#L68 нужно придерживаться одного стиля создания переменных либо это var, либо это let, const
- https://github.com/HelgaAnt/codejam-virtual-keyboard_1/blob/8c9d12f3fce459d80f37e1bc259505689f7d3dcb/script.js#L93 неиспользуемый код потянется клиенту, нужно удалять
- https://github.com/HelgaAnt/codejam-virtual-keyboard_1/blob/8c9d12f3fce459d80f37e1bc259505689f7d3dcb/script.js#L155 это не класс => со строчной буквы должно начинаться название (changeKeys)
- https://github.com/HelgaAnt/codejam-virtual-keyboard_1/blob/8c9d12f3fce459d80f37e1bc259505689f7d3dcb/script.js#L157 по сути содержимое этих условий и циклов выполняется каждый раз когда вызывается changeKeys функция. Код в условиях идентичный за исключением содержимого innerHTML. Соответственно эти 4 цикла можно объединить в один и менять значение innerHTML в зависимости от условий. Summary 90/110 Можешь добавить eslint и PR в течение 3-х дней, чтобы улучшить оценку. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub , or unsubscribe .
PR-а нет.
Критерий оценки:
Максимальный балл за задание: 110
минимальный набор:
body
в index.html пустой изначально (может находится только тегscript
):+20
backspace
,del
(если она присутствует),enter
,shift
,alt
,ctrl
,tab
,caps lock
,space
, стрелки вниз-вверх-влево-вправо):+10
стандартный набор:
+15
+15
дополнительный набор:
+15
технические требования:
+15
+10
+10
PR-а вообще нетштрафы:
favicon.ico: Failed to load resource: the server responded with a status of 404
не учитываются) или предупреждения eslint-config-airbnb-base:-15
Code quality
Summary
90/110 Можешь добавить eslint и PR в течение 3-х дней, чтобы улучшить оценку.