Skip to content

Revert Flatlist changes #1306

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 29, 2022
Merged

Revert Flatlist changes #1306

merged 2 commits into from
Jul 29, 2022

Conversation

chiuam
Copy link

@chiuam chiuam commented Jul 29, 2022

Reverts #1269

@chiuam chiuam requested a review from a team as a code owner July 29, 2022 16:52
@chiuam chiuam changed the title Revert "[0.66] Flatlist keyboard navigation: Mouse can move selection" Revert Flatlist changes Jul 29, 2022
@chiuam chiuam merged commit 8ce9545 into 0.66-stable Jul 29, 2022
@chiuam chiuam deleted the revert-1269-list-with-mouse-66 branch July 29, 2022 18:39
Saadnajmi added a commit to Saadnajmi/react-native-macos that referenced this pull request Aug 24, 2022
Saadnajmi added a commit to Saadnajmi/react-native-macos that referenced this pull request Aug 25, 2022
Saadnajmi added a commit that referenced this pull request Aug 26, 2022
…1375)

* Revert "Revert Flatlist changes (#1306)"

This reverts commit 8ce9545.

* Refactor handling of keyDown/keyUp (#1338)

This refactors / simplifies certain keyUp|Down event handling.
It will make a later change (adding textInput handling for textInput fields) easier (to review)

Co-authored-by: Scott Kyle <[email protected]>

* Deprecate onScrollKeyDown, refactor Flatlist selection logic (#1365)

* Deprecate onScrollKeyDown

remove pressable diff

Remove JS handling for PageUp/Down, fix flow errors

Add back "autoscroll to focused view" behavior

remove commented code

remove change to pressable

Update documentation

fix flow error

fix lint issue

Fix 'selectRowAtIndex'

More simplification

lock

* Make method public again

* Add initialSelectedIndex

* macOS tags

* fix lint

* remove one more reference

* yarn lint --fix

* RNTester: only show the Flatlist keyboard navigation switch on macOS

Co-authored-by: Christoph Purrer <[email protected]>
Co-authored-by: Scott Kyle <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants