Skip to content

Created a fluid AMP version of the template#283

Open
JayOram wants to merge 1 commit into
emailmonday:mainfrom
JayOram:main
Open

Created a fluid AMP version of the template#283
JayOram wants to merge 1 commit into
emailmonday:mainfrom
JayOram:main

Conversation

@JayOram

@JayOram JayOram commented Feb 14, 2022

Copy link
Copy Markdown
Contributor

Without changing the structure, only be editing the changes needed to make the original fluid template valid in AMP4Email.

Without changing the structure, only be editing the changes needed to make the original fluid template valid in AMP4Email
@TedGoas

TedGoas commented Feb 14, 2022

Copy link
Copy Markdown
Collaborator

I see you created a few file here, does that mean we'd need to go from three templates to six to support AMP? I'm wondering if it's worth it. There is some previous discussion about whether or not to support AMP. Would love to hear your take.

@JayOram

JayOram commented Feb 15, 2022

Copy link
Copy Markdown
Contributor Author

@TedGoas - as AMP supports classes and most HTML with less of the restrictions on HTML/CSS email templates, such as mso fixes/using th/hybrid/responsive/fluid etc. Just one version that I added would work for all of those modules in AMP.

@TedGoas

TedGoas commented Feb 18, 2022

Copy link
Copy Markdown
Collaborator

Just one version that I added would work for all of those modules in AMP.

What do you mean? Could we theoretically consolidate all three existing templates into a single AMP template since it's less code? Or would each template need it's own AMP version. Sorry for the dumb questions, I've not really worked with AMP emails before.

@JayOram

JayOram commented Feb 19, 2022

Copy link
Copy Markdown
Contributor Author

Apologies I didn't explain myself very well and you can ignore the original pull request.

I thought each version of the cerberus template was identical in modules. AMP only works in one way - the closest would be responsive.

  • No need for inline styles
  • All styles can go in the head and no need for important
  • As AMP is supported across clients, no need for mso specific fixes etc. the AMP code will work across all email clients that support AMP4Email

I will code all of the modules from all of the cerberus templates into 1x supported AMP4Email template. Then add a new pull request.

@TedGoas TedGoas added the question nothing reported, just asking a question label Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question nothing reported, just asking a question

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants