Comments 10
Вы провели в криокамере 11 лет. Нет у нас машины не летают, но появился композитный подход, npm, бандлеры, библиотеки и фреймворками с реактивностью.
К чему вы тут мне нагрубили?
Вас так задело, что готовое решение не опубликовано в npm, и придётся потратить пару минут на то, чтобы подружить модуль с используемым бандлером?
Или вы знаете фреймворк, где функциональность, описанная в статье, уже реализована? Если так, напишите об этом сюда, будет полезно.
На счёт реактивности, готовое решение поддерживает отслеживание изменения страницы и при динамическом добавлении DOM-нод с нужным атрибутом всё "подхватится", и у ни каких фреймворков в связке с этим скриптом проблем возникнуть не должно, если это был камень в этот огород.
В конце концов, в статье приведена исчерпывающая информация для создания собственного решения, оформленного и опубликованного, как кто пожелает. Это всё-таки Хабр.
извините если это прозвучало как оскорбление. Просто в js комьюнити устаканились стандарты по оформлению библиотек. Вы сами говорите про пару минут, почему бы их не потратить. Выведите таблицу поддержки браузеров и т.д. Может быть сделать какие-нибудь обложки для популярных библиотек/фреймворков вроде реакт. Опубликовать на популярном CDN. Без негатива. Вы же написали целую статью ради этого
Почитайте что-ли чистый код на досуге
А что конкретно вам в коде не нравится, не напишите?
Названия переменных - fix, extra, current. Не выделены действия в функции с говорящими названиями. Просто адовое тернарное выражение на несколько строк. Можно было бы подробное ревью провести, но не тот формат.
Благодарю, за ответ.
На счёт названий переменных - согласен, хотя в комментариях в коде написано, что они означают.
А с тернарным выражением, разве лучше оно будет в формате if-else? Мне казалось, что наоборот, так лучше читается.
Дело не в if else, а в условиях состоящих из непонятных вычислений. И их много. Обернуть бы каждое условие в одну или несколько функций и читателю было бы понятно что за этими вычислениями кроется.
Вот, как раз момент, за который "Чистый код" обычно критикуют. Создание мелких функций, вызываемых только раз, без которых можно обойтись. Хотя в моём конкретном случае что-нибудь вроде ряда is_top(), is_left_bottom() вполне бы смотрелось, как по мне. Вообщем субъективно.
Благодарю, за конструктивную критику.
Даём возможность пользователям сайта самим менять размер блоков, потянув указателем мыши за край или угол