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

Addresses accessibility issues for privacy policy #78

Closed
wants to merge 1 commit into from

Conversation

jordanAtShopify
Copy link
Member

@jordanAtShopify jordanAtShopify commented Jun 1, 2020

Adds better accessibility for markdown rendered nodes for Link and Header in the privacy policy.

Fixes #40

@jordanAtShopify
Copy link
Member Author

@svinkle let me know if this is any better, I tried adding the external link icon but it broke the link functionality - do you think we are good to skip the icon for this?

@svinkle
Copy link
Member

svinkle commented Jun 2, 2020

@jordanAtShopify The code looks good to me but links are not being generated as I'd expect. Neither are the headings. Is this overriding what the markdown component is generating?

@jordanAtShopify
Copy link
Member Author

@svinkle this should just be overriding the default rendering behavior of the react-native-markdown-display package. When you say Links / Headers not being generated as expected you mean that you don't see the accessibility additions when using voice over tools?

@svinkle
Copy link
Member

svinkle commented Jun 3, 2020

you don't see the accessibility additions when using voice over tools

Exactly. The very top header has the correct accessibilityRole applied, but others do not. Also, links still appear to be purely underlined text. They should be singled out as interactive elements.

You can test this by going to Xcode → Open Developer Tool → Accessibility Inspector → click the cross-hair icon → hover over the content in question.

Screenshots

Screen Shot 2020-06-03 at 11 23 56 AM
Screen Shot 2020-06-03 at 11 24 08 AM

Copy link
Member

@henrytao-me henrytao-me left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to move privacy to json file instead of using .js file?

Regarding to accessibility of link using markdown, we might need to either send PR to react native markdown library or switch to html 🤔

@jordanAtShopify
Copy link
Member Author

@henrytao-me what difference would having it in json make?

@henrytao-me
Copy link
Member

what difference would having it in json make?

One central place for translation?

@jordanAtShopify
Copy link
Member Author

Ah, ya the issue with the json file is that it doesn't support multiline strings and markdown depends on that for formatting. Also, we want it to be as easy as possible to just copy/paste the privacy policy in so we want to avoid any manual formatting steps and keep it close as possible to the original markdown format.

Copy link
Member

@henrytao-me henrytao-me left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we move the .js from assets to ./src/locale/extras?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Privacy] Links are not focusable
3 participants