Preview doesn't show the templated content.

When submitting a newsletter, to a test-email for instance, the previewed content is displayed.

Comments

TBarregren’s picture

TBarregren commented
Assigned: Unassigned » TBarregren
Status: Active » Closed (works as designed)

Strictly speaking, you say that the preview should show how the e-mail, and not as today how the node, will look like. But that would break off the practice that the preview shows how the node will look like on screen. Thus, in this narrow sense, I think the current behavior is correct. I have therefore change the status to "by design".

However, as mentioned in #130766, the display of an newsletter issue probably should include the header and footer, which only are included in the e-mail version as of today.

It might be tempting to extend that feature request to also embrace the output of the theamable function theme_simplenews_template_mail_content(). But that is a bad idea. First and foremost, that would break the theming of the web site itself. Second, that would include a terrible HTML code, which from necessity is violating almost every ascpect of the W3's RFC:s.

To conclude, I don't approve your bug report, but a future version will probably include the template's header and footer in the node, and hence also in the preview.

dtabach’s picture

dtabach commented

the display of an newsletter issue probably should include the header and footer, which only are included in the e-mail version as of today.

Actually, currently headers and footers are also included in the node version of the newsletter (http://drupal.org/node/141635). But I agree they shouldn't.