I’ve found the function that is causing the failure of the icons on the context-menu to fail:
<?php
if(IN_MANAGER_MODE!="true") die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the MODx Content Manager instead of accessing this file directly.");
function constructLink($action, $img, $text, $allowed) {
if($allowed==1) {
?>
<link rel="stylesheet" type="text/css" href="media/style/<?php echo $manager_theme ? "$manager_theme/":""; ?>contextMenu.css<?php echo "?$theme_refresher";?>" />
<div class="menuLink" onmouseover="this.className='menuLinkOver';" onmouseout="this.className='menuLink';" onclick="this.className='menuLink'; parent.menuHandler(<?php echo $action ; ?>); parent.hideMenu();">
<img src='media/style/<?php echo $manager_theme ? "$manager_theme/":""; ?>images/icons/<?php echo $img; ?>.gif' align=absmiddle><?php echo $text; ?>
</div>
<?php
} else {
?>
<div class="menuLinkDisabled">
<img src='media/style/<?php echo $manager_theme ? "$manager_theme/":""; ?>images/icons/<?php echo $img; ?>.gif' align=absmiddle><?php echo $text; ?>
</div>
<?php
}
}
?>
The part in particular that is weird is that the $manager_theme variable works in the stylesheet context, but not in the actual links being built. Very odd.
Also, since the following constuct(?) is used over and over again, isn’t there a simpler way where we could just declare a single variable at the top rather than repeating that over and over again?
<?php echo $manager_theme ? "$manager_theme/":""; ?>
My thought is assigning it as a variable at the top of relevant pages and then using it inline as appropriate via {$varname} if that would work... will it? My coding skills aren’t a bit rusty; they’re virtually nonexistant.
☆ A M B ☆
MODX Staff
MODX Staff