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

switch to old tab if active is closed #37

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

forivall
Copy link

@forivall forivall commented Oct 18, 2023

Fixes #36 (scenario 2)

I also add a bunch of jsdoc and such so typescript can understand the code. (also, changed substr to slice, since substr is deprecated). Could use substring instead, but i prefer slice). If undesired, we can just merge the commit with the change: main...forivall:prevent-duplicate-tabs:feature/switch-to-existing

Copy link

@gabriel-r gabriel-r left a comment

Choose a reason for hiding this comment

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

@forivall I'd be happy to test and review this but my skills are limited.
Can you push a branch that I can easily check out?
PS: Or, teach me how to get the code on my machine.

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.

Redirect to existing tab
2 participants