fix(tailwind): non-inlined styles getting HTML entities on React 18#2858
fix(tailwind): non-inlined styles getting HTML entities on React 18#2858gabrielmfern merged 7 commits intocanaryfrom
Conversation
🦋 Changeset detectedLatest commit: 5af8933 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
commit: |
72a5942 to
594a932
Compare
There was a problem hiding this comment.
cubic analysis
No issues found across 1 file
Linked issue analysis
Linked issue: OSS-18: Find a way to make the <Font> component work properly on Windows Outlook
| Status | Acceptance criteria | Notes |
|---|---|---|
| Make the component work properly on Windows Outlook | Preserves raw CSS but no inlining or selector change | |
| ❌ | Avoid using the universal '*' CSS selector (unsupported on Windows Outlook) | No removal or replacement of '*' selector in diffs |
| ❌ | Inline font-family style into all elements (so Windows Outlook gets the font even without '*' support) | No implementation of inlined font-family across elements |
|
@cubic-dev-ai review this |
@gabrielmfern I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
cubic analysis
No issues found across 2 files
Linked issue analysis
Linked issue: OSS-18: Find a way to make the <Font> component work properly on Windows Outlook
| Status | Acceptance criteria | Notes |
|---|---|---|
| ❌ | Stop relying on the universal (*) CSS selector in the component | No changes to Font or selector usage in diffs |
| ❌ | Apply font-family via inline styles to elements (inline all elements) | No inlining logic or Font component changes present |
| ❌ | Ensure works properly on Windows Outlook (avoid unsupported selectors) | PR only updates Tailwind style injection, not Font compatibility |
Closes #2841
Summary by cubic
Switched style injection to use dangerouslySetInnerHTML so React 18 doesn’t HTML-encode generated CSS.
Written for commit 5af8933. Summary will update on new commits.