search

MoisKsena / epic-book-store

https://moisksena.github.io/epic-book-store/
0 stars 0 forks source link

Проверка #1

Open nicothin opened 5 years ago

nicothin commented 5 years ago

https://github.com/epixx/criteries/blob/master/criteries.md

Общее:

Ошибки:

nicothin commented 5 years ago

Критерии:

...

Так. Я жду от вас коррекции ошибок, перечисленных в первом моём сообщении. Там в самом начале ссылка на критерии, а далее указано название критерия и, если нужно, пояснение его невыполнения. Пример:

image

nicothin commented 5 years ago

Уже довольно хорошо. Лимит проверок исчерпан, но я готов проверить еще 1 раз, ибо это состояние проекта очень близко к тому, которое меня устроит )) Проверить в последний раз смогу в СББ вечером, около 19:00. Дайте отмашку о проверке в ЛС вконтакте.

  1. https://monosnap.com/file/2sv3ZK4bM3MeKHgT2fxE9GzXTAM6sK — гор. скролл на 1280, отсутствуют карточки книг, не работает слайдер (вероятно, из-за смены js-catalog__list на js-catalog, так что в JS такую замену тоже нужно сделать).
  2. модальные элементы (модалка превью страниц книги, попапы результата отправки формы) нужно центрировать на странице, любой заказчик попросит об этом (используйте там position: fixed)
  3. все крестики, закрывающие модальные элементы, — это кнопки, а не дивы.

P04. https://monosnap.com/file/B9IrFWdcJCB9zvsdgVDwA4j9uLhI88 Р07. Не исправлено. Клик на тексте (который не ссылка) чекбокса не приводит к смене состояния чекбокса. Чтобы приводил, нужно поместить текст в тег label, связанный с инпутом чекбокса, например по id и for. С03. гор. скролл, упомянутый выше, внешний вид чекбокса. С04. ошибочное состояние полей: это СОСТОЯНИЕ ПОЛЕЙ, все элементы, показывающие, что поле заполнено ошибочно, должны быть написаны в DOM рядом с полем, а не в отдельном диве. Сделайте первое поле и чекбокс в ошибочном состоянии (допишите там модификаторы --error на нужные классы (по одному на поле) и добавьте разметку сообщений об ошибках). С09. https://monosnap.com/file/QxGwBTV6mW2Mj2HjydWIiGSTFi5vHl и аналогично для стр. рекомендаций — там блоки с фоновой картинкой по-прежнему не имеют указанного фонового цвета и текст почти нечитаем. С10, C12 не исправлено. В файлах main-nav.scss, bookinner.scss, newitems.scss, swiper.scss — сторонние блоки. В файле main-nav.scss в корневом контексте может быть только ондоимённый селектор .main-nav, но не может быть .linkactive, .close и к-л. ещё, а они там есть. Аналогично — для bookinner.scss, newitems.scss, swiper.scss. П03. https://monosnap.com/file/3y3e0EljtEzd5TZImxJkv5YmAM0eOf — формально покатит, но ни один заказчик не порадуется, увидев такое. https://monosnap.com/file/CRVm0YjwM7P0fa6ZhGYI4JGUm03hMu — нет внутненнего ораничителя и фиксирована высота, поэтому так. П04. оно теперь всегда раскрыто :( плюс крест странный — кликабельны лишь «палки» креста, а не его габарит. Меню должно быть скрыто при включённом JS, но раскрыто в случае, если JS выключен. П05. как было, так и есть: к главной подключен не один файл, а два. Нужно взять стили слайдера в сборку своего css-файла. П09. Не исправлено.

nicothin commented 5 years ago

https://monosnap.com/file/jZrFhMHSyBVFom2dgKEXMRa6XI16SL — причина гор. скролла Р04 не исправлено Р07 не исправлено, чекбокс теперь не работает вовсе С03 не испроавлено C04 не исправлено или не показано. зато в стилях есть questions__checkbox-error :( типа, если ошибка, это отдельный элемент должен быть? тогда существенно сложнее (чем класс добавить) перевести чекбокс из состояния ошибки в состояние «нет ошибки». С10, C12 не исправлено. П03 не скорректировано П04 не исправлено