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

Added accessibilityEnabled to allow the accessibility feature to be disabled #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added accessibilityEnabled to allow the accessibility feature to be disabled #35

wants to merge 2 commits into from

Conversation

peter-lewis
Copy link

Added accessibilityEnabled to allow the accessibility feature to be disabled

Added accessibilityEnabled to enable the accessibility to be disabled
Added some fixes to handle leading and trailing spaces on lines properly
@jedateach
Copy link

jedateach commented Jan 7, 2020

Hi Peter - just wondering if you could update your PR description to explain why disabling accessibility is important?

@peter-lewis
Copy link
Author

Wow, time machine on this one!
If I remember correctly, there were bugs with the implementation which potentially caused older browsers to blow up (gotta love IE) when it would try to render the txt, but I might be forgetting? This was from just about 2 years ago!

@jedateach
Copy link

Yeah fair enough. I'm just expanding my understanding of the library to decide whether to adopt it.

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.

2 participants