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

Unclear how to add logo #1

Open
jrgifford opened this issue Aug 5, 2015 · 5 comments
Open

Unclear how to add logo #1

jrgifford opened this issue Aug 5, 2015 · 5 comments

Comments

@jrgifford
Copy link

In the ccs, there are mentions of the email-masthead-logo, but no directions on how to add the logo to the masthead. Is that supported with these templates? I don't see it in any of the demos/screenshots that you've published.

Great templates, can't wait to play with them more!

@derekrushforth
Copy link
Contributor

Hey @jrgifford, this is possible! Here's an example to get you started:

<a href="http://google.com"><img class="email-masthead_logo" src="http://i.imgur.com/P0ZCafk.png" width="90" height="90" border="0"></a>

To support this further I made a small change to .email-masthead_logo, so you'll want to pull the latest. 3507d12

Let me know if this helps and I'll be sure to document this in the README.

@jrgifford
Copy link
Author

Looks good! Thanks!

@brendon
Copy link

brendon commented Aug 1, 2018

The style change never seemed to make it through to your live version on Postmark. Is that a bug? :) I just copied it through but thought you'd want to know :)

@derekrushforth
Copy link
Contributor

Thanks for the heads up @brendon. Looks like this was removed by accident a couple years ago when we switched over to MailMason for our builds.

@derekrushforth derekrushforth reopened this Aug 1, 2018
@brendon
Copy link

brendon commented Aug 1, 2018

Roger :) I think also you could add in a commented out image tag for the masthead_logo just to give people a bit of a hint as to what to do :) Not sure if the actual border, width, and height attributes on the img tag are needed though as in your original example.

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

No branches or pull requests

3 participants