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

1.1.16 goes mobile on Yandex.ru #1

Closed
bago opened this issue Mar 18, 2018 · 2 comments
Closed

1.1.16 goes mobile on Yandex.ru #1

bago opened this issue Mar 18, 2018 · 2 comments

Comments

@bago
Copy link
Member

@bago bago commented Mar 18, 2018

Seems like Yandex is removing the whole style attribute when it find something "unexpected".

From some tests the issue is within the "calc(NUMBERpx - NUMBER%" we use to enable responsivity when media queries are not available. Whenever Yandex parser find a "-" a "_" a "+" an "*", an "&" in brackets it will remove the full style object. If it finds a "- " sequence or a "-NUMBER" or a "NUMBER-" in a property value (even not in brackets) it will also remove the full style object.

Otherwise Yandex removes only the unknown styles.

That calc is our current main responsive solution for multicolumns:
https://github.com/voidlabs/versafix-template/blob/master/template-def/template-versafix-1.mml#L467

I currently don't see how we could workaround Yandex, unless we find some way to target Yandex. We could try replacing the div with a span, but we have to check all of the other clients to see if this would work for them.

@bago
Copy link
Member Author

@bago bago commented Mar 18, 2018

One option to be evaluated could be adding align="left" to the column table tag and adding some fake style to the same tag so that Yandex remove the styles also from the table.

This would also be needed in order to let browsers without any support for "style" to render the email decently.

Loading

@bago bago added this to the 1.1.16 milestone Mar 19, 2018
@bago bago removed this from the 1.1.16 milestone Mar 19, 2018
@bago bago added this to the 1.1.17 milestone Mar 19, 2018
@bago
Copy link
Member Author

@bago bago commented Jul 6, 2018

Loading

@bago bago closed this in 0f32193 Jul 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant