prodderman/Slider

Issues

• тесты

  #43 opened 6 years ago by GoodDayTodayOkey
  0

• Именование коллбэков

  #36 opened 6 years ago by chmnkh
  1

• Архитектура

  #26 opened 6 years ago by kinda-neat
  2

• Использовать шаблонизаторы для верстки

  #41 opened 6 years ago by GoodDayTodayOkey
  2

• Именование переменных jQuery

  #27 opened 6 years ago by chmnkh
  1

• Нарушен БЭМ

  #33 opened 6 years ago by chmnkh
  1

• попробовать перейти на keyof

  #40 opened 6 years ago by Geksanit
  0

• константы хранить в отдельном файле

  #44 opened 6 years ago by Geksanit
  0

• Исключить доступ к Dom элементам через атрибуты

  #42 opened 6 years ago by GoodDayTodayOkey
  0

• Не мутировать, переписать на тернарник

  #35 opened 6 years ago by kinda-neat
  1

• Очепятки №2

  #31 opened 6 years ago by Zarwlar
  1

• Сложный метод

  #14 opened 6 years ago by Gulyaev52
  4

• Переименовать set у VanillaSlider

  #39 opened 6 years ago by chmnkh
  1

• Разобраться с типами слайдера

  #38 opened 6 years ago by chmnkh
  1

• init и update контроллера разнести

  #37 opened 6 years ago by chmnkh
  2

• Именование параметров функций

  #30 opened 6 years ago by chmnkh
  3

• Вынести обработчики событий в отдельные функции

  #28 opened 6 years ago by chmnkh
  1

• Сделать декларативнее

  #23 opened 6 years ago by Zarwlar
  2

• setRoot не должен быть внутри if

  #32 opened 6 years ago by Zarwlar
  1

• Избыточный метод

  #34 opened 6 years ago by Geksanit
  1

• Использование id у HTML-элементов

  #29 opened 6 years ago by chmnkh
  1

• Очепятка

  #24 opened 6 years ago by Zarwlar
  1

• Создать функцию-хелпер для проверки значений на undefined

  #22 opened 6 years ago by Zarwlar
  0

• Убрать неявную конвертацию

  #25 opened 6 years ago by Zarwlar
  0

• Переменные не должны быть аббревиатурами

  #10 opened 7 years ago by Zarwlar
  2

• Использование одной и той же переменой для разных целей

  #13 opened 7 years ago by Gulyaev52
  3

• Именование тестов

  #21 opened 7 years ago by Gulyaev52
  0

• Именование функций

  #18 opened 7 years ago by Gulyaev52
  0

• Непонятное имя класса

  #19 opened 7 years ago by Gulyaev52
  2

• Время жизни переменной

  #17 opened 7 years ago by Gulyaev52
  0

• Слишком абстрактное имя типа

  #16 opened 7 years ago by Gulyaev52
  0

• Привести к единой конвенции именования переменных

  #12 opened 7 years ago by Gulyaev52
  0

• Слишком абстрактное имя переменной

  #15 opened 7 years ago by Gulyaev52
  2

• В некоторых интерфейсах все проперти необязательные

  #20 opened 7 years ago by Gulyaev52
  2

• Описать нормальный тип для функций

  #3 opened 7 years ago by Gulyaev52
  0

• Исправить типы опций

  #7 opened 7 years ago by Gulyaev52
  1

• Проверить обязательность свойств

  #4 opened 7 years ago by Gulyaev52
  1

• Описать возвращаемые значения всех функций

  #5 opened 7 years ago by Gulyaev52
  0

• Переписать логику по изменению опций

  #6 opened 7 years ago by Gulyaev52
  1

• Переименовать название класса Constructor.

  #1 opened 7 years ago by Gulyaev52
  1

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

  #9 opened 7 years ago by Zarwlar
  1

• CodeStyle

  #11 opened 7 years ago by Zarwlar
  1

• Избавиться от всех сокращение переменных.

  #2 opened 7 years ago by Gulyaev52
  0

• Использовать union для orientation и type

  #8 opened 7 years ago by Gulyaev52
  0