Вроде понял.

Вроде бы когда-то так было, не помню.

Интересно это работает.
Если this. sizeWindowToContent (true); "утащить" в начало onLoad(), то в Thunderbird с размером диалога всё будет в порядке, но в Firefox диалог настроек будет заметно выше, т. к. чекбокс ещё не будет скрыт.
Если поставить два вызова this. sizeWindowToContent (true); - в начале и в конце onLoad(), в Thunderbird будет то же самое, что и с одним в конце.

Опять двадцать пять : )
Поиск реализует редактор, а не диалог.
В редакторе с Enter'ом всё в порядке, я настаиваю : )

Не хотелось бы добавлять в контекстное меню. Подумаю.

Infocatcher

Может быть. Или Shift/Ctrl+Left click.

okkamas_knife
Это к автору кнопки.

Меня тут попросили реквест передать...

Так сказать, несколько доработанная идея.
Хорошо бы добавить в редактор кнопку «протестировать», которая бы пересоздавала кнопку без сохранения настроек с возможностью отмены.
По идее, с пересозданием кнопки проблем возникнуть не должно – достаточно пропустить сохранение файла (или обновить напрямую на основании данных редактора – лень смотреть, как там сейчас делается). С отменой сложнее. Наверное, считывать параметры из файла (мы же его не сохранили).
Как вариант – две кнопки: «протестировать» и «отменить», причем вторая становится активной только если если есть, что отменять. Разве что нехорошо, что уже есть стандартная кнопка отмены, закрывающая диалог. А две схожих кнопки как-то не особо смотрятся. Впрочем, undo и cancel.

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

Или вот при сохранении во время редактирования сохраняться во временный файл. При нажатии на Ok файл удалять, а изменения записывать в основной файл, а при отмене – тоже удалять и считывать данные из основного файла. Причем даже имена для файлов есть – что-нибудь + номер кнопки.
Но тогда нужно при запуске проверять, нет ли таких временных файлов, а если есть – предлагать открыть их на редактирование.

Вот как-то так.

Добавлено 26-01-2010 01:11:04
Мне тут подсказывают, что путано излагаю.
Если кратко, то нужна возможность посмотреть, как работает код, а потом, если не понравилось, вернуть всё назад.
Остальное – детали.

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

Добавлено 26-01-2010 01:16:55

Отредактировано Infocatcher (26-01-2010 01:15:34)

Infocatcher

Ок, закину в todo.

CB 0.0.4.7
+ Mozilla/5.0 (Windows; U; Windows NT 6.1; ru; rv:1.9.2.2pre) Gecko/20100224 Namoroka/3.6.2pre
+ тема по умолчанию

А если добавить class="menuitem-iconic", то уже нормально (добавлено только к первому пункту):

А причина – в

menuitem[id^="custombuttons-contextpopup-"] {
   -moz-binding: url("chrome://global/content/bindings/menu.xml#menuitem-iconic") !important;
}

и в

menu.menu-iconic > .menu-iconic-left,
menuitem.menuitem-iconic > .menu-iconic-left {
  -moz-appearance: menuimage;
  padding-top: 2px;
}

из chrome://global/skin/menu.css (берется из firefox/chrome/classic.jar!/skin/classic/aero/global/menu.css)

Добавлено 25-02-2010 02:10:02
И в русской локали вылезают accesskeys (в скобках после названий/меток) на английском...

Добавлено 25-02-2010 02:11:21
Скажем,

<!ENTITY editor.name "Имя:">
<!ENTITY editor.name.accesskey "N">

выводится как

Смотрится не особо.

Отредактировано Infocatcher (25-02-2010 02:19:26)

С той же стороны еще есть отсутствие прямой связи accesskey с текстом, а также отсутствие подобного в других местах.
А на MDC только про японскую локаль отступление: https://developer.mozilla.org/En/XUL_Tu … sskey_text.
Меня вот больше непривычный вид смущает.

Вроде бы, для всяких там независимостей правильнее использовать «обычные» хоткеи.

Может быть, даже есть смысл реагировать на Ctrl+Вниз и Ctrl+Вверх (или что-то вроде того) как на Tab и Shift+Tab, раз уж Tab перехватывается.
Заодно неприятная мелочь: Shift+Tab не позволяет уйти из поля для ввода картинки.

Если accesskeys на английском работают из любой раскладки, а на русском – только на русской, то это уже баг. Там вообще странно как-то – по-моему, еще есть разница, как идет русификация – изначально или в виде языкового пакета.
Вот, скажем, Alt+Ф (Файл) у меня не работает на английской раскладке. В редакторе, видимо, будет также. Тогда надо бы поискать баг на тему, а accesskeys пока оставить как есть.

Это и правда менее удобно. И все равно останутся лишние символы в скобках.

Хм, не знал, что они скрываются.
Есть только кривая идея приписать в редакторе что-нибудь вроде <label cb_accesskey="x" /> и реализовывать фокусирование на контролах самостоятельно. Плюс сделать настройку, чтобы можно было заменить cb_accesskey на accesskey. Но есть большие сомнения в целесообразности таких выкрутасов.

А как насчет принудительного отображения контекстного меню по (редкоиспользуемые модификаторы)+правоклик?
Что-нибудь вроде вот такого:

window.addEventListener("click", function(e) {
    if(e.button == 2 && e.ctrlKey && e.shiftKey && e.altKey) {
        var tar = e.originalTarget;
        if(
            "id" in tar && tar.id.indexOf("custombuttons-button") == 0
            && navigator.preference("custombuttons.mode") & 64
        ) {
            e.preventDefault();
            e.stopPropagation();
            var cm = document.getElementById("custombuttons-contextpopup");
            document.popupNode = tar.ownerDocument.popupNode = tar;
            if("openPopupAtScreen" in cm) // Firefox 3.0+
                cm.openPopupAtScreen(e.screenX, e.screenY, true /*isContextMenu*/);
            else
                cm.showPopup(e.target, e.clientX, e.clientY, "context", "none", "none");
        }
    }
}, true);

Только, пожалуй, обработчик лучше все же снимать при закрытии окна.

Тестовая кнопка:

this.oncontextmenu = this.onclick = function(e) {
    e.preventDefault();
    e.stopPropagation();
};

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

Anton
а можно ли как-то твой аддон сделать совместимым с отличным editor-ом Stylish Custom? это аддон-надстройка для основного аддона (Stylish) и в нём всё сделано адски удобно. Велосипед изобретать не за чем и можно было бы конечно даже просто скопировать его функционал, но наилучшим бы вариантом было б просто сделать эти 2 аддона совместимыми.
Вообще, с чьей стороны в данном случае надо прилагать усилия? Со стороны автора Stylish Custom или же можно как-то аддон CustomButtons подправить?
Если идея покажется тебе интересной - я готов посодействовать в разговорах и налаживании связи с автором Stylish Custom.