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

Check that iframe ref is not null before changing its style #1511

Open
wants to merge 1 commit into
base: stable
Choose a base branch
from

Conversation

n1313
Copy link
Contributor

@n1313 n1313 commented Mar 7, 2022

This PR fixes (?) a rare case when instagram preview throws an exception: https://freefeed.net/support/c30f53b2-e78e-4ba3-a9aa-121a499bc723. I wasn't able to reproduce the bug.

@github-actions
Copy link

github-actions bot commented Mar 7, 2022

🎊 PR Preview 81f278d has been successfully built and deployed to https://FreeFeed-freefeed-react-client-preview-pr-1511.surge.sh

🕐 Build time: 256.303s

🤖 By surge-preview

@davidmz
Copy link
Member

davidmz commented Mar 7, 2022

Я всё же попробовал бы сначала воспроизвести. Насколько я понимаю реакт, в этом месте null не может быть

@n1313
Copy link
Contributor Author

n1313 commented Mar 7, 2022

Я тоже не понимаю, каким образом эта ошибка могла произойти на практике, но формально сразу после useRef(null) значение iframeRef.current должно быть равно null, разве нет? Плюс трейс со скриншота указывает именно на это место в коде

@davidmz
Copy link
Member

davidmz commented Mar 7, 2022

Но функция в useEffect вызывается не «сразу после», она вызывается после первого рендера (и после каждого изменения url, но тут это не должно влиять). После первого рендера iframe существует, потому что он рендерится всегда, нет какого-то if-а, который может его рендерить, а может нет. Ну, то есть, насколько это я понимаю.

@n1313
Copy link
Contributor Author

n1313 commented Mar 7, 2022

Ты, конечно, прав. Не знаю, как это случилось и почему трейс указывает туда, куда он указывает

@indeyets
Copy link
Contributor

это не может быть связано с баннерорезками какими-то? у меня, например, файрфокс отказывается открывать инстаграм в фидике потому что вся Meta выделена в отдельный «контейнер»

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 this pull request may close these issues.

3 participants