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

Replacing Intersection Observer approach #13

Open
alganzory opened this issue Oct 26, 2023 · 0 comments
Open

Replacing Intersection Observer approach #13

alganzory opened this issue Oct 26, 2023 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@alganzory
Copy link
Owner

I am thinking to replace the intersection observer approach by either:
1- relying solely on the mutation observer, the drawbacks are that it doesn't prioritize the current view, it doesn't respond to images being resized/expanded, etc.
2- intercepting network requests and processing them in the background script: I am currently not very clear (rather confused) about the persistence of the background script in manifest v3, so much conflicting and confusing info online about this so I am not sure, also videos are not going to work well with this cause it's hard to send video data between background and content scripts so I'll have to load the models once in the background script and then once for every tab (content script) to handle videos

@alganzory alganzory added help wanted Extra attention is needed enhancement New feature or request labels Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant