Radar Chart label color accessibility

apache / superset

Apache Superset is a Data Visualization and Data Exploration Platform

https://superset.apache.org/

Apache License 2.0

62.95k stars 13.93k forks source link

Radar Chart label color accessibility #27234

Open niall-forrest opened 9 months ago

niall-forrest commented 9 months ago

Screenshot

Description

The Label colour in the Radar Chart is not very legible on the default background, and fails color accessibility standards.

Design input

I would suggest aligning the Radar Chart label color with the other chart types. Aware there is ongoing work around theming which I hope will address label customization, however given this is an accessibility issue it may be a nice quick win.

design:suggest design:accessibility

rusackas commented 9 months ago

This isn't a highly used chart by most people/orgs, so while this issue is valid, it's not likely to get attention anytime soon. Would you be interested in opening a PR to adjust those grays to an acceptably contrast-y (Superset theme based) color?value

niall-forrest commented 9 months ago

Yep no worries, I'll look into it and open a PR

rusackas commented 4 months ago

Hi @niall-forrest just checking in to see if that PR is still forthcoming. If not, we might close this as stale before long. Just trying to keep the backlog as actionable/relevant/prioritized as possible :D