Skip to content
This repository has been archived by the owner on Dec 3, 2020. It is now read-only.

Remove prototype sidebar #33

Closed
Osmose opened this issue Jul 25, 2018 · 0 comments
Closed

Remove prototype sidebar #33

Osmose opened this issue Jul 25, 2018 · 0 comments
Assignees

Comments

@Osmose
Copy link
Contributor

Osmose commented Jul 25, 2018

We added a sidebar to help drive prototyping, but it's time to remove it and the related, non-reusable code that supports it. We should keep things like the pending-landing code for running automated tests that we'll eventually reuse.

@biancadanforth biancadanforth self-assigned this Jul 26, 2018
biancadanforth added a commit that referenced this issue Jul 27, 2018
Removed the sidebar panel, and added a pageAction panel, which opens/closes on pageAction click.

Added a 'priceComparison' object that, similar to the 'product' object, is passed from 'background.js' to the pageAction panel ('./src/popup/index.jsx') via the panel's URL as a query string. This 'priceComparison' object is currently pulled from a stub method, 'getPriceComparisonData', which currently just returns a static object literal. Ultimately this method would call an external API and return the data we need to display in the panel.

TODO: Add CSS and top left-most icon.
@biancadanforth biancadanforth removed their assignment Jul 30, 2018
@Osmose Osmose self-assigned this Jul 30, 2018
@Osmose Osmose closed this as completed in 1700986 Aug 1, 2018
Osmose added a commit that referenced this issue Aug 1, 2018
Fix #33: Remove prototype sidebar.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants