Sitelet https://github.com/carbon-design-system/carbon-components-react/pull/1756
Skip to content
This repository was archived by the owner on Oct 19, 2021. It is now read-only.

fix(Modal): avoid NPE with null check - #1756

Merged
asudoh merged 2 commits into
carbon-design-system:masterfrom
emyarod:1751-modal-focus-button-npe
Jan 22, 2019
Merged

asudoh merged 2 commits into
carbon-design-system:masterfrom
emyarod:1751-modal-focus-button-npe

Conversation

@emyarod

@emyarod emyarod commented Jan 15, 2019

Copy link
Copy Markdown
Member

Closes IBM/carbon-components-react#1751

This PR avoids a NPE in <Modal /> by performing a null check on the focus button before chaining the .querySelector method

@netlify

netlify Bot commented Jan 15, 2019 •

Copy link
Copy Markdown

Deploy preview for carbon-components-react ready!

Built with commit 3fdc424

https://deploy-preview-1756--carbon-components-react.netlify.com

@asudoh asudoh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 - Thanks @emyarod!

@emyarod
emyarod force-pushed the 1751-modal-focus-button-npe branch from 1193e00 to 30897a6 Compare January 21, 2019 16:02

@lovemecomputer lovemecomputer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@asudoh
asudoh merged commit 2f5baa0 into carbon-design-system:master Jan 22, 2019
@carbon-bot

Copy link
Copy Markdown

🎉 This PR is included in version 6.80.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants