riklomas / quicksearch

A jQuery plugin for searching through tables, lists, etc quickly
http://www.lomalogue.com/jquery/quicksearch/
680 stars 261 forks source link

Added support for multiple events at the same time. #25

Open PVince81 opened 13 years ago

PVince81 commented 13 years ago

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.

richardjohnjensen commented 13 years ago

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.

PVince81 commented 13 years ago

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 reply@reply.github.com À : PVince81@yahoo.fr 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