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

Simple Tab Groups refuses Vimium C to open tabs in "No container" #198

Closed
Zeioth opened this issue Jul 18, 2020 · 18 comments
Closed

Simple Tab Groups refuses Vimium C to open tabs in "No container" #198

Zeioth opened this issue Jul 18, 2020 · 18 comments

Comments

@Zeioth
Copy link

Zeioth commented Jul 18, 2020

Problem:
There's no way to dismiss the alert forever.

How to reproduce:

  • Install the plugins 'VimiumC' and 'Simple tab groups'.
  • Create a tab group and select it.
  • On any website, press the key 'B', write some text to search a marker, and press ctrl+shift+enter to open it in a new tab.

Everytime you do it, you'll see the next window.
Captura de pantalla_2020-07-18_17-01-50

What should happen: Give the user an option to remember the selection, and don't show the page again.

Related links:

@gdh1995
Copy link
Owner

gdh1995 commented Jul 27, 2020

I've reproduced this.

This is a logic conflict between common extensions who want to create tabs, like Vimium C, and the Simple Tab Groups.

According to code, Vimium C can not affect the SimpleTabGroups's process of forcing new tabs created in a special tab container. So you'll have to wait for the author to fix Drive4ik/simple-tab-groups#655.

@gdh1995 gdh1995 changed the title This may or not be Vimium-C's fault Simple Tab Groups refuses Vimium C to open tabs in "No container" Jul 28, 2020
gdh1995 added a commit that referenced this issue Aug 21, 2020
@gdh1995
Copy link
Owner

gdh1995 commented Aug 21, 2020

Hello, I got an idea to solve this conflict, and please try this version of Vimium C: vimium_c-1.84.3-firefox-a2a88e7.zip

Installation: see https://github.com/gdh1995/vimium-c/wiki/Test-a-zip-package .

With this version, you'll need to change mappings of createTab (add opener option to those mapping lines):

map *** createTab opener

This is to ensure the newly tab belongs to a same container of a current tab's.

@Zeioth
Copy link
Author

Zeioth commented Aug 21, 2020

Extension Last Updated
August 21, 2020

aaa

  • I clicked "save".
  • I opened a new page
  • I press B to load from bookmarks
  • I press ctrl+shift to load it in a new tab.

But I still get the message. Perhaps my mappings were not correct?

@gdh1995
Copy link
Owner

gdh1995 commented Aug 22, 2020 via email

@gdh1995
Copy link
Owner

gdh1995 commented Aug 22, 2020

Please try this: vimium_c-1.84.3-firefox-9555c57.zip

  • unzip and load it
  • open options page, go to "Vomnibar settings"
  • modify the "actions" field: add opener to it
    • the result will be like "actions": "opener"
  • try Vomnibar again

The above only fixes URLs from Vomnibar would be blocked, and you'll need to re-map all commands which are related about "open a URL" and add a new option of opener to those mappings. It whould be like:

map t createTab opener
map P openCopiedUrlInNewTab opener
shortcut createTab opener
# some other mappings with "opener" option

Note: the code above is only an example, so you'll need to use your preferred keys to replace those t, P and so on.

@Zeioth
Copy link
Author

Zeioth commented Aug 22, 2020

I can confirm, that config fixes the issue! I've tried the next with the parameters of your last post and it works fine:

t
shift+p
b -> ctrl+shift+enter
o -> ctrl+shift+enter

Out of curiosity, I have a question:
There's some way to inmediately go to the tab you open with "b -> ctrl+shift+enter" and "o -> ctrl+shift+enter"?

@gdh1995
Copy link
Owner

gdh1995 commented Aug 22, 2020

On Vomnibar, Ctrl+Enter opens a suggestion item in a new tab and focuses it, and Ctrl+Shift+Enter will only open but not focus.

@Zeioth
Copy link
Author

Zeioth commented Aug 22, 2020

Feel free to close the issue now, or when you push the code. Thank you so much!!

@gdh1995
Copy link
Owner

gdh1995 commented Aug 23, 2020

Hello, the v1.85.0 has been released on Firefox Add-ons, and you may take a try.

@Zeioth
Copy link
Author

Zeioth commented Aug 23, 2020

Thank you for telling me! I've updated to the latest version, and tested i,t and it's working correctly as long as I write the settings you provided me. Maybe It would be a good idea to make them default? As long as there's no more issues involved.

@gdh1995
Copy link
Owner

gdh1995 commented Aug 23, 2020 via email

@Zeioth
Copy link
Author

Zeioth commented Aug 23, 2020

I just noticed, NewTabAdapter is also affected by this issue:

If I try to use any of the previously mentioned shortcuts from NewTabAdapter start page, I get the "Can't open url in the personal container" error. Also when I enter the starting page, if one is provided.

@gdh1995
Copy link
Owner

gdh1995 commented Aug 23, 2020 via email

@gdh1995
Copy link
Owner

gdh1995 commented Aug 26, 2020

Please test this: this zip has a bug. wait for a while.

Show advanced options, enable the "Open target URL in a same container (group)." and then test it with Simple Tab Groups enabled.

@gdh1995
Copy link
Owner

gdh1995 commented Aug 26, 2020

Please try this: newtab-1.2.0.zip.

Currently there's a limit: if Open target URL in a same container (group) is enabled, then Focus web page when creating a new tab will be auto enabled. It's because (I found) my current trick can not work with Simple Tab Groups if not moving focus.

@Zeioth
Copy link
Author

Zeioth commented Aug 26, 2020

I just tested it, I can confirm it works. I can't tell you how happy I am to have that annoying message out of my life. Thank you!!! I'll close the issue as soon as the new version is live.

@gdh1995
Copy link
Owner

gdh1995 commented Aug 26, 2020

NewTab Adapter v1.3.0 has been released.

@Zeioth Zeioth closed this as completed Aug 26, 2020
@waynehoover
Copy link

waynehoover commented Jun 21, 2023

For those coming here later, the solution to this is to add "actions": "opener" to your Vomnibar settings.

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

No branches or pull requests

3 participants