Skip to content

[EuiBasicTable] Fix rendering of noItemsMessage within table caption to not stringify input ReactNode#9341

Merged
tkajtoch merged 2 commits intoelastic:mainfrom
tkajtoch:fix/9333-euibasictable-empty-state
Jan 29, 2026
Merged

[EuiBasicTable] Fix rendering of noItemsMessage within table caption to not stringify input ReactNode#9341
tkajtoch merged 2 commits intoelastic:mainfrom
tkajtoch:fix/9333-euibasictable-empty-state

Conversation

@tkajtoch
Copy link
Copy Markdown
Member

Summary

Fixes #9333

This PR fixes an issue causing [object Object] to be announced in EuiBasicTable caption instead of the proper, rendered ReactNode from noItemsMessage prop.

Why are we making this change?

Because of a bug in the implementation, AT output was affected when announcing EuiBasicTable.

Screenshots #

before After
Screenshot 2026-01-27 at 11 31 12 Screenshot 2026-01-27 at 11 31 38

Impact to users

No negative impact to users. After this gets merged the EuiBasicTable announcements of empty tables will be corrected.

QA

Remove or strikethrough items that do not apply to your PR.

  • Go to , turn on VoiceOver, and confirm the table announcement includes No items found and no [object Object]

General checklist

  • Browser QA
    • Checked for accessibility including keyboard-only and screenreader modes
  • Docs site QA
  • Release checklist
    • A changelog entry exists and is marked appropriately

… caption to not try stringify input `ReactNode`
@tkajtoch tkajtoch self-assigned this Jan 27, 2026
@tkajtoch tkajtoch requested a review from a team as a code owner January 27, 2026 10:53
@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

cc @tkajtoch

@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

cc @tkajtoch

Copy link
Copy Markdown
Contributor

@acstll acstll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Thanks for the fix @tkajtoch! Tested manually and it's working as expected.

@tkajtoch tkajtoch merged commit 59c491c into elastic:main Jan 29, 2026
5 checks passed
weronikaolejniczak added a commit to elastic/kibana that referenced this pull request Feb 5, 2026
## Dependency updates

- `@elastic/eui`: `v112.1.0` ⏩ `v112.2.0`

---

## Package updates

### `@elastic/eui`
[v112.2.0](https://github.com/elastic/eui/releases/tag/v112.2.0)

- Updated `documents` `timelineWithArrow` `indexOpen` `indexClose`
`indexEdit` `indexRuntime` `indexSettings` `folderOpen` `folderClose`
`kubernetesPod` `pagesSelect` `section` `workflow` glyph icons
([#9339](elastic/eui#9339))

**Accessibility**

- Fixed `EuiBasicTable` and `EuiInMemoryTable` empty table announcements
to include the expected `noItemsMessage` content
([#9341](elastic/eui#9341))

---------

Co-authored-by: Tomasz Kajtoch <tomasz.kajtoch@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[EuiBasicTable] Issues with empty state announcement in VoiceOver

3 participants