Set a role to Alert

[sneko]

Hi,

The default HTML with Alert is:

<div id="fr-alert-«rp»" class="fr-alert fr-alert--error">
  <h3 class="fr-alert__title">Erreur</h3>
  <div>
    Le lien de confirmation est invalide, veuillez contacter notre support
  </div>
</div>

I think when severity is error a role should be set on for accessibility, like role="alert". Also for other severities maybe it could be role="status". Since I'm not sure for the latter, maybe the component could allow specifying the role.

What do you think?

[ddecrulle]

Hello,

I don’t have a strong opinion on this, and I believe it is mainly related to @gouvernementFR/dsfr, since we only implement their specifications in React. As far as I can see, there used to be a role before: GouvernementFR/dsfr#182.

I closed the issue because this decision is not up to react-dsfr. If the DSFR team changes their approach, feel free to open a PR here to add the role or to reopen the discussion.