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

Potential fix for #38 #45

Open
wants to merge 250 commits into
base: safari
Choose a base branch
from

Conversation

geoffdutton
Copy link

@geoffdutton geoffdutton commented May 16, 2017

I noticed when I inspect the global page and run window.location.reload(), it's as though all the tabs lose reference to the background page. So one idea would be to reload each tab as we build out the vAPI.tabs.stack object. Maybe this has something to do with #38?

However, this could be a problem if for some reason it kept restarting, or if it restarted when someone was in the middle of something.

Just an idea.

@geoffdutton
Copy link
Author

Instead of reloading the whole tab, we can just make sure they've been added to the tabContexts. Otherwise, it'll return the behind-the-scenes url until the tab is reload.

@madeline03
Copy link

Yes

@el1t el1t force-pushed the safari branch 7 times, most recently from 4e2a6da to 263502f Compare January 13, 2018 22:55
@ruchernchong
Copy link

You might want to rebase and fix the errors.

gorhill and others added 30 commits March 30, 2018 15:40
# Conflicts:
#	.github/ISSUE_TEMPLATE.md
#	CONTRIBUTING.md
#	README.md
#	dist/README.md
#	src/about.html
#	src/js/dynamic-net-filtering.js
#	src/settings.html
Instead of chrome’s manifest version
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