Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move styles in head section of the HTML template #76

Merged
merged 1 commit into from
Feb 12, 2013

Conversation

nmpetkov
Copy link
Contributor

<style> tag must be located in the head section

@cmfcmf
Copy link
Collaborator

cmfcmf commented Feb 12, 2013

BTW: I don't know exactly, but shouldn't it be

<style type="text/css">

?

@nmpetkov
Copy link
Contributor Author

Probably. I'v just moved existing one.

I spend some time to read about html emails and what tags to use in header and document section.
It seems more simple - better.
In my html.tpl for example I'm using document type transitional. I'l consider even removing document type specification, and just simple html open/close tag.

For style tag also I'm not sure what is better. Discussion can be useful.

dmm1 added a commit that referenced this pull request Feb 12, 2013
Move styles in head section of the HTML template
@dmm1 dmm1 merged commit 7e371d2 into dmm1:master Feb 12, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants