47еместо
uLogin - виджет авторизации через социальные сети « все идеи проекта

26

голосов

неправильный полифилл Array.prototype.indexOf

В вашем скрипте http://ulogin.ru/js/ulogin.js неправильно полифилится indexOf
В общем-то метод есть во всех современных браузерах
И для них в общем-то его не нужно полифилить, но родной метод принудительно подменяется на ваш полифил
Полагаю вместо кода
Array.indexOf || (Array.prototype.indexOf = function(a) {
Подразумевался
Array.prototype.indexOf || (Array.prototype.indexOf = function(a) {
При такой замене родной(быстрый) метод не будет заменяться на более медленный полифил, когда этого не требуется

Проблема была обнаружена в первую очередь, из-за того что виджет заменяет глобально метод indexOf, что влияет на производительность остального JS-кода

Виктор Гвоздев, 20.02.2017, 20:18
Статус идеи: ожидает рассмотрения

Комментарии

Иван Пшеницын, 21.02.2017, 09:45
Здравствуйте.

Да, вы абсолютно правы.

Проблему исправили, новая версия скрипта в релизе.

Спасибо, за то что сообщили!
Виктор Гвоздев, 21.02.2017, 23:18
И вам спасибо)

Оставить комментарий