Skip to content

[7.x] [Logs UI][a11y] Announce name of column on remove column button (#41695)#42529

Merged
Zacqary merged 1 commit intoelastic:7.xfrom
Zacqary:backport/7.x/pr-41695
Aug 2, 2019
Merged

[7.x] [Logs UI][a11y] Announce name of column on remove column button (#41695)#42529
Zacqary merged 1 commit intoelastic:7.xfrom
Zacqary:backport/7.x/pr-41695

Conversation

@Zacqary
Copy link
Copy Markdown
Contributor

@Zacqary Zacqary commented Aug 2, 2019

Backports the following commits to 7.x:

…tic#41695)

* [Logs UI][a11y] Announce name of column on remove column button

* Use title instead of aria-label

* ariaColumnName => columnDescription

* Move template string out of i18n

* Revert label id change

* Inject i18n to field title

* Change title to Remove {columnDescription} column

* Replace injectI18N with i18n.translate

* Fix i18n
@Zacqary Zacqary added the backport This PR is a backport of another PR label Aug 2, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@Zacqary Zacqary merged commit 6b6e979 into elastic:7.x Aug 2, 2019
@Zacqary Zacqary deleted the backport/7.x/pr-41695 branch August 2, 2019 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants