[EuiBasicTable] Move to next-to-last page if all items of last pages are deleted.

[ashikmeerankutty]

Summary

Fixes #3422

Before

After

Checklist

• Check against all themes for compatibility in both light and dark modes
• Checked in mobile
• Checked in IE11 and Firefox
  - [ ] Props have proper autodocs
  - [ ] Added documentation examples
  - [ ] Added or updated jest tests
• Checked for breaking changes and labeled appropriately
  - [ ] Checked for accessibility including keyboard-only and screenreader modes
• A changelog entry exists and is marked appropriately

[kibanamachine]

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

[myasonik]

Just for docs accessibility, when the delete button disappears can we place the user's focus somewhere on the page? The checkbox in the table header is a good option.

[ashikmeerankutty]

@myasonik Thanks for the suggestion. I will move the focus to the checkbox in the table header.

[ashikmeerankutty]

Just for docs accessibility, when the delete button disappears can we place the user's focus somewhere on the page? The checkbox in the table header is a good option.

@myasonik I can't find a method to do that without modifying the EuiBasicTable Component. Do you have any suggestions?

[myasonik]

Bleeeh, I forgot that forwarding refs is kind of a whole thing...

I've got a PR that I'm hoping to get back to soon that deals with some similar issues for navigation so I can tackle it then.

Feel free to skip this.

[chandlerprall]

jenkins test this

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_3431/

[ashikmeerankutty]

@chandlerprall Thank you for the review. Changes committed 👍

[chandlerprall]

Changes LGTM! Tested the selection example for basic & in-memory tables and they both work as expected. Will merge on green CI

[chandlerprall]

jenkins test this

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_3431/