Fix GridList disallowTypeAhead #9124
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #9120
disallowTypeAheadwas not correctly being passed through intouseGridListProps.✅ Pull Request Checklist:
📝 Test Instructions:
Run Storybook and navigate to the GridList stories.
Replace the existing
GridListExampleinGridList.stories.tsxwith this code snippet. This setup just setstextValueonGridListItemsand thedisallowTypeAheadprop on the GridList.Before the fix: Interact with the
GridListand type a letter that exists as an item. Observe that type-ahead is still active and selects the matching item, despitedisallowTypeAheadbeing set.After the fix: No item should be selected or focused when typing.
🧢 Your Project: