js

[mglknFSD]

1. Обработчики событий необходимо выноситьв отдельный метод

2. Неиспользуемые пеменные - заменять на undescore или помечать underscore

3. Именование переменных должно быть осмысленным (я не смог понять что такое fd)

4. Где возможно следует использовать деструктуризацию

5. Именование файлов. pages это по сути те же компоненты и устроены они должны быть единообразно т е если опираться на то как устроены компоненты, инициализация должна происходить в index.js, бизнес логика - в классах.

[Rqik]

Я могу передать вызов метода в событие, таким образом? Мне в методе селектор нужен, а сам Event нет.

[mglknFSD]

Я могу передать вызов метода в событие, таким образом? Мне в методе селектор нужен, а сам Event нет.

Best practice

[Rqik]

1. Вынес все обработчики событий в методы. Использовал hoc функции, там где нельзя было обойтись без них. Использовал сам event вместо this.
2. Исправил. Fd это в самой документации к airdatepicker автор так использовал formattedDate
3. Деструктурировал эту переменную на year, month, day. В других местах пока не находил
4. Исправил логику в страницах

[mglknFSD]

1 - все ресурсы должны по максимуму кэшироваться. В данном случае каждый вызов toggleClass будет провоцировать querySelector, что не самая дешевая операция. 2 - названия классов лучше хранить в переменных.

[mglknFSD]

Зачем ты хранишь минифицированные исходники библиотек в проекте? Их нельзя установить через npm?

[Rqik]

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

[mglknFSD]

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