Skip to content
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

refactor(OverflowMenu): remove reference to deleted method #4220

Merged

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Oct 4, 2019

Closes #4219

refs carbon-design-system/carbon-components-react@dbd7a61#diff-be16eec7227e997459598a1b98fd0631

during the v9 to v10 migration and cleanup we removed the internal handleBlur method in <OverflowMenu> but there are still references to that method in the component. This PR removes those dead references

Testing / Reviewing

Ensure <OverflowMenu> functions as expected

@emyarod emyarod requested a review from a team as a code owner October 4, 2019 13:57
@ghost ghost requested review from abbeyhrt and jnm2377 October 4, 2019 13:58
@netlify
Copy link

netlify bot commented Oct 4, 2019

Deploy preview for the-carbon-components ready!

Built with commit bb8e842

https://deploy-preview-4220--the-carbon-components.netlify.com

@netlify
Copy link

netlify bot commented Oct 4, 2019

Deploy preview for carbon-components-react failed.

Built with commit bb8e842

https://app.netlify.com/sites/carbon-components-react/deploys/5d9d316a6b3ff300084b6fbc

@netlify
Copy link

netlify bot commented Oct 4, 2019

Deploy preview for carbon-elements ready!

Built with commit bb8e842

https://deploy-preview-4220--carbon-elements.netlify.com

Copy link
Contributor

@abbeyhrt abbeyhrt left a comment

Choose a reason for hiding this comment

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

Looks good to me! 🎉

@joshblack joshblack merged commit b22cece into carbon-design-system:master Oct 9, 2019
@emyarod emyarod deleted the 4219-overflow-menu-dead-code branch October 17, 2019 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OverflowMenu contains references to removed or unused methods
4 participants