viklysx / first_project

first_project

Учебный проект, выполненный по макету на курсе в Skillfactory.Ссылка: https://viklysx.github.io/first_project/

first_project's Issues

Layout + styles

1. https://github.com/Viklysx/testrep/blob/588e806661f3d063ab0320f65d554e0326a4b45f/styles/style.css#L122
   Вот этот стиль перебивает стиль основного контейнера и у вас слетает центровка блока.

2. https://github.com/Viklysx/testrep/blob/588e806661f3d063ab0320f65d554e0326a4b45f/index.html#L45
   Хорошее переиспользование стилей, но sub - неприемлемое имя для класса. Совершенно непонятно что оно означает. Лучше всего обозначить как btn-transparent или типа того

3. https://github.com/Viklysx/testrep/blob/588e806661f3d063ab0320f65d554e0326a4b45f/index.html#L47-L51
   Следите за форматированием кода.

4. https://github.com/Viklysx/testrep/blob/588e806661f3d063ab0320f65d554e0326a4b45f/index.html#L27
   Сделайте телефон ссылкой

Slider

1. https://github.com/Viklysx/testrep/blob/343f0f6266fc2678e77fa60dcaee6d846247e11d/js/slider.js#L59
   нужно classList.remove('active')
   https://github.com/Viklysx/testrep/blob/343f0f6266fc2678e77fa60dcaee6d846247e11d/js/slider.js#L66
   нужно classList.add('active')

2. https://github.com/Viklysx/testrep/blob/343f0f6266fc2678e77fa60dcaee6d846247e11d/js/slider.js#L64
   Нужно убрать эти проверки и передавать в функцию index вместо id, т.е.

slide(index) {
  sld_1.src = variants[index].img;
  slides[index].classList.add("active");
  ...
}