Email invite design improvements #8681
Conversation
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR Summary
This PR enhances email invite design by improving layout compatibility across email clients and updating styling elements for better visual consistency.
- Replaced flexbox with
@react-email/componentsRow/Column structure inpackages/twenty-emails/src/components/HighlightedContainer.tsxfor better email client support - Added blue color (#5e90f2) to
packages/twenty-emails/src/common-style.tsfor consistent link styling - Updated company address and simplified Twenty's description in
packages/twenty-emails/src/components/WhatIsTwenty.tsx - Added color prop to
packages/twenty-emails/src/components/Link.tsxfor flexible link styling - Removed unnecessary div wrapper in
packages/twenty-emails/src/components/HighlightedText.tsxto fix display issues
6 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings | Greptile
| } as React.CSSProperties; | ||
|
|
||
| export const HighlightedContainer = ({ | ||
| children, | ||
| }: HighlightedContainerProps) => { | ||
| return ( | ||
| <Container style={highlightedContainerStyle}> | ||
| <div style={divStyle}>{children}</div> | ||
| {React.Children.map(children, (child) => ( | ||
| <Row> |
There was a problem hiding this comment.
logic: Row component is missing a key prop for the mapped elements
| type HighlightedTextProps = { | ||
| value: ReactNode; | ||
| centered?: boolean; | ||
| }; |
There was a problem hiding this comment.
logic: The 'centered' prop is defined in the type but not used in the component implementation
| } as React.CSSProperties; | ||
|
|
||
| export const HighlightedContainer = ({ | ||
| children, | ||
| }: HighlightedContainerProps) => { | ||
| return ( | ||
| <Container style={highlightedContainerStyle}> | ||
| <div style={divStyle}>{children}</div> |
There was a problem hiding this comment.
Could you please provide a screenshot in your PR description? I just have the Logs testing (sure I could copy the html - is that what you were doing each time or you have something smarter?)
There was a problem hiding this comment.
Done! I did a simple Gmail SMTP configuration setup.
closes #7140
