Hi,
When you place a mail.css file in your theme base directory, then Mimemail includes that instead of all the CSS files of your default theme. A handy feature to keep your emails clean and small.
When using the Zen theme however, things go wrong when you are used to place all your custom stuff in a Zen subtheme (like your should). Mimemail doens't look for a mail.css file in your Zen subtheme, it will only catch it if you place it in your base Zen theme directory.
I'm suggesting the following patch to have Mimemail's use of themes play nicer with the Zen theme. I know this is kind of a corner case and this patch isn't useful for a lot of people (not everyone uses the Zen theme). But it doesn't hurt for them and it will solve a lot of head aches for the unexperienced Zen/Mimemail/Drupal user.
When looking for a mail.css file, Mimemail now first looks in the Zen subtheme (if available), then in the normal (Zen base) theme. Nothing changes for non-Zen users, things get a lot better for those who do use Zen :)
So, include it if you like, otherwise: no hard feelings :)
Hopla
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | mimemail_338460_01.patch | 509 bytes | sgabe |
| mimemail-zen_integration-0.patch | 780 bytes | hopla |
Comments
Comment #1
andrew.eatherington@gmail.com commentedThe same thing happens with the blueprint theme. mail.css has to be in the root whereas it should be in the css folder.
cheers
Comment #2
andrew.eatherington@gmail.com commentedComment #3
andrew.eatherington@gmail.com commentedComment #4
andrew.eatherington@gmail.com commentedIt's actually version 6.x-1.2 that this applies to...
Comment #5
arski commentedany chance of getting this fixed? cheers
Comment #6
simon georges commentedThe function is now a preprocess function and is located in theme/mimemail.theme.inc file, so the patch needs a little work before being integrated. The code itself looks good.
Comment #7
mpaler commentedFor what it's worth, mail.css works with Zen subthemes if you put it directly within the subtheme directory, rather than within the subtheme/css/ directory.
Comment #8
sgabe commentedThis seems to be enough, see the attached patch.
Comment #9
sgabe commentedCommitted to the development snapshot.