Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[WebProfilerBundle] Add a "preview" tab in mailer profiler for HTML email #43751

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

Merged
merged 1 commit into from
Oct 29, 2021

Conversation

lyrixx
Copy link
Member

@lyrixx lyrixx commented Oct 26, 2021

Q A
Branch? 5.4
Bug fix? no
New feature? yes
Deprecations? no
Tickets
License MIT
Doc PR no need

I did not work with mail for a long time, and I just discover the new entry in the profile. This is very nice 👏🏼
But I miss a preview rendering, so here we go

image

@carsonbot carsonbot added this to the 5.4 milestone Oct 26, 2021
@lyrixx lyrixx changed the title [WebProfilerBundle] Add a 'preview' tab in mailer profiler [WebProfilerBundle] Add a "preview" tab in mailer profiler for HTML email Oct 26, 2021
@fabpot
Copy link
Member

fabpot commented Oct 29, 2021

Thank you @lyrixx.

@fabpot fabpot merged commit 46cda1e into symfony:5.4 Oct 29, 2021
@lyrixx lyrixx deleted the mailer-preview branch October 29, 2021 16:06
<div class="tab">
<h3 class="tab-title">HTML preview</h3>
<div class="tab-content">
<pre class="prewrap" style="max-height: 600px">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why using a pre tag as a wrapper of the iframe ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To force the mail "area"
But I might be removed, I did not test it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.