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 tab restoration #1166

Merged
merged 3 commits into from
Oct 26, 2014
Merged

Fix tab restoration #1166

merged 3 commits into from
Oct 26, 2014

Conversation

mrmr1993
Copy link
Contributor

This adds the "sessions" permission to manifest.json, which enables actual tab restoration.

This PR should:

@mrmr1993
Copy link
Contributor Author

Rebased.

@smblott-github smblott-github merged commit f18f3ea into philc:master Oct 26, 2014
@smblott-github
Copy link
Collaborator

Excellent, thanks @mrmr1993.

Looks like the cost of this is:

  • We need an extra permission; but, if we, then so be it.
  • We lose the feature whereby certain non-vimium enabled tabs are skipped; but that's not obviously a great feature anyway.
  • If the sessions permission is requested by an unpacked extension on an older chrome version which doesn't support it, then you get an error message on the Extensions page, but the extension does work. @mrmr1993: Do we know what happens in this case if the extension comes from the Chrome Store?

However, restoring tabs without their history was always a bit hokey. So this is great to have.

@mrmr1993
Copy link
Contributor Author

I'll check on this and get back to you, I know nothing about the Chrome Web Store. @philc, do you know?

@philc
Copy link
Owner

philc commented Oct 27, 2014

@mrmr1993 I don't know what will happen in this case if Vimium is installed from the chrome store on an old version of chrome.

Out of curiosity, which version of Chrome was this introduced in?

@mrmr1993
Copy link
Contributor Author

@philc Chrome 37, I believe

@hpurmann
Copy link

I'm still having this issue of not providing the tab's history when restoring it, using Vimium 1.45 installed from the Chrome Web Store. My Chrome is up-to-date (currently version 38.0.2125.122).

However, the permissions in the chrome://extensions tab seem fine (see screenshot).

vimium_permissions

@smblott-github
Copy link
Collaborator

I'm still having this issue of not providing the tab's history when restoring it, using Vimium 1.45 installed from the Chrome Web Store.

The fix is merged into master but not yet released. It'll be in in the next release, which should be in the coming weeks.

@hpurmann
Copy link

Ah ok. Thx for the fast response. So you are not using some model like git flow? It would be nice to have a development branch and a release branch (master) with the released versions. Then if such an issue is fixed we could just merge the hotfix branch into master and release a new version (e.g. 1.45.1, last version number increases). Releases would be more frequent.

What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants