Conversation
This removes the requirement on ES6 support in the browser, which will allow to use unsee on older browsers
Tenzer
left a comment
There was a problem hiding this comment.
It might be good to establish somewhere what range of browsers we aim to support for any contributors. Usually terms like "the current version plus 1/2 older" works.
|
Without a CI job that would test this it's going to be very hypothetical |
|
It would, but it at least still documents what the intended range of browser support is instead of leaving people guessing. It also means that people who experience problems on an older browser can report problems if it's inside the range of those we intent to support, rather than just assume it doesn't work due to their old browser being unsupported. |
|
I don't know what that range currently is or should be. Even if I pick some random browser I have no way of knowing if any new change breaks any of those browsers. |
This removes the requirement on ES6 support in the browser, which will allow to use unsee on older browsers