Hello.. Nice work, I like the results!

Just a suggestion here, but rather then telling people to modify a core Drupal theme like Garland, you can just include a little wrapper like the one I've included here. Drupal 6 makes this very easy. :)

I've just included one I've called "GarlandPlus" which will included your fieldset override *if* nodeform is enabled and then use the Garland parent theme. If you included this folder with your module people could just copy it to their themes directory (under sites) and have a safe option to switch to.

Just a thought, not a bug :)

CommentFileSizeAuthor
#1 nodeform.patch1.25 KBBevan
garlandplus.zip14.95 KBMoonshine

Comments

Bevan’s picture

Bevan commented
Title: Simple theme wrapper » Implement phptemplate_ function in module
StatusFileSize
new nodeform.patch1.25 KB

A different, arguably cleaner, approach to the same problem. I'm actually not sure if this works, because I don't know if drupal includes module file and then template.php or the other way round. I suspect it's the former. In which case this can be fixed by include(path_to_theme() .'/template.php') before if (!function_exists('phptemplate_fieldset')) .

hass’s picture

hass commented

Why not using nodeform_fieldset here?

hass’s picture

hass commented
Status: Active » Closed (duplicate)

#253767: form alter for our fieldset class