Conversation
|
Cloud deployment initiated, see credentials at: https://buildkite.com/elastic/kibana-deploy-cloud-from-pr/builds/493 |
|
@elasticmachine merge upstream |
…badge fixes an AXE aria-command-name violation
- fixes an AXE aria-command-name violation - translations are missing
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
|
I cannot be certain but the failing FTRs look flaky and unrelated to the changes in this PR. In all cases it's like there's a delta of 2 in an array resulting of an api call (?):
I'm opening for review. Any hints more than welcome. |
|
Pinging @elastic/eui-team (EUI) |
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
cc @acstll |
peteharverson
left a comment
There was a problem hiding this comment.
ML changes with the internationalized aria-labels for the EuiBetaBadge LGTM
awahab07
left a comment
There was a problem hiding this comment.
obs-ux-logs changes LGTM! Just two comment lines are removed.
@elastic/eui:v107.0.0⏩v107.0.1@elastic/eslint-plugin-eui:v2.4.0⏩v2.5.0Questions? Please see our Kibana upgrade FAQ.
Changes
Related to elastic/eui#9100
aria-labelforEuiBetaBadgenode scripts/i18n_check— is there anything else that should be done in this regard?Package updates
@elastic/euiv107.0.1Bug fixes
prismjstheme inEuiCodeBlockto improve highlighting for theyamllanguage (#9089)EuiTablewhere the border for rows in dark mode wasn't applied correctly (#9115)Dependency updates
@elastic/prismjs-esqlto v1.1.2 (#9102)Accessibility
EuiIconandEuiBetaBadge(#9100)EuiBasicTablerespect user's reduced motion setting by not animating when in loading state. (#9095)@elastic/eslint-plugin-euiv2.5.0accessible-interactive-elementrule. (#9093)tooltip-focusable-anchorrule. (#9051)EuiButtonEmptyfrom theno-unnamed-interactive-elementrule. (#9046)Bug fixes
no-css-colorrule to allow CSS keywords likecurrentcolor,transparent, andinherit(#9092)