Скрипт под новую шапку

Ловите пятиминутку ненависти. Ну, в смысле -- критики. Вот прям конкретно распишу.))
Первые два пункта лулзов ради, остальное всерьез.)

1) Код всяко лучше админского, ей-ей, но не шибко. Иначе говоря, вы б прекрасно вписались в админский коллектив.)

2) Если уж не гнушаетесь самоиронии, то приписку "#говнокод" стоило поместить в самое начало.)

3) ES6 совершенно не годится для конечных скриптов в браузере юзера. Многие до сих пор используют старые браузеры, на секундочку. Вот сложно было через компилятор код пропустить? Понты, блин.)

4) В первых строках, в инфо-блоке для TM, читаем: "https{0,1}" вместо адекватного "https?". И тут же на конце: ".+". Зачем движок лишней проверкой нагружать? Словом, первые строки -- первый звоночек.)

5) Далее: выход из функции при наличии "cgame" в адресе. Следует добавить еще проверку на страницу выполнения квеста и на главную страницу (heroeswm.ru/).

6) "settings['isDarkTheme']". Скажите, а чем вас так смущает обычная точечная нотация, тем паче учитывая, что в отношении встроенных объектов вы используете именно ее? Непоследовательность, отсутствие стремления к порядку -- звоночек номер два.

7) Во всем коде, в однотипных, подчеркиваю, конструкциях, используется то один вид кавычек, то другой. Примерно то же можно сказать о завершающей ";": она то есть, то ее нет. Вот как бы сущая мелочь на самом-то деле, но явно же отсылает к предыдущему пункту.

• 7.1) Алсо: элементы с ID ищутся то через функцию "$", то через обычный "document.querySelector" (почему не "getElementById"? Ай-яй, о производительности не думаем). Ну и про "$" отдельно. Сразу видно, что вы ее у всяких хочух позаимствовали: иначе б расширили под любые селекторы.)

8) "classList.value". Вы это серьезно? Нет, в самом деле?))

9) "outerHTML.toString()". Данное свойство как бы и так строку возвращает.

10) "let timeCount = isPremium ? 60 : 60". Вы сделали мой день.))

11) "let destinationOffset = +3". Унарный плюс для тех, кто в бронетанке?))

12) По части стилей: вы, как и админ, напихали везде ненужных флексов. И даже банальный строчный контент, причем в блоках с ФИКСИРОВАННОЙ высотой, тоже выравниваете флексами. Ad absurdum.

13) Раз уж типа переделываете с нуля, то, Христа ради, сделайте все субменюшки частью самой шапки, удалите лишние обработчики и отображайте через CSS. Админу за это прям по голове настучать хочется.

14) Зачем, добавляя таблицу стилей, отдельным элементам вы все же прописываете стили в атрибут? Режим админа включен, называется.) Можно подумать, над кодом целый табор поработал.

15) Ну и самое главное, пожалуй: таймеры. Работа с таймерами здесь самое слабое место. Вот примеры для наглядности:

• 15.1) "let timePlace = $('hwm_time')". Почему переменная не закеширована? Вы понимаете, что каждую секунду почем зря делаете запросы к DOM? Каждую секунду, круг за кругом. Ума не приложу, где такому учат сегодня.))
  
  
• 15.2) "new Date().getTime()". Опять же: учитывая бесконечную рекурсию, это не гуд. Ведь есть же добрый производительный "Date.now()".
  
  
• 15.3) Функция "hwmDateToHuman" для ведущего нуля излишне громоздка. Оптимальное решение задачи такое: "('0' + нужное_значение).slice(-2)", безо всяких ненужных проверок.
  
  
• 15.4) Все таймеры продолжают тикать даже при нулях на счетчиках.

П.С. Я, конечно, выделываюсь, ага, но критика лишней не бывает.) Я вам явно даю понять, почему никогда не стал бы юзать подобные скрипты. Если работу админа оценивать на кол, то ваша работа -- на двоечку.

Щас бы скрипты по коду оценивать, а не по тому, что они делают

Меня никто не учил. Это не язык программирования, которым я зарабатываю деньги. И использую его только для гвдшных скриптов

Замечания ок, но скрипты я делаю не для себя, а для пользователей. Для меня главное поддерживаемость и масштабируемость. Фронтенд ненавижу, никогда не буду с ним работать в реальной жизни

Гроза_ГВД, скрипт заработал, только у меня драконы не по нижней части шапки выравнены:

