-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Remember last searched phrase & instant search paste fix. #11869
Conversation
… last searched phrase
|
Should the
a) "Hot dogs" (persisting searches) I would be partial for all searches persisting (a)) There's another variation of the previous where it's even more mixed up:
👍 for this feature/fix otherwise |
|
Thanks @petetnt for reviewing. |
|
Tested against 30716f4 again and it fixes everything I mentioned in #11869 (comment) 👍 LGTM as it is. Here's some food for though: should it remember the position of your search too? Consider something:
Search for |
|
I prefer the current implementation, the search progressing closest to the current cursor. That might be individual preference. |
|
I have verified the changes. Looks good to me. |
Remember last searched phrase & instant search paste fix.
|
🆒 |
|
Thanks all :) |
fix: instant search doesn't work after ctrl+v on textfield
Remember last searched phrase #11735