Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.There are some extra unusefull code in the uc_catalog.module.
I guess it will be better to remove tags <strong> from the strings 850, 1334, 1345, 1360
The problem is in IE6 when it shows catalog menu. You can see this code and bug in design on the attached pic.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | strong_tag.patch | 6.06 KB | Island Usurper |
| #3 | strong_tag.patch | 4.22 KB | Island Usurper |
| uc_catalog.PNG | 62.69 KB | Santa Claus |
Comments
Comment #1
rszrama commentedAgreed. We already have a uc_catalog.css. Instead of putting this markup in, you should use a class to designate the currently active item and a CSS property to make it
font-weight: bold;.Comment #2
Island Usurper commentedComment #3
Island Usurper commentedI'm really not happy with the way the catalog module renders its block. The strong tag is just one of the many problems. This patch really only deals with that one, though, because I have to figure out how it should be done before I can go in and fix things.
Comment #4
Island Usurper commentedFixed one of the additional problems. The catalog block theme was spread out over three different functions, which was unnecessarily difficult for themers to deal with. The hook_theme() was changed already, so I decided to consolidate them into a single function.