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 font color assessment #238

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

Added font color assessment #238

wants to merge 3 commits into from

Conversation

mlewand
Copy link
Contributor

@mlewand mlewand commented Feb 14, 2017

This PR fixes issue #237.

@Tade0
Copy link
Contributor

Tade0 commented Feb 14, 2017

Chrome and Firefox look good, in IE11 there's an alert that there are no accessibility issues, which is unexpected.

No error in the console to indicate what could be wrong though.

@mlewand
Copy link
Contributor Author

mlewand commented Mar 27, 2018

I have rebased branch onto the latest master branch. I can see that the issue mentioned in comment above is still valid, although it works with Edge - so IE11 is the only browser that does not support that.

It should be checked whether this could be verified on IE11.

@mlewand mlewand removed the request for review from Tade0 March 27, 2018 16:42
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