Skip to content

Fix @elastic/eui/require-table-caption lint violations in drilldown manager UI#271377

Merged
alexwizp merged 5 commits into
mainfrom
copilot/eslint-issue-reporterfix-require-table-caption-vio
May 27, 2026
Merged

Fix @elastic/eui/require-table-caption lint violations in drilldown manager UI#271377
alexwizp merged 5 commits into
mainfrom
copilot/eslint-issue-reporterfix-require-table-caption-vio

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 27, 2026

Two EuiInMemoryTable instances in the drilldown manager UI were missing the required tableCaption prop, violating @elastic/eui/require-table-caption.

Changes

  • drilldown_template_table — Added tableCaption with i18n string: "Drilldown templates"
  • drilldown_table — Added tableCaption with i18n string: "Drilldowns"
<EuiInMemoryTable
  tableCaption={txtTableCaption}
  items={drilldowns}
  // ...
/>

Captions describe the dataset per EUI accessibility guidelines and use i18n.translate for localization.

Adds the required `tableCaption` prop to two `EuiInMemoryTable` instances
in the drilldown manager UI to fix `@elastic/eui/require-table-caption`
ESLint violations for the @elastic/kibana-presentation team.
Copilot AI changed the title [WIP] Fix lint violations for @elastic/eui/require-table-caption Fix @elastic/eui/require-table-caption lint violations in drilldown manager UI May 27, 2026
Copilot AI requested a review from alexwizp May 27, 2026 07:41
@alexwizp alexwizp marked this pull request as ready for review May 27, 2026 07:53
@alexwizp alexwizp requested a review from a team as a code owner May 27, 2026 07:53
@botelastic botelastic Bot added Feature:Drilldowns Embeddable panel Drilldowns Feature:Embedding Embedding content via iFrame labels May 27, 2026
@alexwizp alexwizp added Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes backport:version Backport to applied version labels v9.5.0 v9.3.5 v9.4.2 and removed Feature:Embedding Embedding content via iFrame Feature:Drilldowns Embeddable panel Drilldowns labels May 27, 2026
@infra-vault-gh-plugin-prod
Copy link
Copy Markdown

Pinging @elastic/kibana-accessibility (Project:Accessibility)

@alexwizp
Copy link
Copy Markdown
Contributor

/ci

@alexwizp alexwizp added the a11y:agent-pr PR has been created by the agent label May 27, 2026
@kibanamachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Scout Lane #33 - serverless-observability_complete / default / local-serverless-observability_complete - Hosts Page - Empty State - should show onboarding page when no data is present

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
embeddable 89.6KB 89.9KB +261.0B

cc @alexwizp @Copilot

Copy link
Copy Markdown
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

LGTM
code review only

@botelastic botelastic Bot added Feature:Drilldowns Embeddable panel Drilldowns Feature:Embedding Embedding content via iFrame labels May 27, 2026
@alexwizp alexwizp merged commit fa6e1f1 into main May 27, 2026
208 checks passed
@alexwizp alexwizp deleted the copilot/eslint-issue-reporterfix-require-table-caption-vio branch May 27, 2026 12:22
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 9.3, 9.4

