-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Tests #806
Comments
+1 |
Tests are good +1 |
Generally speaking, I support IE8+. That said, what did you have in mind for tests? Been trying to figure out the best way to do it. There are so many combinations of settings its almost overwhelming. Plus unit tests don't always cover visuals/browser quirks. |
Not with that attitude they don't. |
@austinpray lets see what youre working with sir, I'm all ears. |
@kenwheeler excellent! It won't come quick but I'm working on it. |
👍 |
@austinpray any traction on this? |
Working on this independently |
Any updates on tests? |
I noticed there is no definitive word on browser compatibility.
Would you guys be interested in some unit and or e2e tests? I could do the heavy-lifting on this if need be.
The text was updated successfully, but these errors were encountered: