This PR closes no issue, but fixes that the customName property was always set on icons in empty-state, giving a customName deprecation warning for icons even if a builtin icon name was used.
Additionally it deprecates the empty-state customIconName similarly to the icons customName.
Does this PR introduce a breaking change?
[ ] Yes
[x] No
Are there any additional context?
Checklist:
The following tasks should be carried out in sequence in order to follow the process of contributing correctly.
Reminders
[ ] Make sure you have implemented tests following the guidelines in: "The good: Test".
[ ] Make sure you have updated the cookbook with examples and showcases (for bug fixes, enhancements & new components).
Review
[ ] Determine if your changes are a fix, feature or breaking-change, and add the matching label to your PR. If it is tooling, dependency updates or similar, add ignore-for-release.
Which issue does this PR close?
This PR closes no issue, but fixes that the customName property was always set on icons in empty-state, giving a customName deprecation warning for icons even if a builtin icon name was used.
Additionally it deprecates the empty-state customIconName similarly to the icons customName.
Does this PR introduce a breaking change?
Are there any additional context?
Checklist:
The following tasks should be carried out in sequence in order to follow the process of contributing correctly.
Reminders
Review
When the pull request has been approved it will be merged to
develop
by Team Kirby.