-
Notifications
You must be signed in to change notification settings - Fork 54
fix a11y Talkback reading nested spans incorrectly in in psammead-brand #4586
Conversation
serviceLocalisedName ? ( | ||
// id={`BrandLink-${linkId}` is a temporary fix for the a11y nested span's bug experienced in TalkBack, refer to the following issue: https://github.com/bbc/simorgh/issues/9652 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had a look at the issue. Can I ask why this is a temporary fix? Is it only temporary until Google/TalkBack provide a fix or is it a temporary fix for us over a fixed period? Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
until Google/Talkback fixes it. But we think they are going to take a very long time. I will specify this in the issue link.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well it could take a long time for Google to fix, this bug has been present for some time, though we hope they might fix it soon, it's out of our control and really affecting the user experience of our product, so we have a temporary fix until such time that Google fix it.
Resolves #9561
Overall change:
fix a11y Talkback reading nested spans incorrectly in psammead-brand
Code changes:
-concatenate span content