ButtonGroup - use slots to have IconButtons and TextButtons in the ButtonGroup

[thrbnhrtmnn]

Description

From the Merge- and Review-Session on 26.09. a few open/additional topics for the #246 came up that are followed up in this task.

After discussing this task in the refinement on 26.10. we decided to use slots to integrate the IconButton and TextButton component into the Button Group. Right now, the Button Group is not using these components but has Buttons that are just styled like a text button.

---

Acceptance Criteria

• The Button Group can contain Icon Buttons and Text Buttons inside slots, as well as both at the same time.
• Props documentation has been updated in Storybook and in Props Excel (on the All Props page and on the Components page)

---

Background information

• look at the figma design to see what combination examples of icon button and text button can exist

[thrbnhrtmnn]

As discussed in the refinement today, we have some scope change:

• In order to add an iconButton, we need to figure out how to add slots/children
• The buttons in the buttonGroup should be instances of the existing TextButton or IconButton components

[RubirajAccenture]

The tech concept of passing the children must be discussed with the technical team. Kindly contact @ChristianHoffmannS2 before picking up the ticket

[thrbnhrtmnn]

Hey @ChristianHoffmannS2 , Hey @RubirajAccenture ,

I had a look at the new button group today and already checked both ACs. Before closing the ticket, I have some questions though:

• Is it possible to also change the size for all the buttons in the group? Currently when switching the size, only the item-spacing changes
• Does the ButtonGroup also need events? I guess not, since the Text and the Icon Button both have the events.

[ChristianHoffmannS2]

the button group does not need events. if we want a global submit event for instance, we properly will do that with a "formgroup" thingy.

inheriting things like theme or size would be great, but needs investigation. i am not sure yet how, but would love to see that as well

[thrbnhrtmnn]

Okay, I will create a follow up task to investigate this. But this task I will close as this investigation is not part of the scope.

Link to the follow up issue: #593