gamabuntat / super-slider Goto Github PK

Архитектура проекта по заданию должна MVP. Почему у тебя Model называется Service. Хотя внутри Service ты вызываешь методы с model

В требованиях описано, что на выходе должна получиться MPV архитектура с пассивной вьюхой. Вьюха вообще ни коим образом не должна знать о Модели, а все взаимодействия между вью и моделью должны быть выполнены через контроллер с ивент эмиттером.
У тебя вью принимает модель из-за чего можно спокойно обратиться к любому свойству или методу модели напрямую, что нарушает инкапсуляцию и увеличивает сцепление слоев.

Сейчас в проекте нет фавайкона и присутствуют не все метатеги из стандартов.

Также не всегда выполняется ЭТОТ пункт стандартов. (в View.ts констуркторе, например)

Сейчас не везде выполняется вот этот пункт стандартов, как минимум именование фонтов не соответсвует

Не во всех местах используется js- префикс для поиска по DOM.

Сейчас в проекте не соблюдаются многие BEM правила, этот проект тоже должен соответствовать всем BEM-у :)
И многие стилистические элементы отличаются от первого проекта, например для модификаторов тут используется double dash вместо underscore.

Какие-то тесты не проходят проверку посмотри пожалуйста

У interval слайдера не должно быть возможности выбрать одинаковое значение для двух поинтеров. Иначе получается, что у Нас есть step, но при этом значения не различаются, да и 'интервала' в таком случае никакого не будет.

Сейчас с помощью не совсем валидных значений - можно просто крашнуть приложение :)

Также значения при переключении опций не должны сбрасываться, сейчас при переключении интервала второе значение всегда сбрасывается на максимальное.

В панели не должно быть не валидных значений:

Также сейчас есть какие-то не совсем очевидные взаимодействия между опциями:
Как выбор интервала влияет на шкалу, какое расположение элементов шкалы верное?

Надо поправить тесты, они начали фейлится

Добавь в зависимости pug

1. По бп нельзя делать неявные преобразования
   
   
   почитать можно тут

Опечатался

2. Импорты сортируем по след правилам тык
Пройдись еще раз по коду по бп

1. По клику на слайдер позунок должен двигаться

2. Интервалы под шкалой визуально не совпадают
   
3. Шаг влияет на Max значение. Не могу изменить Max значение если шаг больше
   https://user-images.githubusercontent.com/89850975/132414927-c4683ae5-e063-4fbb-964a-2befe9e6fbc8.mp4
4. Ползунки не перемещаются при вводе через input-ы

5. Кажется я сломал)
   https://user-images.githubusercontent.com/89850975/132415751-33cadbff-e0c7-4f74-b218-0e5b58746c29.mp4

У тебя рендерится слишком много элементов которые не отображаются. Надо сделать обработку перед рендером. Чтобы попросту не создавать элементы

Опиши способ подключения, через дефолтный селектор без инициализации в js.
Опиши как можно передать параметры через data-* атрибут.
То что start tests $ npm test понятно из названия, опиши что конкретно делает каждая команда. (P.s. Пример npm run dev
Запускает development сборку и сохраняет результат в папку /dist.)