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

html source code error in multi-column #109

Open
confettidc opened this issue May 8, 2018 · 16 comments
Open

html source code error in multi-column #109

confettidc opened this issue May 8, 2018 · 16 comments

Comments

@confettidc
Copy link

Dear Support,

Just found that if create the edm layout with multi-column with left text + right image.

Sometimes the left text with html source code error, the left text cannot display after the eDM sent out.

For example:
screen shot 2018-05-08 at 10 24 53 am

After checked the source code, when create the eDM format like that. Sometime the system will generate the error codes like this:

<divstyle="text-align: left;font-family: arial;font-size: 14px;color: #000000;line-height: 21px;mso-line-height: exactly;vertical-align: middle;mso-text-raise: 3px;">

The "div" merged with "style", it made the eDM cannot display the left text after sent out.

Please help to solve the problem.

Thanks.

Jeff

@confettidc
Copy link
Author

Dear Support,

For more information:

The left text + right image is the not "TEXT + IMAGE" element.

I added the multi-column element, then add "Text" element at left column and "Image" element at right column.

Thanks.

Jeff

@smiska
Copy link

smiska commented May 8, 2018

Hi Jeff,
Thanks for the detailed bug report. The reason is that we needed to change line-breaks (from "\r\n" to "\n") for the HTML and in some systems it seems that this Unix-like newline is not respected. We're already working on a fix to avoid problems with this.

@confettidc
Copy link
Author

Dear Support,

Thanks for you help!

Just found more source code has same issue.

I found the similar error on:

<spanstyle .......
<pstyle ......

Hope this information can help you.

Thanks.

Jeff

@smiska
Copy link

smiska commented May 8, 2018

Hi, we had a similar issue recently, and it there the underlying issue seems to be that the HTML generated by us is processed and linebreaks are removed by the partner's system. Could you check if you don't modify the HTML similarly?

@ademirdiniz
Copy link

Hi.

We have the same issue.

<spanstyle .......
<pstyle ......

What do we need to do to solve it?

@smiska
Copy link

smiska commented May 8, 2018

@ademirdiniz As you gave us a temporary account I could take a look and it seems that for the preview (and eventually the test email and email campaing sending as well) you process the HTML, remove all the linebreaks and make the whole email HTML a single one line string. This way you remove those linebreaks, which we insert especially for long lines in the HTML, after the opening tag. I don't know the reason behind this implementation, but this seems to be the cause behind this issue.

@ademirdiniz
Copy link

Hi, @smiska. Thanks.

We have changed our tool to works with this EDM update.

Now is working again.

Is possible to receive an email when you publish the releases?

Thanks

@smiska
Copy link

smiska commented May 9, 2018

Hi @ademirdiniz , I'm glad that you could solve it.
@confettidc , could you check how you handle HTML on your side,
and if it's not processed in a similar way?

Some form of release notification sounds well. We'll look into how this could be implemented, thanks for the suggestion.

@confettidc
Copy link
Author

HI @smiska, any updates of the issue?

My customers reported this issue again.

Please help to solve it or provide some workaround.

Thanks!

@smiska
Copy link

smiska commented May 10, 2018

Hi @confettidc , we identified that the issues by the recent update may have it's root cause with how you handle our HTML code. Could you look into it please?

Or you could set up a test account for us and we inspect what may go wrong. If you choose this option please email login credentials to the test account to [email protected] Thanks.

@confettidc
Copy link
Author

After we replace below, the issue is fixed:

<pstyle => <p style
<spanstyle => <span style
<divstyle => <div style
<strongstyle => <strong style
<ahref => <a href

Thanks for help.

@smiska
Copy link

smiska commented May 11, 2018

Thanks for the update. I'm glad that you could solve it.

@confettidc
Copy link
Author

Dear Support,

Just found new error source codes, these error occur at bullet point function:

<spanlang
<spanclass
<tdstyle

@confettidc confettidc reopened this May 21, 2018
@smiska
Copy link

smiska commented May 21, 2018

Can you provide projectId where this occurs?
Is the rest of the template showing up correctly?
Do you have reproduction steps, or doest it occur for each and every list?
Any further information is greatly helpful. Thanks.

@confettidc
Copy link
Author

Dear Support,

One more error source code ofund:
<h1style

You may reference project id 5b0cb114743e7a26009711d2 for reference, thanks.

@smiska
Copy link

smiska commented Jun 6, 2018

Apologies for the late reply.

I checked the generated HTML from this project (with multiple generation),
and I could'nt find <h1style. In your implementation, probably a line-break is not respected,
I highlight the one that is most likely removed when you process our HTML.

generate

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