-
Notifications
You must be signed in to change notification settings - Fork 11
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
FF 55 bug #38
Comments
On FF57 (linux) it works, but the tab has a different background colour (grey) than when testing (where it was white). I don't know how to fix this. (I can hardly make official releases only for testing...) |
Maybe it helps if you start with a clean profile? I had already realized since quite a while that I have to remove addonStartup.json.lz4 to get proper updates of system-wide installed extensions. |
The background colour seems to be related with some profile settings which I had. Still no idea why it does not work for you with FF 55; all features of the tab API which I use are listed to work with FF 45. |
By accident, I had now access to a Windows laptop w/ FF 57. It also works there with a sane background colour (almost white). It really seems to be related with FF 55 (or possibly with the profile). |
I tried with a new clean profile and its didn't help. |
I still have no idea why it does not work. I checked again all functions used in background.js, but they are all documented to work since 45.0. If one could rely that the xxxxx is fixed for all versions of the extensions, perhaps I should document the workaround. I have no idea what the xxxxx refers to: I had explicitly named the ID of the extension in the manifest.json to avoid getting assigned such a "random" number, but using that ID instead of xxxxx does not work. |
It seems that all WE-addons show generated ID instead "name" in their tabs address. |
The workaround is now documented in the README.md and on the mozilla page for the addon. |
Sad, but now I realized that this random ID is generated individually, so the workaround will not help the most users, i guess so. P.S. |
I was suspecting this ID issue. |
I forgot: I do not think that it is possible to mark an extension incompatible with a particular FF version; I could claim that it is only FF 57+ compatible, but this would probably exclude FF 52 users which is presumably a bad idea. |
To be honest, I don't know how to delete (or rebase) this PR. |
I don't think it is possible: The PR was based on a commit which no longer exists. I guess that in order to keep the PR, github has to keep a reference on that commit somehow. |
Hurray, support deleted this PR at my request. Also, seems that I found a way to change ID. 0 run FF After these steps the ID will be re-generated. |
Thanks a lot!
OK. So it is clear that this is not an easy workaround for the problem, because the users have no easy way to find out the ID unless they run a working firefox installation in parallel. SInce it seems to be a problem with FF 55 only (and probably FF 55 should not be used for a long time for security reasons anyway), I think I will just leave the issue open and keep it documented as "known bugs". |
upd |
A few minutes before your posting, I had released 4.2: |
Yes, it works. |
Great! Thanks for the testing! |
Looks like the "star" button doesn't work in Firefox 55.0.3 (x64).
When I click on it - nothing happens.
Also, I tried to launch a browser with disabled addons (except Bookmark Dupes).
The text was updated successfully, but these errors were encountered: