Skip to content

fix(web-console): grid unresponsiveness after scroll #425

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 1 commit into from
Apr 28, 2025

Conversation

emrberk
Copy link
Contributor

@emrberk emrberk commented Apr 27, 2025

Fixes #227

  • Call scroll() after scrolling grid by dragging scrollbar to prevent unresponsiveness, just as in render()
  • Check if focusedCell and activeRow is the actual item with focusedRowIndex before applying focus styles. This prevents focused row and cell to be shown indefinitely as the user scrolls the list.

@emrberk emrberk requested a review from bluestreak01 April 27, 2025 12:01
@bluestreak01 bluestreak01 merged commit 4130281 into main Apr 28, 2025
4 checks passed
@bluestreak01 bluestreak01 deleted the fix/grid-scroll branch April 28, 2025 07:53
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.

Keyboard navigation is broken when results grid is scrolled down
2 participants