We received user reports that the HTML elements for the pocket dropdown were being included on every page before a user had saved the page. This was adding unnecessary clutter to the dom.
Todos:
[x] Remove injectDomElements call from initial page load
[x] Bumped manifest
Implementation Decisions
We were attempting to inject the elements into the dom twice; once on page load, and once when a user saved the page. I removed the call to the inject script from page load.
Goal
We received user reports that the HTML elements for the pocket dropdown were being included on every page before a user had saved the page. This was adding unnecessary clutter to the dom.
Todos:
injectDomElementscall from initial page loadImplementation Decisions
We were attempting to inject the elements into the dom twice; once on page load, and once when a user saved the page. I removed the call to the inject script from page load.
The call on page save is here: https://github.com/Pocket/extension-save-to-pocket/blob/master/src/containers/dispatch.js#L70