Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Method to get data from canvas elements charts #22142

Closed
Tracked by #300
marius-dr opened this issue Aug 17, 2018 · 3 comments
Closed
Tracked by #300

Method to get data from canvas elements charts #22142

marius-dr opened this issue Aug 17, 2018 · 3 comments
Labels
Feature:Timelion Timelion app and visualization Team:Monitoring Stack Monitoring team Team:Visualizations Visualization editors, elastic-charts and infrastructure test tests_needed triage_needed

Comments

@marius-dr
Copy link
Member

marius-dr commented Aug 17, 2018

Right now there is no way to get data from a chart created with the canvas elements (not the Canvas app) from our automation tools. And from what we can tell there is no way for a screen reader to get the data from those charts.
What we would propose is to have a something like a hidden element with the values from the charts that a screen reader could access and that we could also access from the automation framework.

We're open for any other suggestions on how to make the chart data accessible.

cc @LeeDr

@marius-dr marius-dr added test Feature:Timelion Timelion app and visualization Feature:Visualizations Generic visualization features (in case no more specific feature label is available) Project:Accessibility Team:Monitoring Stack Monitoring team labels Aug 17, 2018
@timroes timroes added Team:Visualizations Visualization editors, elastic-charts and infrastructure and removed Team:Visualizations Visualization editors, elastic-charts and infrastructure Feature:Visualizations Generic visualization features (in case no more specific feature label is available) labels Sep 13, 2018
@myasonik
Copy link
Contributor

I agree with the premise that making charts more screen reader friendly will also make them easier to test but I want to divest this issue from accessibility and leave it focused on testing.

There are other tickets in Kibana tracking the accessibility of charts (#11842 is a good broad issue) so I'm going to divest this one from accessibility.

I'm leaving it open in case testing folks want it for their own tracking but feel free to close if it's not needed.

@markov00
Copy link
Member

We have added a debugState flag, currently under the alpha tag, in charts to provide the internal representation of the data, lines, areas, bars, axis and legend of the chart: elastic/elastic-charts#834
We are using this to replace the existing functional tests that use to get the SVG elements heights, compare them with the axis element height to get the original data back.
We can extend to cover the accessibility providing the data not only as part of our status element attribute, but in a HTML table inside the chart element if preferred.

@timroes
Copy link
Contributor

timroes commented Mar 5, 2021

Since we're planning to switch using elastic-charts, I'll close this issue in favor of a11y (elastic/elastic-charts#300) and automated testing (#70176) issues related to elastic-charts.

@timroes timroes closed this as completed Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Timelion Timelion app and visualization Team:Monitoring Stack Monitoring team Team:Visualizations Visualization editors, elastic-charts and infrastructure test tests_needed triage_needed
Projects
None yet
Development

No branches or pull requests

6 participants