Hello,

I've attached two patches to add this function:

When you display the advanced taxonomy blocks on a page where a node is loaded, the terms (in the blocks) which are related to the terms of the node are getting a class. So you can 'highlight' such terms when you define a layout for that class with css.

In order to be able to add such an extra attribute, I wrote also a patch for the jquerymenu module to add dynamically attributes to the $options variable.

greetz,
Wim

CommentFileSizeAuthor
#3 jquerymenu.dynamic.attributes.patch1.22 KBCauliflower
#3 taxonomyblocks.highlight.patch2.76 KBCauliflower
#1 taxonomyblocks.highlight.patch1.41 KBCauliflower
#1 jquerymenu.dynamic.attributes.patch991 bytesCauliflower
jquerymenu.dynamic.attributes.patch991 bytesCauliflower
taxonomyblocks.highlight.patch1.43 KBCauliflower

Comments

Cauliflower’s picture

Cauliflower commented
StatusFileSize
new jquerymenu.dynamic.attributes.patch991 bytes
new taxonomyblocks.highlight.patch1.41 KB

It's better to use the taxonomy_node_get_terms() function to get the terms of a node.

PixelClever’s picture

PixelClever commented

Does this patch apply to version 1.8? I had been working on the code on my testing server fixing some bugs (which I give first priority). Your patch seems useful though. If you and other users have time to test it and verify it I am open to putting it in. I personally don't have much time for testing right at the moment.

Cauliflower’s picture

Cauliflower commented
StatusFileSize
new taxonomyblocks.highlight.patch2.76 KB
new jquerymenu.dynamic.attributes.patch1.22 KB

Hello,

I remade both patches for the last version from the cvs,

greetz,
Wim

PixelClever’s picture

PixelClever commented

The jquerymenu patch is commited in version 2.0 as for the taxonomyblocks patch I think I am going to have to create a dev release for this one in the next week since there a couple of pending patches and I don't have enough time to sufficiently test.

PixelClever’s picture

PixelClever commented
Status: Needs review » Needs work

I looked at this patch and there is no option to turn off the highlighting functionality. I think this should be set as off by default and be turned on only if the admin wants it on. Otherwise you are using cpu and slowing down the page even if it isn't desired.

summit’s picture

summit commented

Subscribing, greetings, Martijn

saranyamohan’s picture

saranyamohan commented

nothing will happen .it is not a good patch