feat(Alert): Use `data-iui-status`

iTwin / iTwinUI-react

A react component library for iTwinUI.

https://github.com/iTwin/iTwinUI

Other

83 stars 23 forks source link

Closed adamhock closed 1 year ago

adamhock commented 1 year ago

class="iui-alert-(status) -> class="iui-alert" data-iui-status="(status)" where (status) can be informational, positive, negative, or warning.
class="iui-alert iui-sticky" -> class="iui-alert" data-iui-sticky="true"

These are the changes in React as a result of this Alert PR in CSS

This PR will come out of draft once the CSS PR is merged

mayank99 commented 1 year ago

@adamhock I don't get it. Why did you merge this before updating to the latest css that I specifically released for your PR? The visual tests are now failing in CI: https://github.com/iTwin/iTwinUI-react/actions/runs/3440659055/jobs/5739369985