Page MenuHomePhabricator

Replace LTR/RTL buttons on docs site with ToggleButtons
Closed, ResolvedPublic

Description

Right now this uses a hacky pair of buttons that fake a selected state by setting action="primary" on the selected one. We should instead use ToggleButtons, now that that component exists.

In the future we may want to develop a ButtonGroup component for this use case, but for now a pair of ToggleButtons would work well.

Event Timeline

Change 784725 had a related patch set uploaded (by DannyS712; author: DannyS712):

[design/codex@main] DirectionSwitcher: use ToggleButton for direction options

https://gerrit.wikimedia.org/r/784725

Change 784725 merged by jenkins-bot:

[design/codex@main] DirectionSwitcher: use ToggleButton for direction options

https://gerrit.wikimedia.org/r/784725