https://github.com/elastic/kibana/actions/runs/26510828016

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
9.3
9.4

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request May 27, 2026
…down manager UI (#271377) (#271442)

# Backport

This will backport the following commits from `main` to `9.4`:
- [Fix @elastic/eui/require-table-caption lint violations in drilldown
manager UI (#271377)](#271377)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT
[{"author":{"name":"Copilot","email":"198982749+Copilot@users.noreply.github.com"},"sourceCommit":{"committedDate":"2026-05-27T12:22:43Z","message":"Fix
@elastic/eui/require-table-caption lint violations in drilldown manager
UI (#271377)\n\nTwo `EuiInMemoryTable` instances in the drilldown
manager UI were\nmissing the required `tableCaption` prop,
violating\n`@elastic/eui/require-table-caption`.\n\n### Changes\n\n-
**`drilldown_template_table`** — Added `tableCaption` with i18n\nstring:
*\"Drilldown templates\"*\n- **`drilldown_table`** — Added
`tableCaption` with i18n
string:\n*\"Drilldowns\"*\n\n```tsx\n<EuiInMemoryTable\n
tableCaption={txtTableCaption}\n items={drilldowns}\n //
...\n/>\n```\n\nCaptions describe the dataset per EUI accessibility
guidelines and use\n`i18n.translate` for
localization.\n\n---------\n\nCo-authored-by: copilot-swe-agent[bot]
<198982749+Copilot@users.noreply.github.com>\nCo-authored-by: Alexey
Antonov
<alexwizp@gmail.com>","sha":"fa6e1f14d584a5fc26a2e62c3a4bbad2cb1c68ec","branchLabelMapping":{"^v9.5.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Embedding","Project:Accessibility","release_note:skip","Feature:Drilldowns","💝community","backport:version","a11y:agent-pr","v9.5.0","v9.3.5","v9.4.2"],"title":"Fix
@elastic/eui/require-table-caption lint violations in drilldown manager
UI","number":271377,"url":"https://github.com/elastic/kibana/pull/271377","mergeCommit":{"message":"Fix
@elastic/eui/require-table-caption lint violations in drilldown manager
UI (#271377)\n\nTwo `EuiInMemoryTable` instances in the drilldown
manager UI were\nmissing the required `tableCaption` prop,
violating\n`@elastic/eui/require-table-caption`.\n\n### Changes\n\n-
**`drilldown_template_table`** — Added `tableCaption` with i18n\nstring:
*\"Drilldown templates\"*\n- **`drilldown_table`** — Added
`tableCaption` with i18n
string:\n*\"Drilldowns\"*\n\n```tsx\n<EuiInMemoryTable\n
tableCaption={txtTableCaption}\n items={drilldowns}\n //
...\n/>\n```\n\nCaptions describe the dataset per EUI accessibility
guidelines and use\n`i18n.translate` for
localization.\n\n---------\n\nCo-authored-by: copilot-swe-agent[bot]
<198982749+Copilot@users.noreply.github.com>\nCo-authored-by: Alexey
Antonov
<alexwizp@gmail.com>","sha":"fa6e1f14d584a5fc26a2e62c3a4bbad2cb1c68ec"}},"sourceBranch":"main","suggestedTargetBranches":["9.3","9.4"],"targetPullRequestStates":[{"branch":"main","label":"v9.5.0","branchLabelMappingKey":"^v9.5.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/271377","number":271377,"mergeCommit":{"message":"Fix
@elastic/eui/require-table-caption lint violations in drilldown manager
UI (#271377)\n\nTwo `EuiInMemoryTable` instances in the drilldown
manager UI were\nmissing the required `tableCaption` prop,
violating\n`@elastic/eui/require-table-caption`.\n\n### Changes\n\n-
**`drilldown_template_table`** — Added `tableCaption` with i18n\nstring:
*\"Drilldown templates\"*\n- **`drilldown_table`** — Added
`tableCaption` with i18n
string:\n*\"Drilldowns\"*\n\n```tsx\n<EuiInMemoryTable\n
tableCaption={txtTableCaption}\n items={drilldowns}\n //
...\n/>\n```\n\nCaptions describe the dataset per EUI accessibility
guidelines and use\n`i18n.translate` for
localization.\n\n---------\n\nCo-authored-by: copilot-swe-agent[bot]
<198982749+Copilot@users.noreply.github.com>\nCo-authored-by: Alexey
Antonov
<alexwizp@gmail.com>","sha":"fa6e1f14d584a5fc26a2e62c3a4bbad2cb1c68ec"}},{"branch":"9.3","label":"v9.3.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.4","label":"v9.4.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
smith pushed a commit to smith/kibana that referenced this pull request May 27, 2026
…anager UI (elastic#271377)

Two `EuiInMemoryTable` instances in the drilldown manager UI were
missing the required `tableCaption` prop, violating
`@elastic/eui/require-table-caption`.

### Changes

- **`drilldown_template_table`** — Added `tableCaption` with i18n
string: *"Drilldown templates"*
- **`drilldown_table`** — Added `tableCaption` with i18n string:
*"Drilldowns"*

```tsx
<EuiInMemoryTable
  tableCaption={txtTableCaption}
  items={drilldowns}
  // ...
/>
```

Captions describe the dataset per EUI accessibility guidelines and use
`i18n.translate` for localization.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 29, 2026
@kibanamachine
Copy link
Copy Markdown
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.
cc: @alexwizp

dej611 pushed a commit to dej611/kibana that referenced this pull request May 29, 2026
…anager UI (elastic#271377)

Two `EuiInMemoryTable` instances in the drilldown manager UI were
missing the required `tableCaption` prop, violating
`@elastic/eui/require-table-caption`.

### Changes

- **`drilldown_template_table`** — Added `tableCaption` with i18n
string: *"Drilldown templates"*
- **`drilldown_table`** — Added `tableCaption` with i18n string:
*"Drilldowns"*

```tsx
<EuiInMemoryTable
  tableCaption={txtTableCaption}
  items={drilldowns}
  // ...
/>
```

Captions describe the dataset per EUI accessibility guidelines and use
`i18n.translate` for localization.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
kibanamachine added a commit that referenced this pull request May 29, 2026
…down manager UI (#271377) (#271440)

# Backport

This will backport the following commits from `main` to `9.3`:
- [Fix @elastic/eui/require-table-caption lint violations in drilldown
manager UI (#271377)](#271377)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT
[{"author":{"name":"Copilot","email":"198982749+Copilot@users.noreply.github.com"},"sourceCommit":{"committedDate":"2026-05-27T12:22:43Z","message":"Fix
@elastic/eui/require-table-caption lint violations in drilldown manager
UI (#271377)\n\nTwo `EuiInMemoryTable` instances in the drilldown
manager UI were\nmissing the required `tableCaption` prop,
violating\n`@elastic/eui/require-table-caption`.\n\n### Changes\n\n-
**`drilldown_template_table`** — Added `tableCaption` with i18n\nstring:
*\"Drilldown templates\"*\n- **`drilldown_table`** — Added
`tableCaption` with i18n
string:\n*\"Drilldowns\"*\n\n```tsx\n<EuiInMemoryTable\n
tableCaption={txtTableCaption}\n items={drilldowns}\n //
...\n/>\n```\n\nCaptions describe the dataset per EUI accessibility
guidelines and use\n`i18n.translate` for
localization.\n\n---------\n\nCo-authored-by: copilot-swe-agent[bot]
<198982749+Copilot@users.noreply.github.com>\nCo-authored-by: Alexey
Antonov
<alexwizp@gmail.com>","sha":"fa6e1f14d584a5fc26a2e62c3a4bbad2cb1c68ec","branchLabelMapping":{"^v9.5.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Embedding","Project:Accessibility","release_note:skip","Feature:Drilldowns","💝community","backport:version","a11y:agent-pr","v9.5.0","v9.3.5","v9.4.2"],"title":"Fix
@elastic/eui/require-table-caption lint violations in drilldown manager
UI","number":271377,"url":"https://github.com/elastic/kibana/pull/271377","mergeCommit":{"message":"Fix
@elastic/eui/require-table-caption lint violations in drilldown manager
UI (#271377)\n\nTwo `EuiInMemoryTable` instances in the drilldown
manager UI were\nmissing the required `tableCaption` prop,
violating\n`@elastic/eui/require-table-caption`.\n\n### Changes\n\n-
**`drilldown_template_table`** — Added `tableCaption` with i18n\nstring:
*\"Drilldown templates\"*\n- **`drilldown_table`** — Added
`tableCaption` with i18n
string:\n*\"Drilldowns\"*\n\n```tsx\n<EuiInMemoryTable\n
tableCaption={txtTableCaption}\n items={drilldowns}\n //
...\n/>\n```\n\nCaptions describe the dataset per EUI accessibility
guidelines and use\n`i18n.translate` for
localization.\n\n---------\n\nCo-authored-by: copilot-swe-agent[bot]
<198982749+Copilot@users.noreply.github.com>\nCo-authored-by: Alexey
Antonov
<alexwizp@gmail.com>","sha":"fa6e1f14d584a5fc26a2e62c3a4bbad2cb1c68ec"}},"sourceBranch":"main","suggestedTargetBranches":["9.3","9.4"],"targetPullRequestStates":[{"branch":"main","label":"v9.5.0","branchLabelMappingKey":"^v9.5.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/271377","number":271377,"mergeCommit":{"message":"Fix
@elastic/eui/require-table-caption lint violations in drilldown manager
UI (#271377)\n\nTwo `EuiInMemoryTable` instances in the drilldown
manager UI were\nmissing the required `tableCaption` prop,
violating\n`@elastic/eui/require-table-caption`.\n\n### Changes\n\n-
**`drilldown_template_table`** — Added `tableCaption` with i18n\nstring:
*\"Drilldown templates\"*\n- **`drilldown_table`** — Added
`tableCaption` with i18n
string:\n*\"Drilldowns\"*\n\n```tsx\n<EuiInMemoryTable\n
tableCaption={txtTableCaption}\n items={drilldowns}\n //
...\n/>\n```\n\nCaptions describe the dataset per EUI accessibility
guidelines and use\n`i18n.translate` for
localization.\n\n---------\n\nCo-authored-by: copilot-swe-agent[bot]
<198982749+Copilot@users.noreply.github.com>\nCo-authored-by: Alexey
Antonov
<alexwizp@gmail.com>","sha":"fa6e1f14d584a5fc26a2e62c3a4bbad2cb1c68ec"}},{"branch":"9.3","label":"v9.3.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.4","label":"v9.4.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y:agent-pr PR has been created by the agent backport:version Backport to applied version labels 💝community Feature:Drilldowns Embeddable panel Drilldowns Feature:Embedding Embedding content via iFrame Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes v9.3.5 v9.4.2 v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[eslint-issue-reporter] Fix @elastic/eui/require-table-caption lint violations across @elastic/kibana-presentation files

4 participants