Combine AppIconButton
and HelpIconButton
#2716
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes proposed in this Pull Request:
The
Button
from@wordpress/components
supports having an icon. However, the existence ofAppIconButton
makes it appear that there is a duplication of implementation between them.Considering that the only use case for
AppIconButton
in the last four years has beenHelpIconButton
, this PR combines them into one.Additional changes:
@return
tag in JSDocHelpIconButton
use cases only set theeventContext
prop.Detailed test instructions:
HelpIconButton
componentHelpIconButton
component is presented the same as beforeAdditional details:
💡 In the future, if there are more buttons need to set the position of icon relative to the top of the button text, it's preferred to extend
AppButton
to supporticonPosition="top"
.Changelog entry