-
Notifications
You must be signed in to change notification settings - Fork 55
fix(AlertBehavior) add missing aria-describedby to AlertBehavior when… #2208
Conversation
… type is different than warning/danger microsoft#2197
Perf comparison
Potential regressions comparing to master
Perf tests with no regressions
Generated by 🚫 dangerJS |
Need a tip how can I fix screener tests in that case. Edit. After checking screener.yml I guess it may be related to branch and PR via fork |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@delprzemo sorry for the delay, this looks good. please update the branch to master and check if the changelog entry gets to the unreleased section. thanks for your contribution!
@jurokapsiar No problem. Branch updated to master - changelog entry in the unreleased section. I don't have write access so could you merge it? |
foreced push despite failing screener - currently the integration does not work for forked repos. This PR does not introduce any visual changes. |
This PR fixes issue with lack of
aria-describedby
inAlertBehavior
when dismiss action is defined and type is different than warning/dangerFixes #2197