-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Minimum browser versions? (i.e. minimum IE version?) #324
Comments
Minimum browser support still is important as some of peoples (including me) uses starts at (see below): Currently in my country statistic for browser/device usage, that matters:
* -
|
Can you share idea for class? Maybe i am implement it in ES6 version. Thanks |
@dalisoft I'll make a PR soon, so you can see what it looks like. |
@dalisoft Alright, here's a PR: #325, all tests passing. Instead of a private WeakMap and doing |
Oh, wait, |
How do you feel about just using the underscore convention? |
Here's another PR that uses the underscore convention: #326 |
There we go, removed |
I am fine with the underscore convention for private variables. |
Closing, let's eventually add a transpile step that can support older browsers. |
Hey ya'll! I'm circling back to this, and would like to make some improvements and also fix some long-standing bugs. What are the oldest browsers we need to support nowadays? (Personally I wish we could support Edge/Chrome/Firefox/Safari, and just drop IE). But maybe we can have a "legacy" build just for the ancient browsers anyways, like |
Hi all, what are your thoughts on, f.e., supporting only IE11+?
This would allow us to use
WeakMap
for private properties on theTween
class, and then we can store public methods on the prototype.The text was updated successfully, but these errors were encountered: