Enhanced viewer for acomics.ru

Preload, navigation and other enhancements for acomics.ru comics viewer

< Feedback on Enhanced viewer for acomics.ru

Review: OK - script works, but has bugs

§
Posted: 2020-09-24
Edited: 2020-09-24

1. улучшеННый
2. разделить на языки
// @name:ru Улучшенный просмотрщик для acomics.ru
// @name Enhanced viewer for acomics.ru
3. раз уж скрипт билингва, надо идти дальше и завести раздельно @description и @description:ru
4. путь иконки может измениться, вики Гризманки советует заменять на специальный сервис гугла для поиска иконок
// @icon https://www.google.com/s2/favicons?domain=acomics.ru
5. инклуды сейчас заменяются на более безопасный матч, и можно объединить протоколы по маске *
// @match http*://acomics.ru/*
6. консоль: Uncaught ReferenceError: $ is not defined
рекомендую добавить библиотеку Жквери и проверить версию на совместимость со скриптом. пример:
// @require http://ajax.googleapis.com/ajax/libs/jquery/3.5.1/jquery.min.js
но лучше переделать на ванильный жс, т.к. сам Акомикс на Жквери ездит и возникает конфликт версий. Но на работу юзерскрипта в текущем виде он не распространяется почему-то

Sanya_ZolAuthor
§
Posted: 2020-09-26

Спасибо за отзыв!

Не знал, что можно добавлять несколько языков через @name:ru.

Скрипт писался в 2015-м, и поэтому использует несколько спорных решений, которые были нужны ради совместимости с Opera 12, и с тех пор практически не изменился. Обновления были минимальные и только для исправления багов.

возникает конфликт версий. Но на работу юзерскрипта в текущем виде он не распространяется почему-то

Конфликта версий не возникает, т.к. тело скрипта копируется в тег <script> в теле страницы, т.е. доступа к jQuery, загруженному через @require у него нет.

Я попробую внести предложенные улучшения в следующем обновлении.

Post reply

Sign in to post a reply.