Primary-links issue
macrocosm - May 2, 2009 - 05:15
| Project: | Signwriter |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Jump to:
Description
Something seems to be keeping this from working. AFAICT the snippet below is correct .. it is in my template.php I have tinkered with it some and flushed all caches plenty. It seems like something is preventing it from working.
<?php
/**
* Implementation of theme_menu_item_link()
*/
function phptemplate_menu_item_link($link) {
if (($link['menu_name'] == 'primary-links' || $link['menu_name'] == 'navigation')) {
if (empty($link['localized_options'])) {
$link['localized_options'] = array();
}
$profile = signwriter_load_profile('MenuLinkNormal');
$link['localized_options']['html'] = TRUE; // Because we'll be sending HTML into the l() call.
return l(signwriter_title_convert($link['title'], $profile, array(), TRUE), $link['href'], $link['localized_options']);
}
else {
return theme_menu_item_link($link); // Use the inherited behavior for everything else.
}
}
?>I have been able to replace block headers page titles and have some input filters working too so I know the module works.. its just the primary links wont change! Anyone have an idea what could be blocking it?

#1
in 2.x branch there's good support for menus but for some reason whatever menu is set as primary links will not be processed by Signwriter. the same menu as normal menu shown as a block will.
#2
subscribe
#3
Is this the same issue I'm having? I've set my secondary links to be enhanced by signwriter but the hover and active states don't show. Any help?
#4
I am also having problems with the active and hover states. Background images will not show either. Anyone else having problems with this?
#5
@subaudio
Background images not working for active and hover states is not part of this issue (although it is an issue)
Please open a new issue for that and keep issues separate.
Thanks.
Hopefully I'll get time to have a run at this issue queue vary soon.