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(connect-webextension): Trezor popup displaying UNDER browser extension popup on Linux (Firefox) #13527

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

superKalo
Copy link
Contributor

Description

When a browser extension opens its own popup (via chrome.windows.create as a type: popup) on Linux (Firefox), this window stays always on top. If the Trezor popup then gets requested within this extension popup, the Trezor popup was appearing behind the extension popup, because the Trezor's one had the default type ("normal").

This made the Trezor popup hardly accessible (visible) for the user, if one doesn't think to move away the extension popup.

With this change, when a Trezor popup gets opened from a browser extension popup, it will appear on top of the extension popup.

Related Issue

#13526

When extension popup is open (via chrome.windows.create as a type: popup) on Linux, it stays always on top. If the Trezor popup then gets requested within this extension popup, the Trezor popup was appearing behind the extension popup, because it had the default type of "normal". With this change, it will now appear on top of the extension popup.
@superKalo superKalo changed the title Fixed: Trezor popup displaying UNDER browser extension popup on Linux (Firefox) fix(connect-webextension): Trezor popup displaying UNDER browser extension popup on Linux (Firefox) Jul 26, 2024
@mroz22
Copy link
Contributor

mroz22 commented Jul 29, 2024

do you have a public reproduction where we could test it?

@superKalo
Copy link
Contributor Author

@mroz22 thanks for stepping in. It's a bit tricky, you can simulate it by:

  1. On Linux, with Firefox.

  2. Open the browser console in any extension and please run: browser.windows.create({ url: "https://example.com/", type: 'popup' }) (in the browser console of an extension, so that the browser is available). This will open a browser window (type: popup) and will simulate the problem - this window stays always on top.

  3. Now please trigger signing with Trezor within any extension popup (for example with MetaMask when signing a swap on UniSwap or signing a message via Ambire's sig tool). The Trezor popup will appear UNDER the https://example.com/ window.

This is making the Trezor popup hardly accessible (visible) for the user, if one doesn't think to move away the https://example.com/ window popup (which simulates an extension window popup).

With my changes, same scenario, the Trezor popup will appear as a popup now and on top of the https://example.com/ window popup.

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.

2 participants