This occurs when you enable a separate theme for the administration section and enable "Use administration theme for content editing" (via Site Configuration > Administration Theme). The node creation and node editing pages use the separate administration theme just like anything under admin/*. However, when you go to delete a node, the confirm screen that displays does not use the administration theme. As you can see from my patch, the logic that determines what theme to use only handles for the edit and add options; I added 'delete' as another option.
P.S. Not sure if my patch will apply, my local CVS repo is misbehaving right now. It's a simple patch if someone needs to roll another.
Comment | File | Size | Author |
---|---|---|---|
#12 | Use-admin-theme-on-content-delete-672370-12.patch | 804 bytes | elimuir |
#7 | admin-theme-on-content-delete-672370-7.patch | 803 bytes | Michael Zetterberg fd. Lopez |
#5 | admin_theme_delete_02.patch | 804 bytes | squarecandy |
admin_theme_delete.patch | 794 bytes | jgoldberg | |
Comments
Comment #1
robynover CreditAttribution: robynover commentedI tried this and it got me part-way there. I got the admin theme but no longer go the confirmation form.
Here was my solution, maybe it will help someone:
I'm using the Admin module as my admin theme, and I added code in admin.module around line 30:
It treats node delete pages as admin pages. It's hacky but it works.
Comment #2
robynover CreditAttribution: robynover commentedIf you're not using the admin module, you could probably do something similar in your own module, using the same if statement and
as in the patch above
Comment #4
squarecandy CreditAttribution: squarecandy commentedI know this thread is 10 months old - but this is still an issue in 6.20 core. I just applied the suggested change in the patch manually - works great! Sorry I haven't quite figured out the patch thing yet... this one's so simple, maybe I'll give it a shot tomorrow.
Comment #5
squarecandy CreditAttribution: squarecandy commentedsee if this patch is better...
Comment #7
Michael Zetterberg fd. Lopez CreditAttribution: Michael Zetterberg fd. Lopez commentedThe patch fixes it for me too.
Re-rolled against 6.x
Comment #8
Michael Zetterberg fd. Lopez CreditAttribution: Michael Zetterberg fd. Lopez commentedComment #10
Michael Zetterberg fd. Lopez CreditAttribution: Michael Zetterberg fd. Lopez commented#7: admin-theme-on-content-delete-672370-7.patch queued for re-testing.
Comment #12
elimuir CreditAttribution: elimuir commentedThe former patch from comment #7 was missing close parentheses after 'delete'.
Re-rolled patch with fix.
Comment #13
elimuir CreditAttribution: elimuir commented