search

IrshakAnna-Mariia / GeekHub-2020

0 stars 0 forks source link

Домашняя работа №9 - React, drag&drop #7

Open IrshakAnna-Mariia opened 3 years ago

IrshakAnna-Mariia commented 3 years ago

Гарного дня, @redexp ! Домашню роботу виконала.

redexp commented 3 years ago

https://github.com/IrshakAnna-Mariia/GeekHub-2020/blob/363c2407b0b4b7f6453cc14d00f4295b1ee56ef3/react-slider/src/Slider.js#L4 таких переменных быть не должно. Шо за непонятные 360...

IrshakAnna-Mariia commented 3 years ago

Гарного вечора, @redexp ! Ніби переробила

redexp commented 3 years ago

min и max - это не пиксели, это абстрактные числа для выбора, они могут быть как 100 - 1000, так и 10000 - 10000000 и размер слайдера от этого не должен меняться

IrshakAnna-Mariia commented 3 years ago

@redexp ,так розмір і не змінюється від цих параметрів

IrshakAnna-Mariia commented 3 years ago

Типу, я в index.js змінюю min i max - розмір слайдерів від цього не змінюється. Чи може я щось не так зрозуміла?

redexp commented 3 years ago

меня смутило, почему ты используеш мин макс для вычисления ширины бара https://github.com/IrshakAnna-Mariia/GeekHub-2020/blob/f98f84b55c2fa6965df6aa9b9a48beb5e96a6408/react-slider/src/Slider.js#L60

IrshakAnna-Mariia commented 3 years ago

Зрозуміла. В мене була логіка, що кольорова лінія (додаткове завдання) має змінюватися від значення, що отримано. Перероблю. А щодо основного і ускладненого завдання все більш-менш нормально? Вибачаюсь, що напрягаю.

redexp commented 3 years ago

норм