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

wrong count/total #1

Open
jarredou opened this issue Aug 11, 2017 · 4 comments
Open

wrong count/total #1

jarredou opened this issue Aug 11, 2017 · 4 comments

Comments

@jarredou
Copy link

jarredou commented Aug 11, 2017

Current window | 49
All windows | 48

This impossible situation seems to be a conflict with some other plugins that open their own window(s).
In my situation DTA seems to be the culprit (when the DTA window is opened) as the total count was corrected when I've closed its window (but as DTA is not and will not be compatible with WebExtensions, it is maybe not revelant).

@jarredou jarredou changed the title wrong total wrong count/total Aug 11, 2017
@grahamperrin
Copy link

grahamperrin commented Oct 3, 2017

Sometimes count for 'all' is correct (compared with Tab Tally) but here it's way off:

2017-10-03 14 25 24

Is that the known limitation?

  • The badge gets updated globally, so it cannot show the number of tabs in two windows at once. In inactive windows, the number of tabs in the active window is shown. If no window is active, it is the number of the last active window instead.

After pulling three other counters into the mix, adding a third tab to a window where the first tab was pinned, Sum Tabs counted only two of the three:

2017-10-03 14 30 20 add-ons manager

@Timvde
Copy link
Owner

Timvde commented Oct 11, 2017

Oops, apparently I'm not watching my own repository. That's kinda unfortunate.

In my limited testing when I created the extension, I could not find broken cases. I definitely believe that there are situations that I haven't covered, though. If you could provide clear steps to reproduce on a clean profile with only my add-on (steps can include installing another add-on if that's the cause), that would greatly help me find the issue.

@grahamperrin
Copy link

grahamperrin commented Oct 13, 2017

I aimed to use a session file from my everyday profile, with a clean profile, but with Mozilla's .jsonlz4 restriction, I found no way to do so 👎


Instead, I used a clean profile with Sum Tabs alongside one other extension, which allows a more flexible approach to session management: Session Manager.

I opened a .session file from a session with 1,052 tabs. Sum Tabs counted 526:

2017-10-13 20 20 41

Also in that screenshot, the count for the current window (33) is probably wrong. My recollection (from use of the everyday profile) was that the window contained more than two hundred tabs. Subsequent use of the everyday profile confirms that – here, 206:

2017-10-14 00 34 58

– that number presented by Clear Tabs. Plus, the small size of the scroller in the sidebar view of Tab Center Redux suggests a long list, of more than 33 items.

@Timvde
Copy link
Owner

Timvde commented Oct 16, 2017

I found that my add-on is counting one tab too many per window. I think that something actually changed in Firefox, because I can easily reproduce it, and this definitely was not an issue before.

With regard to the other issue (being off by a few hundred), I expect that it is counting tabs before all windows have opened. I never tried with more than two or three windows open, but I imagine it can be an issue when you have over a dozen in your session restore. I'll see if I can delay initialization.

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

No branches or pull requests

3 participants