Hey @ntsekouras, thanks a lot for opening this follow-up PR!

I think it would be great if we could list knobs for all properties of both the ToggleGroupControl and the ToggleGroupControlOption components — and if those knobs all had the same default values that those components would start with (for example, help should default to undefined)

A good example to follow would be the Card component's story, which has knobs for both the Card components and its subcomponents. That story also showcases how values from the Context System are applied to the children sub-components (code here

Also, while reviewing this PR, I realised that the hideLabelFromVision prop is marked as required in the types definition, in contrast with it being defined as not required in the README — I believe we should update the types file and make it optional there, what do you think?

ToggleGroupControlOption option just has value (which IMO is irrelevant for the story) and label props. Do you think it gives value to be able to just change the label of an option?

It would be probably good to expose the label, if anything for illustrating that it exists as part of the component's APIs

Also, another small nit: regarding the knobs, for the Card story we called them like the props. We should probably apply the same convention here as well, e.g.

• hideLabelFromVision
• isBlock (renders ToggleGroupControl as a CSS block element)
• isAdaptiveWidth (renders segments with equal widths)
• help (Help text)
• ...

Finally, while reviewing the story, I came across a potential glitch: the black highlight behind the currently selected item doesn't update when the isAdaptiveWidth is updated

Although this can (and probably should) be tackled in a separate PR :)