И ты говорил, что включишь поиск персонажей и кланов, на данный момент его нет, либо я не там смотрю.

Гроза_ГВД

И не поспоришь.)

Учил, не учил -- камень преткновения не в этом. Вы ж в конце концов свои знания и навыки откуда-то да почерпнули. И если уж активно юзаете ES6, то элементарные вещи и подавно понимать должны. Гуру, само собой, быть не обязаны, но элементарное, элементарное понимать...

Вот есть у меня старенький ноут с ХР-шкой, так браузер на нем не обновляется уже лет сто как, потому что Chrome с ХР больше не работает. Ну так вот на энтом ноуте ваш скрипт слёту сгенерит ошибку. Да и стопицот неоптимизированных таймеров, вероятно, вызвали бы различной степени фризы.
Короче говоря, ратуя за поддерживаемость, вы меж тем не в ту сторону смотрите.

Ну, это могу понять.)

В принципе, у меня всё. Можете не отвечать.)

Sargonnas, смотри что в посте красной строчкой выделено. Не всё сразу

Мифист, Вообще не связанные понятия. Я до сих пор не полностью понимаю синтаксис джс.

Я имел в виду поддерживаемость кода

Если у человека компьютер и браузер не могут поддерживать мои скрипты, то последнее, чем нужно этому человеку заниматься, это играть в гвд

Гроза_ГВД, не забудь про включаемые всплывающие уведомления/звуки об окончании таймеров.

Динозавры должны умереть ©

Гроза_ГВД,
Напрямую не связаны, да, но и только. Если человек пишет скрипты на 1000+ строк, значит он во всем этом явно же не первый день варится. А между тем вопрос о переменных и DOM-дереве -- это, считай, вершина айсберга. И потом, переменные штука такая, которая, пожалуй, во всех ЯП работает примерно одинаково.

Сущая истина. Благо я с таких компьютеров не играю и не работаю. Пример был всего лишь намеком на то, что ваш скрипт в теории может не поддерживаться у доброй части сообщества, для которого вы, казалось бы, и пишете. На предыдущей странице вам, кстати, прямо об этом сказали: дескать, на Chrome 49 не работает. А не работает вот хотя бы и потому, что, например, "Object.entries", не единожды используемый вами в коде, пашет только с 54-ой версии того же браузера; в реальности же таких моментов намного больше, тысячи их.

Sargonnas,
Несомненно.

Мифист, не нужно бросаться в крайности. Никто в здравом уме не будет поддерживать работоспособность для хрома, который работает на виндовс хп. Я и так пошел на компромисс и не использую fetch и matchAll.

Гроза_ГВД, всё, что работает на ХР, работает не только на ней. И дело тут вовсе не в системах, дело в браузерах. Однако, если для вас крайность -- поддерживать браузер 2016-го года, тогда как в мире до сих пор еще не умер окончательной смертью IE, то мне тут остается только развести руками. Когда б админ гвд везде и всюду юзал новый синтаксис и новые возможности языка, онлайн в игре явно был бы меньше процентов эдак на 20-30.

Ну да будет, впрочем, из пустого в порожнее. Адью, что ли.)

вроде пока что неплохо.

Особенно понравились:

- темная тема,
- текст вместо кнопок.

Но скрипт пока что не использую, так как нету "3 в 1", а это один из самых часто используемых скриптов...

Кстати, при отключении графического меню, темная тема остается :)

Откройте форточку, тут слишком душно

Paveza86, 3в1 и несколько других полезностей добавлю на следующей неделе)

Таймер с секундами плохо виден. Его бы поярче.

Мифист, спасибо за первый комментарий, что быстро и понятно поправил
остальное надо думать

Гроза_ГВД, спасибо за скрипт,
драконов убрал, семиколоны убрал, на .-нотацию переставил
бум пилить

3в1 и несколько других полезностей добавлю на следующей неделе)

Гроза_ГВД,

зачем let timeCount в каждом if? обычно один раз декларируют и используют на протяжении всей ф-ции

Мифист,

без проверок, зачем всегда 0 прилеплять?

А можно сделать кликабельным значки в верхнем левом углу экрана?. Например, значок работы, значок вступления в гт. На данный момент у меня они отображаются, но переход по ним по адресу назначения не доступен - т.е. они не кликабельны

Установил HWM-UI-UX-MOD  но кнопка с меню с настройками скрипта не появилась, скрипты все отключал, браузер хром.