-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(Notification): add NPE check in escape close handler #17528
fix(Notification): add NPE check in escape close handler #17528
Conversation
73027d9
to
6af2c70
Compare
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #17528 +/- ##
=======================================
Coverage 76.95% 76.95%
=======================================
Files 408 408
Lines 13979 13979
Branches 4343 4343
=======================================
Hits 10758 10758
Misses 3047 3047
Partials 174 174 ☔ View full report in Codecov by Sentry. |
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
e4fe031
Closes #16874
This PR adds a NPE check to the Notification Esc key handler to prevent console errors after dismissal
Changelog
Changed
useEscapeToClose
handlerTesting / Reviewing