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

[Bug] using F opens links twice on Firefox Nightly #3964

Closed
s-marios opened this issue Nov 30, 2021 · 37 comments · Fixed by #3985
Closed

[Bug] using F opens links twice on Firefox Nightly #3964

s-marios opened this issue Nov 30, 2021 · 37 comments · Fixed by #3985

Comments

@s-marios
Copy link

Hi,

When I'm using shift + f to open a link in a new tab, instead of one tab two tabs with the same content appear.

I'm using Firefox Nightly and this started happening 1-2 days ago. Here's at least one more user experiencing the same problem #3579 (comment) .

Anybody else affected?

@gdh1995
Copy link
Contributor

gdh1995 commented Dec 3, 2021

I also noticed this days ago.

@Artturin
Copy link

Artturin commented Dec 4, 2021

im on nightly too

@Artturin
Copy link

Artturin commented Dec 7, 2021

https://github.com/gdh1995/vimium-c does not have this problem though a weird thing is that when i disable vimium-ff and then install vimium-c, the vimium-c will have the same problem but it is fixed after i restart firefox

(only tested once so may not be reproducible)

@gdh1995
Copy link
Contributor

gdh1995 commented Dec 7, 2021

@Artturin Vimium C uses a trick which is not safe on a loaded page, so it only enables this trick if it runs when a page begins loading. If you want to test Vimium C again, please refresh tabs after you enable it and before tests.

@evenlee
Copy link

evenlee commented Dec 8, 2021

I am using firefox 96.0b2 dev edition, have same problem

@JoshMock
Copy link

JoshMock commented Dec 9, 2021

Also experiencing this on 96.0b2 developer edition.

@sandeshghanta
Copy link

Also experiencing this issue in Firefox developer edition 96.0b4 version.

@pngdeity
Copy link

Experiencing the issue on Firefox Nightly 97.0a1

@rvighne
Copy link

rvighne commented Jan 8, 2022

This has also been happening in beta for a long time. Is a fix possible / planned as of now? FF 96 will hit the release channel very soon on 1/11 which will affect a lot more users.

@QHayward
Copy link

I installed Firefox today on a new computer, and can confirm that I am having this issue

@keepiru
Copy link

keepiru commented Jan 11, 2022

I'm experiencing this after upgrading to FF 96.0 today.

@mcauley-penney
Copy link

Just upgraded to 96.0 and also experiencing

@ashuradji
Copy link

Same error here, i just updated to FF 96. Only happens in Firefox

@philc
Copy link
Owner

philc commented Jan 13, 2022 via email

gdh1995 added a commit to gdh1995/vimium that referenced this issue Jan 13, 2022
It seems Firefox 96 allows extensions to trigger native click behaviors.
See philc#3964 for reports.

Although this is not listed in https://www.mozilla.org/en-US/firefox/96.0/releasenotes/,
I think it's safe enough to skip preventing since Firefox 96.
@gdh1995
Copy link
Contributor

gdh1995 commented Jan 13, 2022

@philc I've filed #3985 . And I think #3867 is also worth merging.

@R2robot
Copy link

R2robot commented Jan 15, 2022

Happening on the general public releases now. 96.0.1

15cm pushed a commit to 15cm/vimium that referenced this issue Jan 15, 2022
It seems Firefox 96 allows extensions to trigger native click behaviors.
See philc#3964 for reports.

Although this is not listed in https://www.mozilla.org/en-US/firefox/96.0/releasenotes/,
I think it's safe enough to skip preventing since Firefox 96.

(cherry picked from commit cf420fb)
usk pushed a commit to usk/vimium that referenced this issue Jan 18, 2022
It seems Firefox 96 allows extensions to trigger native click behaviors.
See philc#3964 for reports.

Although this is not listed in https://www.mozilla.org/en-US/firefox/96.0/releasenotes/,
I think it's safe enough to skip preventing since Firefox 96.
@cupcakearmy
Copy link

Same issue here on 96.0.1

1 similar comment
@snprajwal
Copy link

Same issue here on 96.0.1

@sarrietav-dev
Copy link

Same issue here on 96.0.1. I hope this gets fixed soon.

@freefal
Copy link

freefal commented Jan 19, 2022

Also seeing this issue.

philc pushed a commit that referenced this issue Jan 20, 2022
It seems Firefox 96 allows extensions to trigger native click behaviors.
See #3964 for reports.

