Javascript-форум (https://javascript.ru/forum/)
-   Элементы интерфейса (https://javascript.ru/forum/dom-window/)
-   -   Нужна помощь с доработкой JS, исправить 1 баг, можно даже за $ (https://javascript.ru/forum/dom-window/21317-nuzhna-pomoshh-s-dorabotkojj-js-ispravit-1-bag-mozhno-dazhe-za-%24.html)

DjDiablo 06.09.2011 13:33

Гы, он предложил практически то же самое)))

можно посмотреть на код страницы которая вызывает плагин. Есть мнение что можно решить это не меняя код плагина.

что касается плагина
тут интересна функция prePicResize и previewShown
особенно интересно почему в них такой схожий код
Если первая отвечает за маштабирование, то вторая за
маштабирование и установку img в popupWindow :lol:

ещё очень забавляет
this.popupWidth = (imgWidth / scale) +20;
тобиш вычисляется ширина окна, но нигде и никак не применяется ))))
ширина окна по сути устанавливается строкой this.popupWindow.style.height = 'auto'.
Что наталкивает на мысль о том что popupWindow меньше чем изображение
возможно this.popupWindow.style.height=this.popupHeigh+"px" ; было бы более удачным решением.


даже особенно не вникая чувствую что плагин на самом деле очень кривой.
Всё таки выложи страницу с багом целиком, мне свой тест к этому плагину писать дико обламывает)

Kanzaki 06.09.2011 14:13

http://narod.ru/disk/24212975001/add....7-fx.xpi.html
Вот сам плагин, открывается с помощью любого архиватора. Там уже найдете все скрипты и файлы настроек.
"Код страницы, которая вызывает плагин" в файле "imagepreviewOverlay.xul"

Собственно исправив код, можете сразу и проверить его у себя :)

Kanzaki 08.09.2011 21:01

Смелых пока нет :(

Gvozd 08.09.2011 23:02

Цитата:

Сообщение от Kanzaki
Смелых пока нет

наверно потому что:
Цитата:

Сообщение от Kanzaki
Конкретно готов заплатить в качестве благодарности 10$.


Kanzaki 09.09.2011 06:22

Может и так, а может и нет. Не все же тут ради денег, верно Gvozd?

melky 09.09.2011 09:51

страница с примером есть?

Gvozd 09.09.2011 14:10

Цитата:

Сообщение от Kanzaki
Не все же тут ради денег, верно Gvozd?

верно.
многим здесь, как и мне приятно просто помогать людям.
но, в вашем случае это уже не помощь(ведь автор скрипта уже помог вам с решением), а уже выполнение работы для вас.
тут уже вступают в силу законы денег, раз вы хотите чтобы кто-то сделал для вас работу.

DjDiablo 09.09.2011 17:17

Наконец дошли руки посмотреть. И тут я с удивлением обнаружил что плагин несовместим с последними версиями firefox. пробывал 6.1 и 6.2.
Кажется он устарел ещё до выпуска ))

Как его менять чтобы соответствовал я ХЗ. Как то плагины писать ни приходилось.

Kanzaki 14.09.2011 21:21

Да, оригинальная версия расширения устарела давно, но её просто адаптировать под любую версию:
Чтобы адаптировать плагин, нужно сохранить его на жестом диске, в формате xpi, открыть архиватором и в файле install.rdf в строке <em:maxVersion>x</em:maxVersion> заменить x на версию браузера, например 6.2 или 6.*
Хотя вроде та версия, которую я выкладывал уже была адаптирована, но можете попробовать вручную.


Часовой пояс GMT +3, время: 01:02.