@greenwoodt I'm trying to check out your change and it doesn't seem to work in the page that you're adding the screenshot:

Also we should use vh (viewport height) instead of h (hardcoded height that doesn't take into account different viewports). If we add something equivalent to height: 40rem; we will have problem with other views (like mobile and tablets in portrait/horizontal mode).

@andreslucena see what you think of the latest commit changes.

@andreslucena see what you think of the latest commit changes.

Now it's too big, and we can't see the buttons to actually send the newsletter. We should add the sticky buttons. In the other hand, I see other newsletter previews that we need to fix.

As this is growing in scope, and I want to fix the whole process with @carolromero, I'll take care of this. Thanks for the PR @greenwoodt. I'll let you know when it's ready for you to take a look.

@andreslucena see what you think of the latest commit changes.

Now it's too big, and we can't see the buttons to actually send the newsletter. We should add the sticky buttons. In the other hand, I see other newsletter previews that we need to fix.

As this is growing in scope, and I want to fix the whole process with @carolromero, I'll take care of this. Thanks for the PR @greenwoodt. I'll let you know when it's ready for you to take a look.

Ok just let me know what you two would prefer to propose as a change. I was having difficultly adjusting the iframe using those tailwind classes. h-lvh wouldn't work without h-full.

@greenwoodt I went ahead an made the PR with all the changes, as it was just faster to just do it instead of explain the changes in all the pages. You can find the result at #12550

Closing this in favor of #12550