[openstreetmap/openstreetmap-website] Replace augment.js with a modern set of polyfills where needed (#2125)

Tom Hughes notifications at github.com
Thu Jan 31 08:38:19 UTC 2019


I deliberately used a whitelist, after some consideration.

You of course correct that it's impossible to know the set of browsers that don't need it. It's equally impossible to know the set that do however, and as the polyfills do nothing when not needed I decided it was safer to err on the side of caution and send them when they might be needed - not sending them is basically just a performance optimisation.

I would also suggest that a substantial proportion of "other browsers" that would count as modern enough are in fact WebKit based anyway and would therefore be covered by this whitelist unless they mask that fact from their user agent.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2125#issuecomment-459261097
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20190131/f245ddef/attachment.html>


More information about the rails-dev mailing list