Skip to content
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

Не работает Modal в IE #211

Closed
2 tasks
ransoq opened this issue Aug 22, 2022 · 14 comments · Fixed by #245
Closed
2 tasks

Не работает Modal в IE #211

ransoq opened this issue Aug 22, 2022 · 14 comments · Fixed by #245

Comments

@ransoq
Copy link

ransoq commented Aug 22, 2022

Опишите проблему

При открытии модального окна возникает ошибка в IE 7-11 после обновления core до актуальной версии

Ожидаемое поведение

При нажатии на кнопку должно появится модальное окно в IE 7-11

Чек лист

  • Тесты
  • Документация

Внешний вид

Ожидаемый Фактический
image telegram-cloud-photo-size-2-5434125544003321768-y

Тестовый стенд

Десктоп (если данных нет оставьте блок пустым):

  • OS: Windows
  • Browser: IE
  • Version: 7-11

Смартфон (если данных нет оставьте блок пустым):

Дополнительная информация

Дополнительная информация

@RG100-EMUX
Copy link

RG100-EMUX commented Aug 25, 2022

IE 7-11 уже канули в лету, их использует очень низкий процент людей, насколько релевантно поддерживать эти версии IE? Можно на matches в своем проекте добавить полифил отсюда https://developer.mozilla.org/ru/docs/Web/API/Element/matches

@ransoq
Copy link
Author

ransoq commented Aug 25, 2022

У подавляющего числа сотрудников браузером по умолчанию является Internet Explorer, из-за чего отказаться от его поддержки, на текущем этапе, не представляется возможным

@jkikksa
Copy link
Contributor

jkikksa commented Aug 25, 2022

примерно 25% пользователей портала AlfaPeople заходят из IE11 и дропать его поддержку мы не можем

@jkikksa
Copy link
Contributor

jkikksa commented Aug 25, 2022

полифил на matches не помогает. Опытным путём определил, что модалка начинает падать с версии 27.10.0, но не понимаю, что к этому могло привести )
v27.9.0...v27.10.0

@RG100-EMUX
Copy link

RG100-EMUX commented Aug 26, 2022

Добавление каретки автоматически может брать версии, например если ^3.9.2 зависимость, где версии будут 3.x.x, что потенциально может поломать в минорных версиях. Потенциально может быть в этом причина, но самостоятельно я проверить к сожалению не могу. А по стектрейсу можно как-то определить, в каком модуле происходит и на какой строчке?

@SiebenSieben
Copy link
Contributor

SiebenSieben commented Aug 26, 2022

Фиксим в IE11, более старые версии в этой библиотеке изначально не поддерживаем

@jkikksa
Copy link
Contributor

jkikksa commented Sep 1, 2022

опытным путём обнаружил, что проблема в обновлении react-focus-lock c 2.5.0 до 2.9.1

react-focus-lock@^2.5.0:
  version "2.9.1"

@v-gevak
Copy link
Contributor

v-gevak commented Sep 7, 2022

Для того, чтобы модальные окна работали в IE, необходимо установить полифил для element matches, например вот этот https://www.npmjs.com/package/element-matches-polyfill.

@v-gevak v-gevak linked a pull request Sep 7, 2022 that will close this issue
@jkikksa
Copy link
Contributor

jkikksa commented Sep 7, 2022

проблема не в element matches, а в обновлении версии react-focus-lock до 2.9.1
Вот код, на котором падает
image
image

мы используем следующий workaround

"resolutions": {
    "react-focus-lock": "2.5.0"
},

@v-gevak
Copy link
Contributor

v-gevak commented Sep 7, 2022

FocusLock использует под капотом element matches(https://github.com/theKashey/focus-lock/blob/master/src/utils/tabUtils.ts#L12). Я протестировал в IE, проблем не было. Значит у вас не хватает других полифилов.

@jkikksa
Copy link
Contributor

jkikksa commented Sep 7, 2022

а какие у тебя установленны полифилы? ) у нас используются
react-app-polyfill/ie11 и react-app-polyfill/stable

кстати вот изменения в react-focus-lock theKashey/react-focus-lock@v2.5.0...v2.9.0 после которых IE крашится, видимо тут тоже деле в какой-то зависимости глубже )

@v-gevak
Copy link
Contributor

v-gevak commented Sep 7, 2022

@v-gevak
Copy link
Contributor

v-gevak commented Sep 7, 2022

C react-app-polyfill/ie11 и react-app-polyfill/stable тоже открылась
image

@v-gevak
Copy link
Contributor

v-gevak commented Sep 7, 2022

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants