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

Fix embedded iframe #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Yambr
Copy link

@Yambr Yambr commented May 11, 2021

before fix - main frame on load navigated to iframe url

now - if webview has iframe - its will be ignored


steps to reproduce
add headers to url with embedded iframe for example https://yambr.ru/test/


some screenshots before and after fix

  1. before module is redirecting to embedded iframe
    photo_2021-05-11_00-13-08
  2. add some code to js library
    photo_2021-05-11_00-13-14
  3. fixed!
    photo_2021-05-11_00-06-44

Yambr added 2 commits May 11, 2021 00:15
before fix - main frame on load navigated to iframe url

now - if webview has iframe - its will be ignored
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.

None yet

1 participant