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

& displayed in toolip in Firefox browser when accessibility module included #7383

Closed
ff-sgupta opened this Issue Nov 9, 2017 · 5 comments

Comments

Projects
None yet
3 participants
@ff-sgupta

ff-sgupta commented Nov 9, 2017

Expected behaviour

Chart title set as "A&B", svg title should be set to "A&B"

Actual behaviour

svg title is set to "A&B"

Live demo with steps to reproduce

http://jsfiddle.net/hogztz4o/

Access the above link in Firefox browser.
Hover over the chart. Observe tooltip displayed as "A&B" since svg title has text "A&B"

Affected browser(s)

Firefox

@pawelfus

This comment has been minimized.

Contributor

pawelfus commented Nov 9, 2017

Hi @ff-sgupta

Thank you for reporting the bug. I was able to replicate the issue in Firefox 56.0.2, attaching screenshot:

screen shot 2017-11-09 at 11 26 57

@oysteinmoseng

@oysteinmoseng oysteinmoseng self-assigned this Nov 9, 2017

@ff-sgupta

This comment has been minimized.

ff-sgupta commented Feb 23, 2018

@oysteinmoseng do we have any ETA for this issue?

@ff-sgupta ff-sgupta closed this Feb 23, 2018

@ff-sgupta ff-sgupta reopened this Feb 23, 2018

@pawelfus

This comment has been minimized.

Contributor

pawelfus commented Feb 23, 2018

@oysteinmoseng

This comment has been minimized.

Collaborator

oysteinmoseng commented Feb 23, 2018

This is a combination issue, firstly of encoding the characters in the title element, and secondly we need a method for configuring this text. I have suggested a fix for this, and we should have this solved by the next release.

With the suggested fix you can set the lang.accessibility.svgContainerTitle option to empty to disable the title entirely. We also have modified the encoding logic so that & isn't shown as &.

@ff-sgupta

This comment has been minimized.

ff-sgupta commented Feb 26, 2018

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment