Page MenuHomePhabricator

Selecting a RadioOptionWidget programmatically scrolls it into view
Closed, ResolvedPublic

Description

In general, changing form fields programmatically should not scroll them into view.

This was introduced in https://gerrit.wikimedia.org/r/c/oojs/ui/+/189989

Event Timeline

Change 745839 had a related patch set uploaded (by Esanders; author: Esanders):

[oojs/ui@master] Don't always scroll RadioOptionWidget when selected

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

After this, the remaining OptionWidgets with scrollIntoViewOnSelect = true are MenuOptionWidget, OutlineOptionWidget & TabOptionWidget. As these typically exist inside their own scrollable container, I think it makes sense for these to always scroll into view.

Change 745839 merged by jenkins-bot:

[oojs/ui@master] Don't always scroll RadioOptionWidget when selected

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

Change 753557 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/core@master] Update OOUI to v0.43.0

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

Volker_E triaged this task as Medium priority.Jan 13 2022, 4:42 PM
Volker_E moved this task from Backlog to OOUI-0.43.0 on the OOUI board.
Volker_E edited projects, added OOUI (OOUI-0.43.0); removed OOUI.

Change 753557 merged by jenkins-bot:

[mediawiki/core@master] Update OOUI to v0.43.0

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

Volker_E assigned this task to Esanders.