Refactor Blade component children

[zepfietje]

E.g. remove icon prop from button and simply pass an icon component.
Same for dropdown list item badge and probably others.

Donate 💰 to fund this issue

• You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
• 100% of the funding will be distributed between the Filament core team to run all aspects of the project.
• Thank you in advance for helping us make maintenance sustainable!

[danharrin]

@zepfietje Can you explain the benefits here please? I think I agree about badges, but I think the icons should remain as they are (I encourage you to take a look at how icons are output in v4 too)

[danharrin]

Badges refactored in #15506, I can't see other places where this would be useful but open to suggestions.

[zepfietje]

The current system supports only a single icon in a specified position.
Passing an icon element as child of the button component would allow for much more flexible use in Blade and just feels more like HTML.

[danharrin]

Yeah, there are lots of considerations that would make this tricky though. For example on a button, the icon needs to be replaced with a loading indicator when loading