Selectors clean-up: ajax.js

Rerolled for latest dev

um what about the search.api.php and search.module changes? shouldn't be in the patch.

There was a change from .attr('href') to .attr('id') it doesn't look like this will work, do you have a page where we can make sure it still works as expected?

Ah this was just a straight re-roll minus settings.php(???), so more of a statement of intent to look at it further! Will remove those changes as wasn't sure what the intention of editing search.api.php and search.module was with the original one.

Oh I see, didn't catch the changes in the previous patch. At the time there was a i18n sprint going on in the same room, there seems to have been some problems during patch generation :)

Re-rolled and fixed a few issues more.

Patch rerolled

+++ b/core/misc/ajax.js
@@ -923,26 +924,26 @@
+        $new_content.hide();

Missing update on `var new_content = new_content_wrapped.contents();`

Corrected Patch Re rolled

Finally finally rerolled....

Hey! glad to see you here Madhav :)

Patch was missing a new_content replacement and removed a }. Reroll, added some cache for an ajax command too.

Look great for me
+1 to RTBC

Perform manual testing on 8.0.4-dev. Looks good to me.
+RTBC

Hey! Theo, thanks for updating my patch!!

Committed/pushed to 8.1.x, thanks!