quicksearch icon indicating copy to clipboard operation
quicksearch copied to clipboard

Added support for multiple events at the same time.

Open PVince81 opened this issue 13 years ago • 2 comments

Before this fix, pasting text into a field with the mouse (right click + paste) didn't trigger the event. Some other plugins like clearableTextField (https://github.com/ono/clearable_text_field/) only trigger the "change" event which didn't refresh the search.

PVince81 avatar Jun 30 '11 15:06 PVince81

Using your patch and the latest clearable_text_field, I can paste into the selector to trigger the quicksearch. Cool! Did you have to modify the clearable_text_field to remove the quicksearch filter?

I was looking for a solution when I ran against your pull using the same components, but cannot get the clear text to reset the search.

richardjohnjensen avatar Aug 22 '11 21:08 richardjohnjensen

Hello Richard,

Thanks for merging my changes.

I didn't have to modify clearable_text_field for that. Clicking on the cross seems to trigger the quicksearch events automatically.

One problem though is that if you paste text into the field while it's empty, the clearable_text_field's icon won't appear. To fix this I made a similar change (adding events) here: https://github.com/ono/clearable_text_field/pull/3

Cheers,

Vincent

----- Mail original ----- De : richardjohnjensen [email protected] À : [email protected] Cc : Envoyé le : Lundi 22 Août 2011 23h55 Objet : Re: [quicksearch] Added support for multiple events at the same time. (#25)

Using your patch and the latest clearable_text_field, I can paste into the selector to trigger the quicksearch. Cool! Did you have to modify the clearable_text_field to remove the quicksearch filter?

I was looking for a solution when I ran against your pull using the same components, but cannot get the clear text to reset the search.

Reply to this email directly or view it on GitHub: https://github.com/riklomas/quicksearch/pull/25#issuecomment-1874943

PVince81 avatar Aug 23 '11 07:08 PVince81