Standardized filter/search behavior; surface filter matches regardless of their containing box's open state;

There are code comments and incorrect code implying this was always intended. It's just that .show() doesn't work for details.

That is slick. I manually tested, it works great.

+1 RTBC

+1 as well

Nice :)

I just tested the steps in the OP on simplytest.me (after testing locally and seeing the same thing) on the modules page, but didn't see the reported behaviour. If I collapse "core" and search for "actions" it still says "No results found." Did I do something wrong?

This is the link I clicked to test: http://simplytest.me/project/drupal/8.x?patch[]=https://drupal.org/files...

I'm surprised that happened with simplytest, but your browser could still be caching the old JS.
If you're using Chrome, the web dev tools (console, firebug, whatever you call it) has an option to disable the JS cache

When doing this, the patch works as intended.

Ok, to clarify what I was seeing, this behaviour is broken in Firefox (using latest 23.0 release on OS X). It works fine in Safari in Chrome, though.

problems on FF were because of the collapsible polyfill kind of thing we're using for details. It uses jQuery to set style="display:none" or block instead of using the open attribute like it's supposed to.

patch gets rid of the animations for details and use open to say what's visible or not when it's useful.

forgot to remove a line.

Awesome, that fixed my problem! Can't really speak to the JS changes though, so leaving at needs review.

Hm, still a bit of weirdness. I'll try to explain.

1. Collapse "Core"
2. Mis-type "actions" as "acrions".
3. All of the stuff below the search box goes away (as expected; there's no "acrions" module).
4. Backspace to just "ac" in order to correct the typo.

Expected: Core + "Actions" should appear below the search box. Actual: nothing happens.

Interestingly, if you backspace all the way back to just "a", things start appearing in the table again. Weird.

I'm not 100% sure if this is coming from this patch though or a pre-existing behaviour, so not marking "needs work" yet.

fixed the undocumented "feature" from #14.

Attached a quick & dirty interdiff, sue me :p

Postponed #2056089: UI problems on the Modules/Extend page on this one.

#15: core-js-module-search-2060573-15.patch queued for re-testing.

#15: core-js-module-search-2060573-15.patch queued for re-testing.

8 months.. Not surprisingly it doesn't seem to want to apply.

Re-roll of #19.

small nitpick, indent missing from .removeAttr() and .attr() other than that it's good to go, thanks!

Indent fixed. Thanks

Made the original patch and all but we're starved for reviewers so RTBC. Still works as intended.

( edit ) apologies, I didn't make the original patch, tim and jesse did.

Committed 0a66db1 and pushed to 8.x. Thanks!