Ben Wittenberg

SelectPanel2 Optimization: Do not add and remove keydown listener for escape key on every render

1

### Description `SelectPanel2.SelectPanel` will call `dialogEl.removeEventListener` and `dialogEl.addEventListener` on every render, because the call to `React.useEffect` is not passed an array of dependencies at this callsite: https://github.com/primer/react/blob/main/packages/react/src/drafts/SelectPanel2/SelectPanel.tsx#L195 The React docs...

fix(DataTable): sort table on initial render with `initialSortColumn` or `initialSortDirection` props

7

Closes #3951 ### Changelog #### New #### Changed - The `useTable` hook is responsible to sort the data for the `DataTable`. If the data changes, then it sorts according to...

fix(SelectPanel2): only bind keydown event when necessary

2

Closes #4581 ### Changelog #### New - Adds additional `jest` test coverage for the `SelectPanel` #### Changed - Adds optimization to `SelectPanel` so that event listeners are not added and...