brandon1024 / find

A find-in-page extension for Chrome and Firefox that supports regular expressions.
https://chrome.google.com/webstore/detail/find%2B-regex-find-in-page/fddffkdncgkkdjobemgbpojjeffmmofb
GNU General Public License v3.0
409 stars 53 forks source link

[please revisit - icon width] conflict with firefox reader mode hotkey / maybe add replace mode icon? #313

Closed jakobi closed 3 years ago

jakobi commented 4 years ago

Issue Description

Ctrl-Alt-R is a firefox-internal shortcut reserved for firefox reader.

Most pages do not offer a reader-reduced-cruft display, so I see some flicker on the top right as if something wants to present a large popup list, then the find-popup changes to find-replace: Correct behaviour.

What happens on a page that has an alternate "reduced-cruft" display however is this:

Page is re-rendered and changed (no flicker visible), popup remains open but can no longer match text (possibly reader-mode outputs something similar to a protected page a la mozext urls?).

Expected: Change find-popup to find-replace.

A second Ctrl-Alt-R now does change page again to original. Somehow find does see some of the hotkey and now displays find-replace, but again cannot match text.

A partial workaround would be to offer an icon to switch to replace mode (all other modes have icons).

Steps to Reproduce

brandon1024 commented 4 years ago

Hi @jakobi , thanks for your interest! Adding a find-replace icon is easy enough, and has low impact to existing users who use the ^+Alt+R shortcut. I'll implement this soon :-)

jakobi commented 3 years ago

Please see description of the bug in my comment in #331 - you seem off by the width of 1 icon, enabling icons shifts all icons and hides the start of any entered expression.