Ну просто класс, спасибо тебе большое.

bunda1
а можно переделать двойной клик левой по строке поиска (сейчас это действие переключает подсветку найденных на странице вхождений) на двойной клик левой по счётчику вхождений (#hits)?
а то нынешнее поведение ломает:
1. возможность выделения в строке поиска отдельных слов (двойной клик по слову);
2. выделение по словам (если сделать двойной клик по слову и не отпуская второй клик начать вести курсор в сторону);
3. выделение по абзацам (тройной клик) [понятное дело, что строка поиска там однострочная, поэтому там выделение абзаца равняется хоткею CTRL+A, т.е. выделяет весь текст введённый в строке поиска].

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

Я тоже сначала именно об этом подумал, но двойной правый не удобен тем, что нужно гасить появление контекстного меню: потом будут проблемы с тем, чтобы его вызвать.
А в каких случаях трудно кликнуть по счётчику вхождений? Даже если там одна цифра - это уже ~ 16х16 пикселей, как маленькая иконка.

bunda1
Снова появился этот баг

если их нет, то зачем его переключать? всё равно ничего на странице не подсветит же

Dumby
Красивое решения, я не думал что можно так просто и про фреймы я тоже не подумал. Спасибо
А зачем нужно добавлять свойства для ссылок

bunda1
пока проверил на release:
1. таймер:
а. не настраивается
б. скрывает файндбар, даже если фокус в нём и я в него что-то пишу. Таймер может тикать пока я в файндбар ничего не пишу, но считаю, что пока файндбар в фокусе - он не должен скрываться, даже если таймер уже истёк. Если считаешь иначе - прошу приделать для этого настройку "запретить закрытие по таймеру, если панель поиска в фокусе", хотя бы скрытую.
2. хоткей:
а. ты поменял на двойной клик правой, а в меню это упомянуть - забыл.
б. двойной клик правой работает работает не только на поле поиска, но и на счётчике. Мне удобней дабл кликать по счётчику, но удобней всё же делать это левой - можешь ли тогда, пожалуйста, подправить на компромиссный вариант так, чтобы пускай переключение срабатывало двойным правым по полю вводу, но тогда и двойным левым по счётчику? Полное описание кликов для переключения подсветки - в таком случае не уместится в текст менюшного менюитема и его надо переименовать во что-то короткое вроде "перекл. подсветку кликами по панели поиска", а полное описание добавить в попап, появляющийся при наведении на этот менюитем.
в. при включении подсветки - у меня иконка стала переключаться не на фиолетовую, а на умолчальную CustomButtons'овую жёлтую звезду пятиконечную.
3. понравилось, что ты счётчик переделал в "x / y" правда пробелы до и после "/" считаю, что не нужны.

Отредактировано iDev.Pi (21-07-2013 14:28:30)

мне двойной клик ПКМ кажется неудобным потому, что я как юзер к нему ну совсем не привык: я не знаю ни одного другого элемента интерфейса в лисе или в любой другой программе, где бы использовался двойной ПКМ для чего-то: двойной левой - используется, а двойной правой - не-а.

так работает, но лучше бы ты эту иконку в код запрятал.

bunda1
спасибо за поправки и за двойной клик левой по счётчику (я не против двойного правого и не говорю, что раз он нигде не используется - то его не надо использовать, просто это вопрос предпочтений и уже просто исторически так сложилось, что это предпочтение одинаково для абсолютного большинства пользователей: двойной клик ПКМ не используется нигде).

Кстати, заметил ещё пару багов:
1. с выделением: не срабатывает для поисковых запросов, не содержащих букв т.е. если запрос состоит из цифр и пунктуационных знаков: 0123456789()[]\/,!@#$%^&*-=_+`~<>?'";:{}№.
2. мельчайший баг с переключением галочки "учитывать регистр": если на этой странице поискать по запросу (С то найдётся сколько-то вхождений и счётчик показывает x/y, но если переключить чекбокс, то вместо x/y показывается просто y, пока не прокрутишь до следующего/предыдущего вхождения).

И ещё у меня есть ещё один фичереквест: а можно ли всё-таки добавить настройку для управления тем, будет ли выделенной на странице автоматически попадать в поле поиска при открытии панели поиска? Сейчас есть настройка "всегда открывать панель поиска пустой", но это немного не то (эта настройка - overkill для меня): хочется, чтобы между открытиями панели запоминался последний введённый пользователем запрос. Он и сейчас сохраняется, но если перед открытием панели - на странице что-то выделено - то сохранённый запрос заменяется выделением - и это поведение лично для меня - нежелательно, поэтому хочется, чтобы оно управлялось настройкой.

p.s.: сейчас посмотрю на кнопку в Nightly (только обновлю её, а то давно не пользовался).

Отредактировано iDev.Pi (21-07-2013 17:48:31)

bunda1
глянул на Nightly: там всё обрушилось из-за переделанной DOM-структуры панели (слева - как было, справа - как стало):

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

Отредактировано iDev.Pi (21-07-2013 17:48:07)

bunda1
Извини, это я тебе зря только голову морочил: я просто забыл, что подсветка срабатывает только если вписано хотя бы 3 символа, а я вписывал 2.
Всё ок.

Таймер включается не с момента выделения, а с любого момента открытые панели, если передвинуть курсор на панели поиска таймер сбрасывается, если курсор убрать с панели поиска таймер опять включается заново.
И что такое момент бездействия пользователя ?
Кстати, попробуй http://forum.mozilla-russia.org/uploaded/Toggle Find+3 eksperiment 2.txt там по просьбе трудящихся работа таймера усовершенствована.

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

Отредактировано bunda1 (23-07-2013 19:05:41)

// Переключить поиск на странице двойным средним кликом на странице ................................
addEventListener("mouseup", function(e) {
  if ( e.button !== 1 ) return;
  if ( !cbu.getPrefs("FindBar.ToggleWithDoubleMiddleClick") ) return;
       var _this = this;
       
       if ( _this.open ) { 
            gFindBar.hidden ? gFindBar.onFindCommand() : gFindBar.close(); 
            Find(e);
            gFindBar._findField.focus();
            };
            
       _this.open = true;
       
       setTimeout(function() { _this.open = false }, 600); // сбросить флаг по таймеру
}, false, gBrowser );

Поменяй в if ( e.button !== 1 ) return; 1 на 2