Skip to content
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

[EuiDataGrid] Cells in focus aren't always in view #3151

Closed
myasonik opened this issue Mar 23, 2020 · 4 comments · Fixed by #5515
Closed

[EuiDataGrid] Cells in focus aren't always in view #3151

myasonik opened this issue Mar 23, 2020 · 4 comments · Fixed by #5515

Comments

@myasonik
Copy link
Contributor

myasonik commented Mar 23, 2020

Screen Shot 2020-03-23 at 13 45 41

If a cell is partially cut off by the scrolling container, there's no way to see the whole thing with just a keyboard. If the cell is completely cut off, the browser will scroll it into view for you.

We should scroll the container to completely show the currently focused cell.

(May be a Firefox only issue.)

@barlowm
Copy link

barlowm commented Mar 23, 2020

@myasonik if you can give me an example I can check it out in Chrome.

@myasonik
Copy link
Contributor Author

Thanks for the offer Mike but @chandlerprall got to it first in another thread and reports that Chrome works as expected.

(Also, attached a screenshot to the description)

@barlowm
Copy link

barlowm commented Mar 23, 2020

Ahh good. Though really do wish things would work better cross browser :D

@cchaos cchaos added this to EuiDataGrid in Tables Sep 19, 2020
@cchaos cchaos changed the title Cells in focus aren't always in view [EuiDataGrid] Cells in focus aren't always in view Sep 20, 2020
@myasonik myasonik added this to Needs triage in a11y Dec 14, 2020
@myasonik myasonik moved this from Needs triage to Serious in a11y Dec 15, 2020
@myasonik myasonik moved this from Serious to Moderate in a11y Dec 15, 2020
@1Copenut
Copy link
Contributor

@barlowm Would you consider this issue working as expected and we can close it, or it's still a defect that needs remediated?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
a11y
Moderate
Tables
  
Closed
Development

Successfully merging a pull request may close this issue.

4 participants