Dropdown

[Everrest1]

Библиотека внутренней реализацией обязывает использовать классы iqdropdown, .iqdropdown-menu и т.д. Но ты можешь добавить им и своё именование, и использовать его в своём коде. js-iqdropdown и подобные выглядят странно

[Everrest1]

Эти правки остались без внимания

https://github.com/n0sfer666/FSD-Task-2/issues/39#issuecomment-699042846 https://github.com/n0sfer666/FSD-Task-2/issues/39#issuecomment-699043125 https://github.com/n0sfer666/FSD-Task-2/issues/39#issuecomment-699044058

[Everrest1]

Почему файлы со стилями находятся вне своих блоков?

[n0sfer666]

Почему файлы со стилями находятся вне своих блоков?

Прошлый ревьювер просил

[n0sfer666]

Библиотека внутренней реализацией обязывает использовать классы iqdropdown, .iqdropdown-menu и т.д. Но ты можешь добавить им и своё именование, и использовать его в своём коде. js-iqdropdown и подобные выглядят странно

Отрефакторил (e17d9bb). Остался только один кусок .iqdropdown-item-controls который создается библиотекой (пробовал продублировать по логике мой-класс+класс-библиотеки - всё равно создаётся).

[n0sfer666]

Эти правки остались без внимания

#39 (comment) #39 (comment) #39 (comment)

Сделал

[Everrest1]

Надо разобраться в чём проблема и найти решение. Может стоит не пересоздавать инстанс библиотеки, а просто обнулять эти значения?

[n0sfer666]

Надо разобраться в чём проблема и найти решение. Может стоит не пересоздавать инстанс библиотеки, а просто обнулять эти значения?

Сделал (7d81780)

[Everrest1]

Массив results лишний здесь

[Everrest1]

Можно тернарником это условие сделать

[Everrest1]

map возвращает массив

[Everrest1]

И тут тоже лучше этим воспользоваться, чтобы было меньше мутаций

[n0sfer666]

Массив results лишний здесь

Исправил (e849c1e)

Можно тернарником это условие сделать

Исправил (150ac32)

map возвращает массив

И тут тоже лучше этим воспользоваться, чтобы было меньше мутаций

Сделал вот так (19bfee0)