-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Borders: Remove unused BorderStyleControl component #40635
Borders: Remove unused BorderStyleControl component #40635
Conversation
Size Change: -350 B (0%) Total Size: 1.23 MB
ℹ️ View Unchanged
|
Looks good to me, I couldn't see any other uses and everything built and ran ok, but I would be interested to hear from someone that has had experience with removing experimental APIs before signing off on it, just to confirm no deprecation period is needed. |
@glendaviesnz I've asked around with those more in the know and was given the following breakdown of our deprecation strategy.
The first point above is also supported by the Coding Guidelines So for this PR, we can delete this without any deprecation steps. I've updated the PR description to remove that question. Are you happy to approve this PR on that advice or shall we wait on a separate approval? cc/ @noisysocks |
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.
Looks good 👍
What?
Removes the
BorderStyleControl
component from the block editor.Why?
The
BorderStyleControl
component is no longer in use and has been superceded by the newBorderControl
andBorderBoxControl
components which include a border style picker.How?
Given the component was experimental, it has simply been removed.
Testing Instructions