[PM-31776] Fix email-verified Send OTP email design discrepancies

[mcamirault]

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-31776

📔 Objective

This PR fixes a couple of discrepancies with the design of the OTP code email that is sent when accessing email-verified Sends. Since some of the requested changes were to the shared <mj-bw-learn-more-footer> component I had to regenerate some other email templates and also ran npm run prettier afterwards for formatting fixes

📸 Screenshots

Before

After

[github-actions]

Checkmarx One – Scan Summary & Details – 5111e953-507e-4298-81f1-e6fc01c68e52

Great job! No new security vulnerabilities introduced in this pull request

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.28%. Comparing base (946a032) to head (5aa14c8).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6990      +/-   ##
==========================================
+ Coverage   56.26%   56.28%   +0.01%     
==========================================
  Files        1983     1986       +3     
  Lines       87651    87660       +9     
  Branches     7815     7814       -1     
==========================================
+ Hits        49318    49339      +21     
+ Misses      36503    36490      -13     
- Partials     1830     1831       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[claude]

Claude finished @mcamirault's task in 3m 31s —— View job

---

Code Review Summary

PR: PM-31776 Fix email-verified Send OTP email design discrepancies

Overall Assessment: Approved -- Clean PR with targeted design fixes and formatting cleanup.

Changes Reviewed

Intentional design changes:

• send-email-otp.mjml: Token display updated from <b> tag to font-weight: 500, copy text refined, line-height: 24px added to info section
• mj-bw-learn-more-footer.js: Removed custom padding (now uses MJML default 20px 0), changed font-weight: bold to font-weight: 500, added margin-top: 0px

Formatting/cleanup:

• Prettier formatting applied across all MJML source files and JS components
• Handlebars variable spacing normalized ({{var}} to {{ var }})
• README markdown formatting improvements

Generated output:

• All 9 affected .html.hbs files correctly regenerated to reflect both the design and formatting changes
• The 2 remaining MJML templates using mj-bw-learn-more-footer (invite.mjml, two-factor.mjml) appear to be development/preview templates without corresponding production HBS files, so no regeneration is needed

Suggestion (not in diff, mentioning here)

The plain text email template src/Core/MailTemplates/Handlebars/Auth/SendAccessEmailOtpEmail.text.hbs (line 8) still contains the old copy:

"Bitwarden Send transmits sensitive, temporary information to others easily and securely."

The HTML version was updated to:

"Bitwarden Send securely shares sensitive information."

Consider updating the text template for consistency between the two versions of this email.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[vleague2]

UIF files look good!