Testing taxonomy manager from drupa/project/taxonomy_manager works great adding and reading several levels, but not for me on either FF or IE. I can add, delete, move terms up and down, but with several layers, I cannot click the + button to enter the next level? I know it works other places, but on my computer it doesn't. This has been tested on both IE6 and FF2. I have tried to reset settings (as I know this great tool utilized javascript) and clean the cache, but nothing works so far! Please help with some advice...

Comments

mh86’s picture

mh86 CreditAttribution: mh86 commented
Status: Active » Fixed

hi!
With many levels and many terms the Taxonomy Manager tends to get a bit slow because of so many JS features. If your computer is not the fastest this can cause problems in FF and IE. But I know that Opera has a much faster JS engine. So if you are still having performance problems, try Opera.
I hope I understood your problem and this helps.

matthias

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)

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

VVN’s picture

VVN CreditAttribution: VVN commented
Status: Closed (fixed) » Postponed (maintainer needs more info)

same problem, but on demo site + buttons work, on mine not.
and my navigation menu (with nice_menus) don't appear on /admin/content/taxonomy_manager/tid
please help me, this module is usfeul!

BoarK’s picture

BoarK CreditAttribution: BoarK commented

AIRUS,

I believe that the problem stems from incompatible Jquery.js versions. I upgraded my jquery.js in order for the some to the functionalities of Jquery UI to work and this has broken Taxonomy Manager in the same manner. I came here to find a fix, but none was found. Too bad the module didn't throw any errors in firebug or else I would have an idea of where to start fixing it.

UPDATE:

I'm using jquery 1.2.1. Here's the fix:

in the file trees.js change:

Drupal.attachTreeview = function(ul) {
  $(ul)
    .addClass("treeview")
    .find("li[ul]").prepend("<div class='hitArea'/>").find("ul").hide().end()
    .find("div.hitArea").click(function() {
      Drupal.toggleTree(this);
    });
}

to: 

Drupal.attachTreeview = function(ul) {
  $(ul)
    .addClass("treeview")
    .find("li:has(ul)").prepend("<div class='hitArea'/>").find("ul").hide().end()
    .find("div.hitArea").click(function() {
      Drupal.toggleTree(this);
    });
}

i.e. li[ul] -> li:has(ul)

It should work.

Nathan.

VVN’s picture

VVN CreditAttribution: VVN commented

Brilliant, it works!
Thanks a lot!

mh86’s picture

mh86 CreditAttribution: mh86 commented

if I change they li[ul] -> li:has(ul), does it still work with the original jquery version?

BoarK’s picture

BoarK CreditAttribution: BoarK commented

mh86,

The li[ul] -> li:has(ul) change was made when Jquery migrated to version 1.2. Taxonomy Manager works fine with the Jquery that comes with Drupal 5.

Nathan.

Macronomicus’s picture

Macronomicus CreditAttribution: Macronomicus commented

It stops all the errors I was getting but I can no longer click the little expand box to show child terms.... Strange...

BoarK’s picture

BoarK CreditAttribution: BoarK commented

Macrocosm,

There are a few other changes that need to be made.

i.e. Replace .gt(n) with .slice(n+1). 

Example: change $('li').gt(2) to $('li:gt(2)') or $('li').slice(3)

Here's a link to a guide for upgrading to Jquery 1.2 Upgrade

Nathan.

ivnish’s picture

ivnish CreditAttribution: ivnish commented
Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (outdated)