Add support for themes

Currently only modules are checked for help files, but this could easily be extended to themes.

I agree – theme support is a terrific idea. I was thinking about documenting my experience with various aspects of the Zen theme and immediately wondered how to use Advanced Help for this.

Definitely +1. We make some complicated themes, would love to have a Drupally way to document things inside the theme itself :)

I was totally wrong

// Search paths:
$paths = array(
path_to_theme() . '/help', // Allow theme override.
drupal_get_path('module', $module) . "/translations/help/$language->language", // Translations.
$info['path'], // In same directory as .inc file.
);

So there you go.

This only really allows overrides, though. Actually having help *for* a theme might be trickier, though not completely impossible. I'm not sure if what's in there is enough so not yet marking fixed, though if it is sufficient for folks, go ahead and do so.

This patch seems to add theme support. Use just as in modules: help directory in the theme, (theme).help.ini file, and pages.

Revised patch includes revision of the help text.

+1 This is huge!

@fgm: Thank you! The patch works great.

Also, I hope it's ok if I change the priority of this feature request to major, because this really is a huge deal for theme developers.

This finally makes it possible for the theme to easily provide deluxe style guides, code snippets, image and graphic specifications, best practices, how-to videos, etc.

Anyone willing/able to port this patch to D7? I'd love to test it out by creating theme documentation, but all the new sites I'm developing are in 7. I agree that this will be an amazing benefit for theme developers and, more importantly, for content administrators who will be able get help on themes, so I'd love to help test and get it committed.

suscribe

@nerilka: please no longer use comments to subscribe to issues on d.o.: click the "Follow" button on top of the issue instead.

Funny that stephthegeek asked me about this very thing today. I bet she forgot that she's asked for it once already! =-)

Now if anyone would commit that patch, or at least RTBC it so I can commit it...

Simply drupal usage: I wrote the patch, so I may not RTBC it.

Committed for D6 (so far 6.x-1.x-dev only). Needs to be ported to D7.

Committed for D7 (so far 7.x-1.x-dev only).

Fixed at last, then, since it's committed to both 6.x and 7.x ?

Yes, it is committed to both, but so far only in the nightly dev snapshot.

I signed on as a new maintainer of this project yesterday, and there is a huge backlog of issues in the queue to look at. I am slowly working my way through it. Expect a new full release for both D6 and D7 in about two weeks time.

PS: A huge "thank you" for this and your other contributions to the Advanced help project.