resend / react-email

💌 Build and send emails using React
https://react.email
MIT License
13.97k stars 630 forks source link

Tooltip content text is invisible #1326

Open Fadingvision opened 6 months ago

Fadingvision commented 6 months ago

Describe the Bug

image

Which package is affected (leave empty if unsure)

react-email

Link to the code that reproduces this issue

https://demo.react.email/preview/notifications/github-access-token?view=source

To Reproduce

https://demo.react.email/preview/notifications/github-access-token?view=source

Expected Behavior

Tooltip content text is visible

What's your node version? (if relevant)

No response

gabrielmfern commented 6 months ago

Seems ok for me, the tooltip seems to appear fine image

Fadingvision commented 6 months ago

@gabrielmfern looked into some details, this bug looks like it is related to os theme, further information:

os info

windows 10 chrome version(121.0.6167.189)

windows light theme:

image

windows dark theme(which looks fine):

image

Looks like the text color should be set to white anyways, hope it helps.

devharipragaz007 commented 6 months ago

is the bug still exists? Shall I give it a try?

gabrielmfern commented 6 months ago

is the bug still exists? Shall I give it a try?

Yup it's still there. PR's are welcome! Would love to review yours if you do make one. You also don't have to ask permission to open one either. :)

ffiona commented 6 months ago

On Mac OS is the same, depends on the system theme.

Screenshot 2024-03-11 at 16 50 35

devharipragaz007 commented 6 months ago

is the bug still exists? Shall I give it a try?

Yup it's still there. PR's are welcome! Would love to review yours if you do make one. You also don't have to ask permission to open one either. :)

Please assign this to me.

kaustabhws commented 4 months ago

Screenshot 2024-04-27 210800

Hello, it's my first time commenting on a issue. but I'd like to add something. I think the issue in here `<TooltipPrimitive.Content {...props} className={cn( 'bg-black border border-slate-6 z-20 px-3 py-2 rounded-md text-xs', '${inter.variable} font-sans', )} ref={forwardedRef} sideOffset={sideOffset}

{children} </TooltipPrimitive.Content>`

instead of giving bg-black we can set the color based on the theme. I haven't explored the entire code so I'm not sure what is being used for themes, but maybe use external package and conditionally change the bg color, for eg dark:bg-black for dark mode and default bg-white. Forgive me if I'm commenting anything wrong

ranafaraznaru commented 1 month ago

@gabrielmfern Hey, I've checked and the issue is still present, can you please assign it to me?