Hello,

This is probably a moot point because I realize the 2.x branch is well underway and is a considerable change from the current 1.7 release. Although the issue I am having is with 1.7, it may be something useful to consider for the 2.x branch as well, though I have, admittedly, not tested against the development branch due to uncertainty about its stability.

Basically the issue is that applying jQuery UI button to checkboxes in the module filter list breaks the functionality of module filter itself. It is an easy issue to fix, and a patch is forthcoming. It comes down to relationships defined within the jQuery selectors. The ends of the selectors that contain the direct descendant character '>' are too explicit (i.e. ... td label > strong). When jQuery UI is applied, <strong> is no longer a child of <label> but rather a descendant of it, thus making the selectors fail. The proposed patch simply removes the '>' part of the selectors.

We've been using this solution in several projects where we have a custom admin theme in place that utilizes jQuery UI button on checkboxes and it doesn't appear to negatively affect any module_filter functionality while it does allow us to utilize jQuery UI button on the modules list page.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kaidjohnson’s picture

Status: Active » Needs review
FileSize
1.16 KB

And the proposed patch...

joekrukosky’s picture

Kaid, thanks.

I'll verify Kaid's patch in #1 corrects the vertical menu issue, in my case caused by jQuery Update.

timaholt’s picture

Status: Needs review » Reviewed & tested by the community

The patch in #1 works for me as well, and fixes the issue.

kjv1611’s picture

Can someone walk me through how to apply this patch? I don't recall for sure which module I installed, may have been jQuery Update or else another related module, but ever since, this module does not work correctly. Thanks for any sort of guidance.

kjv1611’s picture

Okay, perhaps I need to go through this baby steps tutorial here:
https://drupal.org/node/620014

I was hoping I could simply find some code, and copy/paste with a text editor instead of running a script to do it. I don't mind, but I know that varies based on the way each host plan is setup.

kjv1611’s picture

I applied the patch, using this tool:
https://drupal.org/project/patch_manager

Says the patch went through successfully, but I see no difference. Having same issues still.

kjv1611’s picture

To give an idea of what I'm looking at currently, here(attached image) is how the Module Filter module is showing the modules on my site. Seems part of it works, but most of it does not - it was working great prior to JQueryUpdate and possibly a couple other modules I installed which required JQueryUpdate.

greenSkin’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Look for it in the 7.x-1.8 release.

Status: Fixed » Closed (fixed)

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