UAlert description slot in div

[cco3]

Description

Trying to place a div in a UAlert leads to hydration mismatches because both the #title and #description are p tags.

It's not clear to me that the #description should be in a p tag; and it'd be nice to put more substantial content inside an alert.

Additional context

Related issue

[sandros94]

I'm not fully sure to follow you here: would you like explicitly note in the docs that the #description slot is a p tag, or would you more prefer it to be a div?

[cco3]

I'm suggesting that it be a div so that I could nest other divs in it. I'm happy to defer to someone who has a better sense of these things, but I can imagine an alert being used for items more substantial than just phrasing content.

[sandros94]

I've opened a PR to add a default slot to the UAlert component.

This makes us not break the current functionality but adds the ability to insert custom components or code.

[benjamincanac]

I agree that having divs for descriptions would allow more possibilities but wouldn't that worsen SEO?

[moshetanzer]

@benjamincanac cant we just do a dynamic component with default as p tag and document it? Think this would be good as it is what people are most used to adding default tract in slot instead of as prop. Thoughts

[benjamincanac]

Here is how I fixed in in v3, example for the Toast component here: https://github.com/benjamincanac/ui3/blob/dev/src/runtime/components/Toast.vue#L84-L89