CE-3638: Migrate iconbadge and iconmenubutton

IconBadge:

• Migrated to TS
• Removed never used secondIcon from props
• Cleaned up jsx and added gap prop (margins and stuff should be set in consuming applications)

IconMenuButton:

• Migrated to TS
• Removed a weird conditional sx props for the button, (should be set in consuming applications)
• Added a follow-up to revisit this component. It tries to handle too much. Preferably separate it into two components. One IconButton and one IconMenuButton.

For testing in consuming applications there are draft MR:s:
D&M: ce-deploy-ui!631 (merged)
Naming: ce-naming-ui!58 (merged)
Template: ce-template!22 (merged)

assigned to @johannaszepanski

added 2 commits

• 8a9ffcbe - migrated IconMenuButton
• 5c35c2e7 - migrated IconMenuButton stories

Compare with previous version

changed the description

added 2 commits

• 45e7d4b2 - migrated IconMenuButton
• 2716f586 - migrated IconMenuButton stories

Compare with previous version

added 1 commit

• 5a836f1e - migrated IconMenuButton stories

Compare with previous version

added 14 commits

• 5a836f1e...227c2982 - 9 commits from branch main
• 4f2069aa - migrated IconBadge to typescript
• a2945280 - cleaned up IconBadge and added gap prop
• 9697a07e - migrated IconBadge story
• d8129687 - migrated IconMenuButton
• 93849c84 - migrated IconMenuButton stories

Compare with previous version

changed the description

added 1 commit

• 3156cfc3 - 18.1.0

Compare with previous version

marked this merge request as ready

requested review from @skybrewer and @maxfrederiksen

LGTM, good idea to revist the menu icon, waaaay too big

approved this merge request

mentioned in commit 6efa28b6

merged