I just checked what my custom 403 page looks like by trying to visit /admin when not logged in, and was greeted with my admin theme (in this case Garland). While there are hopefully few situations where users would be sent to such a page by accident, it seems better if it used the default site theme for those pages - since you're by definition not accessing the admin area when you see it.

CommentFileSizeAuthor
#1 system-346455-1.patch1.16 KBAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Active » Needs review
FileSize
system-346455-1.patch1.16 KB

This patch checks to make sure that the user has permission to access the specified page before it allows Drupal to overrides the default theme with the admin theme.

Status: Needs review » Needs work

The last submitted patch failed testing.

effulgentsia’s picture

effulgentsia CreditAttribution: effulgentsia commented
Status: Needs work » Fixed

Looks fixed to me.

Status: Fixed » Closed (fixed)

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

wOOge’s picture

wOOge CreditAttribution: wOOge commented

I back ported this patch to D6, see: http://drupal.org/node/759836