-
Notifications
You must be signed in to change notification settings - Fork 54
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
Focusing/scroll issues when virtualization is enabled #424
Comments
HI @cetetesoft |
Sure, @komarovalexander . Here you are: The demo will start with the cell Now, from this point (having the cell I have tested this demo in both Chrome and Firefox with the same result. |
Hi @cetetesoft
im afraid this is default behavior of html table, see example https://stackblitz.com/edit/table-keyboard-navigation-ts-prknhj?file=Demo.tsx click by cell and change focus using "Tab" key |
maybe it is not so obvious, but there are no rendered rows above, so you cant move the focus further. To fix that there is |
Hi.
I have one bug report and one enhancement suggestion regarding the scrolling behavior for virtualized rows.
When calling
table.moveFocusedUp()
, if the current cell is on top (ie from the 1st visible row), the table wrapper won't scroll up. This behavior is different from that oftable.moveFocusedDown()
for the last visible cell. In addition, the cell keeps the:focus-within
style, even though I can tell it's not actually the active cell.Steps to reproduce:
<tr>
with theheight
attribute).ArrowUp
key several times.ArrowDown
key several times. You'll need to press it as many times asArrowUp
.Additionally, it would be nice for the scroll down behavior to more consistent. Right now, if you press
ArrowDown
having the last visible row selected, it will scroll the table by one row only if the last row is completely visible. When that row is partially visible, however, the table will scroll by about 50% of the visible height. Could this be customized in some way, like, scrolling just one row in all cases (like spreadsheet software such as Excel does)?Thanks.
The text was updated successfully, but these errors were encountered: