Currently, it is too hard to know for other modules/javascript to figure out when instantfilter filters results. Unless you want to duplicate the key handlers bound in instantfilter. Which is undesirable, because if that code ever changes, supporting modules have to change too.

At the very least, it would be awesome to have an event fired whenever the results are updated. Or, to be more complete, something similar along the lines of the autocomplete widget of jQuery UI:
http://jqueryui.com/demos/autocomplete/#events

Thoughts?

CommentFileSizeAuthor
#9 events2.patch1.78 KBKiphaas7
#1 events.patch1.4 KBKiphaas7
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Kiphaas7’s picture

Status: Active » Needs review
FileSize
1.4 KB

Something like this?

casey’s picture

Yes we need that. What if I make you a co-maintainer (I am at least until August busy with non-Drupal stuff)?

I am not expecting to much from this module though, rather I am dying to see jQuery UI 1.9 getting released; I would love to integrate its Grid in D8.

Kiphaas7’s picture

I'm afraid I also can't commit to co-maintaining at this moment. Primarly because I can't commit to regular updates/issue queue sweeps.

Sorry :(.

BTW: I used instantfilter in conjunction with a module of myself to add vertical tabs to the module listing. It provides a listing of enabled modules in a tab summary (which is updated realtime). It's kinda smart too: if the current tab does not have any search results, it automatically jumps to a tab with the most search results.

Idea is kinda stolen from: http://drupal.org/project/module_filter, but I wrote it myself before I knew about it.

klonos’s picture

Perhaps then (if you are not able to work on this on a regular basis) you should join forces and co-maintain Module filter instead and contribute as/when you can.

Kiphaas7’s picture

Well, not doing anything on this module is a waste. I have a few interesting small patches for this module, so sign me up (at least temporarily) as co-maintainer!

That is, if you still do not have any time to work on it.

casey’s picture

you're a co-maintainer now

Kiphaas7’s picture

Thanks, will post issues + patches before committing them.

Kiphaas7’s picture

Doesn't show up yet in my dashboard, but I'm guessing I'm impatient ;).

Kiphaas7’s picture

FileSize
1.78 KB

Extended the patch a bit.

Kiphaas7’s picture

Status: Needs review » Fixed
Kiphaas7’s picture

Assigned: Unassigned » Kiphaas7

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.