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
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

vthirteen - July 30, 2009 - 20:57
Version:6.x-1.x-dev» 6.x-2.x-dev

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

xqbzzr - August 10, 2009 - 18:57

subscribe

#3

goye2cz - September 9, 2009 - 08:38

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

subaudio - September 10, 2009 - 14:15

I am also having problems with the active and hover states. Background images will not show either. Anyone else having problems with this?

#5

Justin W Freeman - October 1, 2009 - 02:44

@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.

 
 

Drupal is a registered trademark of Dries Buytaert.