-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Box Control: Rename VerticalHorizontalInputControls to AxialInputControls #33016
Box Control: Rename VerticalHorizontalInputControls to AxialInputControls #33016
Conversation
Size Change: 0 B Total Size: 1.05 MB ℹ️ View Unchanged
|
@ciampo and @sarayourfriend this is not at all an urgent PR, but I've just added you as reviewers in case you get a chance to take a look at this (hopefully fairly simple) rename since you looked at the preceding PR (#32610) 🙂 |
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.
Code changes LGTM, Storybook stories work well!
Thanks for the review! 🙇 |
Description
Following on from #32610 and based on a suggestion from @mtias, rename the
VerticalHorizontalInputControls
component toAxialInputControls
.How has this been tested?
Tested for regressions based on the steps in #32610:
npm run storybook:dev
npm run test-unit -- packages/components/src/box-control/test/
Screenshots
Renamed component in Storybook:
Types of changes
Non-breaking code quality update.
Checklist:
*.native.js
files for terms that need renaming or removal).