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

RangeControl: Remove unused UseDebouncedHoverInteractionArgs type #44411

Merged
merged 1 commit into from
Sep 23, 2022

Conversation

aaronrobertshaw
Copy link
Contributor

Related: #44271

What?

In #44271, useDebouncedHoverInteraction was removed but its related type UseDebouncedHoverInteractionArgs wasn't. As this isn't used anymore, this PR simply removes it.

Why?

🧹 Keeping things clean is good.

How?

  • Deleted unused type.

Testing Instructions

  1. Check there are no type errors.
  2. Ensure there are no other references to UseDebouncedHoverInteractionArgs in the code base.
  3. Run RangeControl unit tests if you feel like it

@aaronrobertshaw aaronrobertshaw added [Feature] UI Components Impacts or related to the UI component system [Type] Code Quality Issues or PRs that relate to code quality labels Sep 23, 2022
@aaronrobertshaw aaronrobertshaw self-assigned this Sep 23, 2022
Copy link
Contributor

@ciampo ciampo left a comment

Choose a reason for hiding this comment

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

🧹 🚀

@aaronrobertshaw aaronrobertshaw merged commit b3c665e into trunk Sep 23, 2022
@aaronrobertshaw aaronrobertshaw deleted the refactor/remove-unused-range-control-type branch September 23, 2022 08:23
@github-actions github-actions bot added this to the Gutenberg 14.3 milestone Sep 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] UI Components Impacts or related to the UI component system [Type] Code Quality Issues or PRs that relate to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants