Page MenuHomePhabricator

Lookup: support readonly state
Closed, ResolvedPublic2 Estimated Story Points

Description

The Lookup component is missing a readonly state even though it uses a text input under the hood. The readonly state should not only match TextInput, ChipInput, etc but also prevent users from interacting with the menu or menu items.

Technical notes:

Noting here that readonly styles and input behavior currently exist in Lookup because the readonly attribute gets passed to the internal TextInput. However, if there are menu items (such as the case of initial suggestions), the menu will open and be accessible even in the readonly state.

Event Timeline

CCiufo-WMF triaged this task as Medium priority.
CCiufo-WMF moved this task from Inbox to Needs Refinement on the Design-System-Team board.
CCiufo-WMF set the point value for this task to 2.

Thank you for tagging this task with good first task for Wikimedia newcomers!

Newcomers often may not be aware of things that may seem obvious to seasoned contributors, so please take a moment to reflect on how this task might look to somebody who has never contributed to Wikimedia projects.

A good first task is a self-contained, non-controversial task with a clear approach. It should be well-described with pointers to help a completely new contributor, for example it should clearly pointed to the codebase URL and provide clear steps to help a contributor get setup for success. We've included some guidelines at https://phabricator.wikimedia.org/tag/good_first_task/ !

Thank you for helping us drive new contributions to our projects <3

Change #1184574 had a related patch set uploaded (by Pushpit; author: Pushpit):

[design/codex@main] [T384256] Lookup: add readonly state and block menu interaction

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

Change #1184574 had a related patch set uploaded (by Pushpit; author: Pushpit):

[design/codex@main] Lookup: add readonly state and block menu interaction

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

Catrope moved this task from In Progress to Code Review on the Codex board.

Change #1184574 merged by jenkins-bot:

[design/codex@main] Lookup: add readonly state and block menu interaction

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

Change #1191541 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/core@master] Update Codex from v2.3.1 to v2.3.2

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

Change #1191541 merged by jenkins-bot:

[mediawiki/core@master] Update Codex from v2.3.1 to v2.3.2

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