История изменений
Исправление TDrive, (текущая версия) :
Был бы хоть какой-то аргумент. Типа твои «правильно написанные» тобой поделия нравятся больше. А так - никаких аргументов нет, только предложения «улучшить», без малейших оснований в необходимости этих «улучшений».
У тебя люди попросили улучшений, возможно как раз те кто должен улыбаться от твоего кода. А ты их обломал.
Напоследок еще про id напишу, если не для тебя то для окружающих.
У атрибута id есть назначение, уникальный идентификатор элемента. Твой код, допустим, будет подключаться в некий готовый сайт и фронтендерам может понадобится заюзать этот атрибут для своих целей, твоя «либа» лишает их этой возможности потому, что ты заюзал id для хранения данных инициализации. И все это типо ради поддержки старых браузеров. Что мешает передать данные инициализации в
ups_gauges = new Gauges( 'ups' )
отдельно для каждого индикатора
как ты вообще представляешь себе использование ups_gauges? через нее даже нельзя получить доступ к отдельному индикатору. Они там массивом в порядке инициализации.
Что мешает сделать по инициализации на каждый индикатор и оставить атрибуты элементов в покое?
Нужна ли вообще в new Gauges( 'ups' ) логика поиска элементов, а если у человека в его JS уже найдены все нужные элементы и он мог бы просто их передать в Gauges но твой код этого не позволяет, придется второй раз искать, такой вот оверхед на пустом месте.
Продолжать можно очень долго, такое вот у меня отсутствие аргументации.
Исходная версия TDrive, :
Был бы хоть какой-то аргумент. Типа твои «правильно написанные» тобой поделия нравятся больше. А так - никаких аргументов нет, только предложения «улучшить», без малейших оснований в необходимости этих «улучшений».
У тебя люди попросили улучшений, возможно как раз те кто должен улыбаться от твоего кода. А ты их обломал.
Напоследок еще про id напишу, если не для тебя то для окружающих.
У атрибута id есть назначение, уникальный идентификатор элемента. Твой код, допустим, будет подключаться в некий готовый сайт и фронтендерам может понадобится заюзать этот атрибут для своих целей, твоя «либа» лишает их этой возможности потому, что ты заюзал id для хранения данных инициализации. И все это типо ради поддержки старых браузеров. Что мешает передать данные инициализации в
ups_gauges = new Gauges( 'ups' )
Отдельно для каждого
как ты вообще представляешь себе использование ups_gauges? через нее даже нельзя получить доступ к отдельному индикатору. Они там массивом в порядке инициализации.
Что мешает сделать по инициализации на каждый индикатор и оставить атрибуты элементов в покое?
Нужна ли вообще в new Gauges( 'ups' ) логика поиска элементов, а если у человека в его JS уже найдены все нужные элементы и он мог бы просто их передать в Gauges но твой код этого не позволяет, придется второй раз искать, такой вот оверхед на пустом месте.
Продолжать можно очень долго, такое вот у меня отсутствие аргументации.