sgadmer / travel-agency Goto Github PK

 <li city="arr_ANC_hotels" class="li_anc">Анкоридж</li>

Если ты добавляешь в разметку новые атрибуты для своих целей, то должен исключить вариант их пересечения с основными зарезервированными
как пример

 <li data-city="arr_ANC_hotels" class="li_anc">Анкоридж</li>

ta - travel-agency

 <li ta-city="arr_ANC_hotels" class="li_anc">Анкоридж</li>

Попробуй разбить css на отдельные файлы инструментом вроде этого https://github.com/mariocasciaro/gulp-concat-css

У тебя сейчас 1 большой. Если будет вида

1. nav.css
2. form.css
3. ....
4. footer.css

И одна тока входа, где ты подключишь все блоки main.css

Использует required поля. Тогда подсказывай пользователю, где он ошибся. Подсвечивай поля, которые ты считаешь обяз и были пропущены пользователем на отправку формы

Определись форматом
https://github.com/Sgadmer/travel-agency/blob/master/src/scripts/main.js#L121-L127

currentSlide, plus_Slides, position_btn_dots_slider, scroll__percent__up, zoom_3 используй какой-то 1 стиль.
currentSlide - самый норм

Посмотри как можно организовать стили, что бы не было пересечений и конфликтов
БЭМ

https://code.jquery.com/

Ты можешь подключить jQuery из офиц хранилища

<script src="https://code.jquery.com/jquery-3.5.0.min.js" integrity="sha256-xNzN2a4ltkB44Mc/Jz3pT4iU1cmeR0FkXs4pru/JxaQ=" crossorigin="anonymous"></script>

https://github.com/Sgadmer/travel-agency/blob/master/index.html#L9-L10