Several Codex components take in menuItems as a prop and use a v-model binding (usually with the default name, modelValue) to indicate which menu item, if any, is selected. We should make sure that the use of these props is as consistent as possible across all components, to make their use more predictable and intuitive.
Some things to address:
- Some components require the selected value prop (usually modelValue), some do not. We should make this consistent.
- Similarly, some components require the menuItems prop, even if it's an empty array, while some do not and default to an empty array. This should also be consistent.
- The default value for the model value may vary across components to some extent, but we should carefully example each default value to create some level of consistency, especially for null/falsy values
QA:
All the components that make use of Menu items have been modified. The QA required for this task is to make sure that the following components are tested and work as expected:
- Select component
- Typeahead component
- Lookout component
- Menu component
- Combobox component
