Skip to content

Execute overlay renderer in source frame#30

Merged
lukasopacity merged 3 commits into
mainfrom
lukas/exec-iframe
Jun 9, 2026
Merged

Execute overlay renderer in source frame#30
lukasopacity merged 3 commits into
mainfrom
lukas/exec-iframe

Conversation

@lukasopacity

@lukasopacity lukasopacity commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

No description provided.


#if DEBUG
if (@available(iOS 16.4, *)) {
self.webView.inspectable = YES;

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets me using dev tools while the app is running, useful for debugging overlay

@adrianopacity adrianopacity left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Honestly looks good, as long as you ve tested on at least one past flow that uses the browser.eval_js

Comment thread src/objc/ModalWebViewController.mm Outdated
}

return opacityStringFromOwnedCString(getter());
}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh so weak attribute didn't work, did it? unlucky
But yeah looks good ig, as long as you tested, all g

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, on android did work but it doesn't work here? Or u forgot to update it here?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will do it

@lukasopacity lukasopacity merged commit f989881 into main Jun 9, 2026
1 check failed
oscar-opacity pushed a commit that referenced this pull request Jun 29, 2026
This reverts commit f989881, reversing
changes made to 1c7fee3.
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