Although this is not listed in https://www.mozilla.org/en-US/firefox/96.0/releasenotes/,
I think it's safe enough to skip preventing since Firefox 96.
@philc
Copy link
Owner

philc commented Jan 20, 2022

Thanks everyone for your patience. I've merged #3985 and will release to the extension stores later tonight.

@WillDHB
Copy link

WillDHB commented Jan 21, 2022

It looks like the Vimium extension has been updated, but I don't see anything for Vimium-FF...what is the difference between the two? It looks like they're both yours.

@Artturin
Copy link

Artturin commented Jan 21, 2022

@philc i think you pushed the update with a wrong extension name
https://addons.mozilla.org/en-US/firefox/addon/vimium/ was created yesterday

https://addons.mozilla.org/en-US/firefox/addon/vimium-ff/ is the original extension

@philc
Copy link
Owner

philc commented Jan 21, 2022 via email

@philc
Copy link
Owner

philc commented Jan 22, 2022 via email

@SafwanLjd
Copy link

@philc I still have this issue even after updating to v1.67.1 the same goes for https://addons.mozilla.org/en-US/firefox/addon/vimium/

@xstnztk
Copy link

xstnztk commented Jan 22, 2022

The update to v1.67.1 fixes the double-open issue for me but seems to have swapped the functionality of LinkHints.activateModeToOpenInNewTab and LinkHints.activateModeToOpenInNewForegroundTab, i.e. LinkHints.activateModeToOpenInNewForegroundTab now opens the tab in a new background tab and activateModeToOpenInNewTab opens it in a new foreground tab.

I restarted Firefox to make sure this was not a transient problem but it persists. TBH I do not see anything in the code modification which could cause this but I am not really a web extension developer.

AFAICT this seems to be caused by setting "When you open a link, image or media in a new tab, switch to it immediately" in the Firefox options, i.e. when this option is activated, the two Vimium functions are swapped. They only work as expected if the Firefox option is not active. OTOH I have this option set since ages and only experienced the incorrect Vimium behavior since the extension update.

For the time being I can work around the problem by simply using the "wrong" Vimium command. Should I file a new issue for this?

@gdh1995
Copy link
Contributor

gdh1995 commented Jan 22, 2022

I have this option set since ages and only experienced the incorrect Vimium behavior since the extension update.

@xstnztk It's because old versions of Vimium "simulates" OpenInNewTab and so on, so it didn't trigger the Firefox's built-in process which considers about the option.

This is a new issue: do we need a new command option for LinkHints.* to swap the ctrl and ctrl+shift sent by Vimium?

@s-marios
Copy link
Author

s-marios commented Jan 24, 2022

Hi,
indeed, 1.67.1 fixes the double open problem! Thank you for that.

As such I'm closing this bug as fixed. Cheers!
[edit: I hadn't had my cup of coffee, topic was already closed]

gdh1995 added a commit to gdh1995/vimium-c that referenced this issue Jan 24, 2022
@decadent
Copy link

decadent commented Jan 26, 2022

Coincidentally, this perturbance might've fixed a long-standing annoyance that new tabs were always opened by Vimium right after the currently active one—instead of one after another rightwards (and FF didn't provide APIs to open them in a proper order).

Unless some other change was made to that end.

@freefal
Copy link

freefal commented Jan 26, 2022

Agreed. This was always a minor annoyance for me as well. Now fixed.

@ashuradji
Copy link

Excuse me, this issue it's fixed in FF Developer Edition but still happens in Firefox to me

@gdh1995
Copy link
Contributor

gdh1995 commented Jan 27, 2022

@drk01 Sorry but what's the versions of your 2 Firefox and their Vimium?

@ashuradji
Copy link

ashuradji commented Jan 27, 2022

@gdh1995 FF 96.0.2 & FF Developer 97.0b3.
Vimium 1.67.1 in both browsers

@C4K3
Copy link

C4K3 commented Jan 28, 2022

Same here, still happening to me on 1.67.1. However if I install the extension in a fresh profile, then it doesn't happen on that profile. Will try to figure out what setting / extension is causing it.

@C4K3
Copy link

C4K3 commented Jan 29, 2022

It's privacy.resistFingerprinting, if I enable that then the two-tabs behavior occurs.

@gdh1995
Copy link
Contributor

gdh1995 commented Jan 30, 2022

Well, it's a same bug caused by fake navigator.userAgent - see #3989 (comment) . I'll file a new PR now.

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 a pull request may close this issue.