fix: add missing tooltip-neutral color variant#4519
Conversation
tooltip-neutral was the only semantic color variant missing from the tooltip component. All others (primary, secondary, accent, info, success, warning, error) were defined but neutral was not, making the class a no-op and absent from autocomplete. Fixes saadeghi#4492
|
Hey, Thanks for the PR. |
|
Good point — you're right that |
I wouldn't suggest doing that. And I won't guarantee that it will work in future versions 😅 Thr goal for daisyUI is not to provide every class name for every color. We're trying to be more practical and be close to real life use cases. Please let me know if you have any questions 🌼 |
|
Makes total sense, thanks for the clear explanation! Keeping modifiers on the same element is definitely cleaner. Appreciate you taking the time to review it 🌼 |
Problem
tooltip-neutralwas never defined intooltip.css, making it a silent no-op. Every other semantic color variant exists (tooltip-primary,tooltip-secondary,tooltip-accent,tooltip-info,tooltip-success,tooltip-warning,tooltip-error) butneutralwas skipped.This means:
tooltip-neutralhas no effect on stylingFix
Added
.tooltip-neutralfollowing the exact same pattern as the other color variants — sets--tt-bgtovar(--color-neutral)and appliestext-neutral-contentto the tooltip content.The placement in the file follows alphabetical ordering relative to the other variants.
Closes #4492