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

Email is not responsive #24

Open
perlpty opened this issue Nov 8, 2016 · 1 comment
Open

Email is not responsive #24

perlpty opened this issue Nov 8, 2016 · 1 comment

Comments

@perlpty
Copy link

perlpty commented Nov 8, 2016

After a lot of reading and testing I managed to build and install Mosaico. At first, I was glad that it appeared to be working with PHP-backend. But when I sent the first TEST to my email address in a desktop enviroment using only the Block "sideArticleBlock" I noted that the image was on top; but because it is a desktop environment it is supposed to be at the left, as it is correctly shown at mosaico PREVIEW (which also works good).

After that, I decided to DOWNLOAD the HTML to see the source code. The Download function also works Good. The HTML showed some things that catched my attention:

Most of lines have this code at the end: 
 I compared to other responsive templates that I found in the web (like foundation, antwort, etc.) and they dont have that character in each line.

The versafix-1 source code does not have this character at the end, so I guessed at that time that the issue could be related somehow to mosaico itself. However, after a long conversation in that threat ( see: voidlabs/mosaico#227 ) it seems that it might be related to the php-backend.

Here I send 2 screenshots to ilustrate the problem. As you can see the target is a desktop email client, so it should display the desktop version.

sc4

sc3

I made this very same test with mosaico demo ( https://mosaico.io/ ) and the email is good in tablet, mobile or desktop.

Please help.

@markalston
Copy link

It's probably due to the premailer which isn't needed anymore since mosaico takes care of that now. Try removing the premailer lines and see what you get. I do have an updated fork over at https://github.com/markalston/mosaico-php-backend which has removed the premailer and fixed some other issues. I know that it is months later but I'd love to know if my changes fix this issue.

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

2 participants