Skip to content

[Advanced settings] Fix boolean input accessibility#218961

Merged
SoniaSanzV merged 7 commits intoelastic:mainfrom
SoniaSanzV:advanced_settings-boolean_input_accessibility
Apr 29, 2025
Merged

[Advanced settings] Fix boolean input accessibility#218961
SoniaSanzV merged 7 commits intoelastic:mainfrom
SoniaSanzV:advanced_settings-boolean_input_accessibility

Conversation

@SoniaSanzV
Copy link
Copy Markdown
Contributor

@SoniaSanzV SoniaSanzV commented Apr 23, 2025

Closes: #217913

Description
Set correct label for Advanced Settings boolean input component.

@SoniaSanzV SoniaSanzV added Feature:Kibana Management Feature label for Data Views, Advanced Setting, Saved Object management pages Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor labels Apr 23, 2025
@SoniaSanzV SoniaSanzV requested a review from alexwizp April 23, 2025 13:05
@SoniaSanzV SoniaSanzV self-assigned this Apr 23, 2025
@SoniaSanzV SoniaSanzV requested a review from a team as a code owner April 23, 2025 13:05
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-management (Team:Kibana Management)

Comment thread x-pack/platform/plugins/private/translations/translations/fr-FR.json Outdated
@SoniaSanzV SoniaSanzV force-pushed the advanced_settings-boolean_input_accessibility branch from 2090f1f to c3f348c Compare April 24, 2025 09:53
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #21 / TemplateTags renders template tags

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
advancedSettings 35.2KB 35.1KB -163.0B
apm 2.6MB 2.6MB -172.0B
infra 1.2MB 1.2MB -172.0B
observabilityAiAssistantManagement 84.0KB 83.8KB -172.0B
profiling 370.9KB 370.7KB -172.0B
telemetryManagementSection 30.3KB 30.1KB -172.0B
total -1023.0B

History

cc @SoniaSanzV

Copy link
Copy Markdown
Contributor

@mattkime mattkime left a comment

Choose a reason for hiding this comment

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

code changes look good and work well

@SoniaSanzV SoniaSanzV merged commit cbf239a into elastic:main Apr 29, 2025
9 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 9.0

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

@SoniaSanzV SoniaSanzV deleted the advanced_settings-boolean_input_accessibility branch April 29, 2025 06:48
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Apr 29, 2025
Closes: elastic#217913

**Description**
Set correct label for Advanced Settings boolean input component.

---------

Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
(cherry picked from commit cbf239a)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
9.0

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 Apr 29, 2025
…219484)

# Backport

This will backport the following commits from `main` to `9.0`:
- [[Advanced settings] Fix boolean input accessibility
(#218961)](#218961)

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

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

<!--BACKPORT [{"author":{"name":"Sonia Sanz
Vivas","email":"sonia.sanzvivas@elastic.co"},"sourceCommit":{"committedDate":"2025-04-29T06:47:52Z","message":"[Advanced
settings] Fix boolean input accessibility (#218961)\n\nCloses:
https://github.com/elastic/kibana/issues/217913\n\n**Description**\nSet
correct label for Advanced Settings boolean input
component.\n\n---------\n\nCo-authored-by: Alexey Antonov
<alexwizp@gmail.com>","sha":"cbf239a512b2787f71fc8efb949af3cc52a02730","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Kibana
Management","Team:Kibana
Management","release_note:skip","backport:prev-minor","v9.1.0"],"title":"[Advanced
settings] Fix boolean input
accessibility","number":218961,"url":"https://github.com/elastic/kibana/pull/218961","mergeCommit":{"message":"[Advanced
settings] Fix boolean input accessibility (#218961)\n\nCloses:
https://github.com/elastic/kibana/issues/217913\n\n**Description**\nSet
correct label for Advanced Settings boolean input
component.\n\n---------\n\nCo-authored-by: Alexey Antonov
<alexwizp@gmail.com>","sha":"cbf239a512b2787f71fc8efb949af3cc52a02730"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/218961","number":218961,"mergeCommit":{"message":"[Advanced
settings] Fix boolean input accessibility (#218961)\n\nCloses:
https://github.com/elastic/kibana/issues/217913\n\n**Description**\nSet
correct label for Advanced Settings boolean input
component.\n\n---------\n\nCo-authored-by: Alexey Antonov
<alexwizp@gmail.com>","sha":"cbf239a512b2787f71fc8efb949af3cc52a02730"}}]}]
BACKPORT-->

Co-authored-by: Sonia Sanz Vivas <sonia.sanzvivas@elastic.co>
Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request May 29, 2025
Closes: elastic#217913

**Description**
Set correct label for Advanced Settings boolean input component.

---------

Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Kibana Management Feature label for Data Views, Advanced Setting, Saved Object management pages release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// v9.0.1 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Platform:StackManagement:Kibana:AdvancedSettings] Config values don't get announced on the toggle switch

5 participants