Обработка расширения DJVU, написанного заглавными буквами

RussCoder / djvujs

DjVu.js is a program library for working with .djvu files online without any connection with the server. DjVu.js Viewer is a widget that allows viewing .djvu files right in the browser and can be easily embedded into any web page.

https://djvu.js.org/

Other

182 stars 36 forks source link

Обработка расширения DJVU, написанного заглавными буквами #72

Closed refringerator closed 3 years ago

refringerator commented 3 years ago

Добавлена обработка урлов, заканчивающихся на djvu, написанного заглавными буквами

RussCoder commented 3 years ago

Проблема действительно есть. Только этих правок недостаточно, чтобы ссылки открывались по клику (только пункт в контекстном меню отобразится). Как я понял, код вы не тестировали, а без этого никак. Там в requestInterceptor есть регулярное выражение, которому надо i флаг добавить, чтобы заработало. Да и по хорошему, надо бы все паттерны дублировать, в том числе и file://. Есть желание доделать или мне это все сделать самому?

Я сегодня/завтра планирую релиз, поэтому стоит быстрее определиться.

refringerator commented 3 years ago

Желание доделать есть, пока нет времени разбораться. Поэтому было бы здорово, если сами по хорошему доделаете. Протестировал локально, в фаервоксе через отладку расширений, проблемный кейс отработал.

RussCoder commented 3 years ago

В Firefox у меня также не работает (именно при клике по ссылке, при отключенном анализе заголовков). Но да ладно. Я завтра сам все сделаю и протестирую все случаи, включая открытие файлов. Сюда дам ссылку на коммит с изменениями, после чего PR закрою.

RussCoder commented 3 years ago

Проблема устранена тут https://github.com/RussCoder/djvujs/commit/2cc03ac38eab244c5a8fd7ba75ddf2dd8442ae74