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] "Allow to open a new tab with the URL in the clipboard, by middle click on the "New Tab" button" not working #3569

Closed
karasuhebi opened this issue Jun 6, 2024 · 4 comments

Comments

@karasuhebi
Copy link

Abstract

The option "Allow to open a new tab with the URL in the clipboard, by middle click on the "New Tab" button" is not working.

Steps to reproduce

  1. Start Firefox with clean profile.
  2. Install TST.
  3. Give TST the permission to read your clipboard
  4. Enable dom.events.asyncClipboard.clipboardItem
  5. Copy a URL into your clipboard
  6. Middle click the + button at the bottom of TST
  7. Nothing happens.

Expected result

The URL in my clipboard should open.

Actual result

Nothing happens

Environment

  • Platform (OS): Windows 10 Version 22H2
  • Version of Firefox: 126.0.1
  • Version (or revision) of Tree Style Tab: 4.0.15
@piroor
Copy link
Owner

piroor commented Jun 8, 2024

Did you turn these checkboxes on?
image
"Open new tab with URL in the clipboard" is checked by default only on Linux (because "browser.tabs.searchclopboard.middleclick" is also active by default only on Linux) and you need to activate it manually on other platforms.

@karasuhebi
Copy link
Author

I did, yeah. Still doesn't work. :(

piroor added a commit that referenced this issue Jul 1, 2024
…ermission is granted but internal option is still deactivated #3569
@piroor
Copy link
Owner

piroor commented Jul 1, 2024

I got what happened. The checkbox become checked even if its internal option is still disabled, when you grant the clipboard permission by the "Permissions" tab provided by Firefox. You need to uncheck the "Open new tab with URL in the clipboard" checkbox and check it again for now, to activate the internal option.

I've pushed a change 3d9d79b to fix this "mismatched checkbox state" bug.

@karasuhebi
Copy link
Author

That worked! Thanks!

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

2 participants