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

Can't open logger window #2249

Closed
8 tasks done
DeepChirp opened this issue Sep 7, 2022 · 14 comments
Closed
8 tasks done

Can't open logger window #2249

DeepChirp opened this issue Sep 7, 2022 · 14 comments
Labels
bug Something isn't working

Comments

@DeepChirp
Copy link

DeepChirp commented Sep 7, 2022

Prerequisites

I tried to reproduce the issue when...

  • uBO is the only extension
  • uBO with default lists/settings
  • using a new, unmodified browser profile

Description

The logger window cannot be opened on Microsoft Edge, clicking the icon of the recorder has no effect, only right-click and open in a new tab.

A specific URL where the issue occurs

No need

Steps to Reproduce

  1. Click on the uBlock Origin icon.
  2. Click the logger icon.

Expected behavior

Logger window appears.

Actual behavior

Logger window doesn't appear.

uBlock Origin version

1.44.0

Browser name and version

Microsoft Edge 104.0.1293.70 (64-bit)

Operating System and version

Windows 11 21H2 (22000.856)

2022-09-07_23-58-28

@gwarser
Copy link
Member

gwarser commented Sep 7, 2022

Click it once with Shift pressed - should open in new tab.

Something is broken with "new window" mode.

@gwarser gwarser added Edge specific to Edge bug Something isn't working labels Sep 7, 2022
@Nojuuu
Copy link

Nojuuu commented Sep 15, 2022

Same seems to happen on Brave but only with latest uBO dev build?

@effleurager
Copy link

This is happening also on Chrome 104.0.5112.101 with µBlock 1.14.4 on macOS 12.5.1, so it can probably be found with a bisect of the Chromium source

@krystian3w
Copy link

krystian3w commented Apr 21, 2023

Maybe corruption in extension folder, for me it helped backup settings and refresh addon.

I didn't checked import backup without reset - maybe possible skip reset.

@gwarser gwarser added the unable to reproduce cannot reproduce the issue label Apr 21, 2023
@Bartolomeus-649
Copy link

I experience this as well in Microsoft Edge Version 113.0.1774.57 (Official build) (64-bit).

What kind of logs/tracing/troubleshoot-info would be helpful to understand what's going on a bit more?

@gorhill
Copy link
Member

gorhill commented Jun 5, 2023

You can put a breakpoint where the logger's tab or window is created. By default it opens as a popup window, unless you changed this.

@Bartolomeus-649
Copy link

as soon as I opened up the devtools for the extension it all started to work again.

@zachpoblete
Copy link

Still happens on Edge v116.0.1938.81. Wish I was knowledgeable enough to implement #2249 (comment):

You can put a breakpoint where the logger's tab or window is created. By default it opens as a popup window, unless you changed this.

@gorhill
Copy link
Member

gorhill commented Nov 29, 2023

Still unable to reproduce, closing as stale issue.

@gorhill gorhill closed this as completed Nov 29, 2023
@ArgonAlex
Copy link

I just ran into this on Chrome 122, Mac 14.3.1. When I open dev tools for background.html I see an error in the console:
Unchecked runtime.lastError: Invalid value for bounds. Bounds must be at least 50% within visible screen space.

And when I set a breakpoint, I can see that webext.windows.create() is indeed throwing the error and returning null.
https://github.com/gorhill/uBlock/blob/c2c80be8976a7f7687733acca341776d4be21a15/platform/common/vapi-background.js#L675

debugging

@gorhill
Copy link
Member

gorhill commented Mar 11, 2024

What's your screen resolution?

@ArgonAlex
Copy link

ArgonAlex commented Mar 13, 2024

I was able to reproduce the issue on a Windows machine as well as my Mac. I don't think screen resolution matters. It seems that when you move the logger window such that most of it is off screen and then close it, the extension is caching that last position in localStorage and then the next time you try to open it, it attempts to use that last position, which fails.

I was able to get the logger to open again by calling vAPI.localStorage.removeItem('popupLoggerBox').

I haven't tested this, but I suspect this may have originally happened when I closed the logger on an external monitor at work, then I attempted to open it again with the monitor disconnected, or at home where I have a different monitor configuration. The cached position would possibly be invalid in these situations.

@gorhill gorhill reopened this Mar 13, 2024
@gwarser gwarser removed the unable to reproduce cannot reproduce the issue label Mar 13, 2024
@gorhill
Copy link
Member

gorhill commented Mar 13, 2024

Thanks for pinpointing the issue, I can also reproduce on my side with Linux, I just drag the popup window as low as possible at the bottom, then close it, and it won't re-open afterward. I will provide a new code path for when the popup window fails to be created.


Forgot to mention that I reproduced with plain Chromium -- so not an Edge-specific issue.

gorhill added a commit to gorhill/uBlock that referenced this issue Mar 13, 2024
@gorhill gorhill removed the Edge specific to Edge label Mar 13, 2024
@MasterKia MasterKia changed the title Can't open logger window on Microsoft Edge Can't open logger window Mar 13, 2024
@gorhill
Copy link
Member

gorhill commented Apr 15, 2024

No feedback, assumed fixed.

@gorhill gorhill closed this as completed Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

9 participants