В версии 0.0.1.3 я сделал, чтобы при нажатии на кнопку проверялось, есть ли метод bind у Function, и если нет -- то он заново инициализируется.

Отредактировано Yan (05-12-2006 16:19:00)

Dark-Demon
Если честно -- влом даже разбираться, т.к. в 2.0.0.1 всё равно поправят, и я этот хак из след. версий уберу.
Ты бы лучше другие баги ковырял, коих немало. Например, добавил я в user.js:

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

что-то addons.mozilla.org не хочет кушать моё расширение
а когда выходит версия 2.0.0.1?

Yan, strict никак не мешает работе, просто засоряет консоль.

Подниму-ка я тему.

Во-первых, нужна ли вообще настройка на скрытие кнопки «применить» в редакторе? Пусть она всегда видна будет, не мешает же.
Во-вторых, чисто визуально лучше смотрится, когда после нажатия видно, что «сохранилось» – т.е. кнопка становится неактивной.

Относительно универсальная схема примерно такая:

<dialog
onchange="changed(event);"
oncommand="changed(event);"
oninput="changed(event);"
ondragexit="changed(event);"

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

Далее два варианта.
1. Просто проверять event.target.localName и если, например, это не "tab", включать кнопку.
2. При каждой загрузке настроек и при каждом сохранении пробегать по всем (сохраняемым) элементам, и записывать их состояние (которое уже сохранено) в атрибут:
checked="true" -> saved_checked="true"
value="qqq" (точнее, свойство value) -> saved_value="qqq"
А в changed(event), соответственно, проверять, изменилось ли.

Самый точный вариант, но и самый медленный, если может быть большой текст (а он может).

Ну, и преимущество в том, что код мало зависит от структуры XUL и от того, как реализовано чтение/сохранение настроек.

P.S. И все же «Показывать кнопку "Применить" в редакторе кода», а не «Отобразить кнопку "Применить" в редакторе кода». Последнее больше применимо к однократному действию.

В основном сохранение нужно на случай, если выполненный по F9 код тем или иным образом не даст сохраниться.
А инициализацию можно запускать только если окно редактора теряет фокус.

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

Это не баг.
Ответственная за это функция предназначена для установки комментариев в начало строки и снятия их оттуда же.
Комментарии внутри строки имеют приоритет.

http://forum.mozilla-russia.org/viewtop … 82#p364082
https://www.mozdev.org/bugs/show_bug.cgi?id=21391

Что вообще дает добавление оверлея через свой протокол кроме запрета кэширования?
Вроде как, там проблема в том, что компонента запускается позже, чем читается chrome.manifest. Так вот, может, есть смысл делать document.loadOverlay() после load/DOMContentLoaded окна или просто из подключенного к обычному оверлею скрипта?

Добавлено 13-08-2009 23:09:01
Хм, как-то странно оно себя ведет. По-моему, что-то не так с компонентой, реализующей протокол.

Кроме версий js это сейчас для чего-нибудь используется?

А когда подключается toolbarpalette?

Ммм... она не работает.
А если серьезно, не знаю. Или что-то глобально не так с оверлеями по своему протоколу, раз идут глюки из-за того, что другое расширение меняет момент загрузки своих скриптов, или что-то не так в реализации протокола. Или баг в Firefox, наконец.
И можно ведь у автора ABP спросить, глядишь, присоветует чего.

В том смысле, что есть планы использовать изменение контента, загружаемого по своему протоколу, для чего-то еще или просто может пригодиться?
И я не совсем понял, как проявляется баг, из-за которого нужно «руками» указывать версию js. Т.е. нельзя ли решить эту проблему не через одно место?

До того, как запустятся скрипты расширений, так?
Тут, по идее, может помочь компонента навроде userChromeJS, если, конечно, есть подходящее событие, по которому можно запускаться.

Т.е. это баг в реализации chrome.manifest?

Тебе видней. Тем не менее, не стоило это исключать без веских оснований, а у меня их не было.

Не совсем понимаю, что дает resource://, он что, тоже не кэшируется или можно сделать, чтобы не кэшировался?

Однако я могу подключить скрипт без указания версии js, содержащий

let x = 5;
alert(x);

, и он будет работать.
И

new Function("let x = 5; alert(x);")();

будет работать.
Нельзя ли переместить методы для выполнения кода кнопок туда, откуда они будут работать без уточнения версии?

Если оверлей через resource:// решает проблему, нет смысла и проверять, понятно, что так сложнее.
Только можно ли при этом будет поставить CB в виде глобального расширения? Или будет использоваться что-то хитрее, чем
resource custombuttons-profile ../../custombuttons/
?
Хотя нет, еще же где-то должна определиться версия js.

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

Я сказал, что «по-моему». К тому же, проявлялось все это довольно-таки странно. Например, у меня почему-то вываливалось исключение на document.loadOverlay() при использовании custombutton-протокола и при том, что в chrome.manifest все строки с custombutton:// были закомментированы.

Н-ну, пусть xbl оповещает наблюдателя, что нужно выполнить код такой-то кнопки.

Это уже вопрос терминологии. Если что-то не работает, то или так и задумано, или это баг.

Тот код не сохранился, потому как не давал нужного результата...
И еще может влиять наличие/отсутствие/установка ABP.

Пока пытался воспроизвести, убил все расширения:

## Firefox

overlay chrome://browser/content/browser.xul chrome://custombuttons/content/test.xul

overlay chrome://browser/content/browser.xul chrome://custombuttons/content/overlay.xul
#   overlay chrome://browser/content/browser.xul custombuttons://content/buttonsoverlay.xul

## Browser
#   overlay chrome://browser/content/browser.xul custombuttons://content/cbbutton.xul
overlay chrome://global/content/customizeToolbar.xul custombuttons://content/cbbutton.xul

test.xul:

<?xml version="1.0"?>
<overlay xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
	<script type="application/x-javascript">
	<![CDATA[

	document.loadOverlay("custombuttons://content/buttonsoverlay.xul", null);
	document.loadOverlay("custombuttons://content/cbbutton.xul", null);

	]]>
	</script>
</overlay>

А

window.addEventListener("load", function(e) {
	document.loadOverlay("custombuttons://content/buttonsoverlay.xul", null);
	document.loadOverlay("custombuttons://content/cbbutton.xul", null);
}, false);

тупо не дает никакого эффекта.

Правда, это все без установленного ABP.

А при установке ABP плюс

setTimeout(function() {
	document.loadOverlay("custombuttons://content/buttonsoverlay.xul", null);
	document.loadOverlay("custombuttons://content/cbbutton.xul", null);
}, 1000);

Firefox вообще падает при запуске. о_О
Закомментировал setTimeout и далее, тогда запустился.

Заодно нашел в userChromeJS коммент про https://bugzilla.mozilla.org/show_bug.cgi?id=330458.

Отредактировано Infocatcher (15-08-2009 03:14:21)