Amndeep7/ptw-extender

Context menu doesn't show up

Closed this issue · 1 comments

Supposedly the browser.runtime.OnInstalled listener wrapper should have been sufficient to make sure that the context menu always shows up, but that was not the case for some reason (FF on Win10). Reinstalling the add-on made it work fine, which makes sense cause it'll definitely hit the oninstalled event then, but apparently there might be some event that happens where the browser starts up without this event having fired so the context menu never gets loaded in. I'll just buck what appears to be the supposedly accepted practice of having that wrapper and instead just on every load of the extension, I'll make it so that all the contextmenus get removed and then I add in my little guy again.

Example of best practice that doesn't always work out :(

Example of hackier practice that I think will be more consistent at the expense of a few cpu cycles on the part of the user.

Solved with #20