I don't know why the box theme is used for pages, but I would request that either there be a consistent use of sitemenu_box (so that it is easily themable) or the creation of a sitemenu_page_box theme so that we could use that instead.

Not sure why there is a difference here, between the page view & other views, but phptemplate already uses the theme_box so it isn't possible to just modify this functions output. I just did the following (but that was an easy hack):

300c300,302
< $output .= theme('box', $subject, $box['content']);
---
> $output .= theme('sitemenu_box', $subject, $box['content']);
> // drupal default
> // $output .= theme('box', $subject, $box['content']);

Comments

kbahey’s picture

Version: master » 5.x-1.x-dev
Status: Active » Fixed

I created a new function theme_sitemenu_box_page for that case

Committed to 5.x-1.x-dev, and should be in the tarball tomorrow.

mgifford’s picture

Thanks.. That's very handy!

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.