DateTimePicker: AM/PM switcher is not screen reader accessible

[mirka]

What problem does this address?

The AM/PM toggle that appears when is12Hour={ true } only expresses state through visual styling, and cannot be distinguished by screen readers.

What is your proposed solution?

A similarly styled component, RadioGroup, is already deprecated in favor of ToggleGroupControl. Since we don't really use this UI pattern anymore, can we just replace it with a ToggleGroupControl? @WordPress/gutenberg-design

[jasmussen]

ToggleGroupControl seems perfect for this case. Would the compact size prop work there? Not a blocker, but can likely be useful in this context.

[jameskoster]

Probably a dumb question and/or rabbit hole, but does this need to be a separate control? Curious whether the time input could use a combobox pattern. This would allow you to select am/pm at the same time you choose the hour, and enable selection-by-mouse.

[mirka]

does this need to be a separate control?

Good point, I see how this can be enhanced to use a ComboboxControl. We can do a quick accessibility fix now by using a ToggleGroupControl for the AM/PM toggle, and once we have a rewritten ComboboxControlV2 it might be worth enhancing the inputs.