Skip to content

fix: correct email template logo sizing and email client compatibility#191

Merged
ManukMinasyan merged 1 commit intomainfrom
fix/email-template-logos
Mar 19, 2026
Merged

fix: correct email template logo sizing and email client compatibility#191
ManukMinasyan merged 1 commit intomainfrom
fix/email-template-logos

Conversation

@ManukMinasyan
Copy link
Copy Markdown
Contributor

Summary

  • Fix aspect ratio distortion: The logo image (960x256px, ratio 3.75:1) was forced to 164x39px (ratio 4.2:1), causing horizontal stretching. Changed CSS width from 164px to auto so the browser computes the correct width from the height.
  • Add inline styles and HTML attributes: Email clients like Outlook, Yahoo Mail, and older Gmail strip <style> blocks and ignore CSS classes. Added width="146" height="39" HTML attributes and inline style as fallback to prevent the logo rendering at its native 960x256px.

Files changed

  • resources/views/vendor/mail/html/themes/default.css.logo width changed from 164px to auto
  • resources/views/vendor/mail/html/header.blade.php — Added width, height, and inline style attributes to <img> tag

Test plan

  • Send a test email (password reset, email verification, or team invitation) and verify logo renders at correct proportions
  • Test in Outlook/Yahoo Mail to verify inline styles prevent oversized logo

Fix aspect ratio distortion (960x256 image forced to 164x39) by using
width: auto. Add inline styles and HTML width/height attributes for
Outlook and other email clients that strip CSS classes.
Copilot AI review requested due to automatic review settings March 19, 2026 18:10
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@ManukMinasyan ManukMinasyan merged commit ba2e5e4 into main Mar 19, 2026
10 checks passed
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

Successfully merging this pull request may close these issues.

2 participants