react-spectrum
c9c838b0 - Adding FocusRing to ListView (Virtualizer persistedKeys) (#3168)

Commit
3 years ago
Adding FocusRing to ListView (Virtualizer persistedKeys) (#3168) * Adding FocusRing to ListView * a pass at ListView no focus and scroll modaility * reverting change prior to virtualizer improvement * fixing a merge conflict mistake * adding persisted keys to virtualizer for ListView FocusRing * handling table keys better * Update packages/@react-aria/virtualizer/src/Virtualizer.tsx Co-authored-by: Robert Snow <rsnow@adobe.com> * Update packages/@react-stately/virtualizer/src/Virtualizer.ts Co-authored-by: Robert Snow <rsnow@adobe.com> * missing semicolon * Update packages/@react-aria/virtualizer/src/Virtualizer.tsx Co-authored-by: Robert Snow <rsnow@adobe.com> * fixingn lint errors (space and unneeded return statement * update to tests based on Virtualizer improvement * replacing recursion with while loop * syntax fix * adding listview scrolling and focus tests * messed up the arrow movement prior to last commit * calling updateSubViews and layoutInfos keeping order matching collection * fixing tests (updateSubviews still needs work) * cleaning up logic issues found by failing picker tests * storing some work in progress before taking a new path * persisted keys refactor for listview and picker (tableview needs work and picker test failure) * adding persistedKey checking to TableLayout * tests pass and added some comments * making persisted keys work with table cells * preventing different loops from looking at the same item Co-authored-by: Robert Snow <rsnow@adobe.com> Co-authored-by: Daniel Lu <dl1644@gmail.com>
Author
Parents
Loading