Size: a a a

2020 August 19

KY

Kyrylo Yakovenko • 𝟰... in Frontend UA
Есть хак
источник

DS

Dmytro Svyrydenko in Frontend UA
Kyrylo Yakovenko • 𝟰𝟮𝟬🦄 engineer
Проблемы с тем что оно не анимируется из-за отсутствия auto?
Да, потому нужно через js быстро считать высоту блока и сетить max-height: blockHeight
источник

DS

Dmytro Svyrydenko in Frontend UA
Это вроде делается через visbility:hidden opacity: 0, считается высота, потом visibility: visible opacity 1 и max-height = calculated height
источник

DS

Dmytro Svyrydenko in Frontend UA
Просто впадлу это писать, а старый код потерял
источник

DS

Dmytro Svyrydenko in Frontend UA
А нагуглить не могу норм, везде какая-то простыня на jquery
источник

SG

Stas G in Frontend UA
ну утилиты, думаю, такой нет. а по факту там же будет 3 строчки кода
источник

SG

Stas G in Frontend UA
Ві тя
не знал что можно вообще транзишн по max-height делать
можно, в основном им и пользуются. можно и просто height, только к значению auto транзишена нет
источник
2020 August 20

Y

Yegor in Frontend UA
Dmytro Svyrydenko
А нагуглить не могу норм, везде какая-то простыня на jquery
Ну то ж в дві строки робиться
const block = document.querySelector('#huj')
block.style.height = ${block.scrollHeight}px
источник

VZ

Volodymyr Z in Frontend UA
Всім привіт!
По тасці треба показувати прогрес загрузки файлу в UI. Реалізував це через передачу callback функції у іншу функцію де виконується запит. Типу
uploadFile(id, file, callback);

В цій функції uploadFile є listener onprogress, який викликає цей callback і віддає мені дані по прогресу в основний компонент, який відмальовує далі рівень прогресу загрузки файлу. Все працює, але я хз наскільки це правильно. Можете підказати чи це коректно взагалі і зокрема чи це не порушує принципу того ж SOLID?
источник

KY

Kyrylo Yakovenko • 𝟰... in Frontend UA
не. Все правильно сделал
источник

SG

Stas G in Frontend UA
да, всё ок
источник

Дп

Джон простоДжон... in Frontend UA
Если не пытаться делать более слабой связь, вводя какие-то пабсабы и ивентэмитеры - твоё решение каноничное
источник

KY

Kyrylo Yakovenko • 𝟰... in Frontend UA
я так понимаю, что внутри функции и есть эмиттер, на который и подписывается колбек
источник

VZ

Volodymyr Z in Frontend UA
Всім дякую за відповідь! Так, всередині функції по суті еміттер
источник

Дп

Джон простоДжон... in Frontend UA
Kyrylo Yakovenko • 𝟰𝟮𝟬🦄 engineer
я так понимаю, что внутри функции и есть эмиттер, на который и подписывается колбек
ну менее связным было бы, если бы была общая шина или у компонента была возможность докидывать/убирать колбеки. Естественно, имеет свой оверхед со всех сторон, поэтому в мире dumb компонентов решается колбеком, но если вдруг прогрес показывается в статус баре + прогресс баром + в урле (или где еще) и нет стора общего/рутового компонента из которого прокидывается колбек, то придется менять
источник

KY

Kyrylo Yakovenko • 𝟰... in Frontend UA
знал, что ты пишешь примерно 🙂
источник

VZ

Volodymyr Z in Frontend UA
Джон простоДжон
ну менее связным было бы, если бы была общая шина или у компонента была возможность докидывать/убирать колбеки. Естественно, имеет свой оверхед со всех сторон, поэтому в мире dumb компонентов решается колбеком, но если вдруг прогрес показывается в статус баре + прогресс баром + в урле (или где еще) и нет стора общего/рутового компонента из которого прокидывается колбек, то придется менять
Мається на увазі, що до прогресу мають/будуть мати доступ і інші компоненти?
источник

Дп

Джон простоДжон... in Frontend UA
в некоторых сеттингах бывает нужным (приложение собирается из виджетов/плагинов, которые хотят быть связанными, но их неопределенное число/могут отсутствовать)
источник

Дп

Джон простоДжон... in Frontend UA
да
источник

VZ

Volodymyr Z in Frontend UA
Ясно) конкретно в цьому випадку немає потреби, якщо потім треба буде я перепишу, але сумніваюсь🤔

Я взагалі зараз намагаюся наперед продумувати по архітектурі чи треба в кожному конкретному випадку якийсь спільний стор і чи матимуть доступ до тих чи інших даних багато компонентів. І залежно від цього вже обирати патерн для розробки
источник