snoozz-tab-snoozing icon indicating copy to clipboard operation
snoozz-tab-snoozing copied to clipboard

[BUG] snoozing a still-loading tab snoozes whole Firefox windows with thousands of tabs

Open DrunkenCommie opened this issue 3 years ago • 1 comments

Description

When you try to snooze a tab that didn't fully load yet (moving dot on the tab, showing "still loading" - maybe you just opened this tab, or it refreshes itself) - that tab will not snooze. Instead - whole Firefox window will snooze (read: it will add itself to Snoozz list, and close the whole window, with all tabs). Workaround is to press Escape on that still-loading tab (breaking the loading), then snoozing it - then it works as expected.

Expected Behavior

Still-loading tab should be re-snoozed for later.

Actual Behavior

Whole Firefox window (with 3000+ tabs) closes, and gets snoozed until {{specified time}}, then at that specified time reopens window with all tabs active (causing 100% CPU/memory hog).

How to Reproduce (if applicable)

Find a website that loads for more than few seconds (something with dynamic loading content -- or just something with lots of data), try to snooze it ("in one hour", "tomorrow morning" - anything). Whole Firefox window will close. When you reopen Firefox and open Snoozz list, you'll see your whole window "started napping at..." (see screenshot).

Your Device

  • Browser and version: Firefox 90.0.1
  • OS: Windows 10, ver 21H1
  • Device Resolution (UI related bugs only): 3000x2000

Which Page

Any slow-loading page.

Screenshots / Videos / Logs

obraz

DrunkenCommie avatar Jul 21 '21 13:07 DrunkenCommie

Cant reproduce this exactly but i think I know what the issue is.

Snoozz disables the "Tab" option when it is not possible to snooze the current tab (in this case the current tab has no url since it has not loaded). When this happens, I check if any other tab in the windows is snoozable, and if there is at least one, then it will switch to the "Window" option automatically.

This has always been the behaviour since I introduced window snoozing, but I can see how it might be a bit problematic in cases like this. I think I will remove this automatic switching and just have the user manually switch to "Window" if needed

rohanb10 avatar Jul 30 '21 06:07 rohanb10