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

Stores sidebar #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

rodrigoheffner
Copy link

Replaced the store filter for a sidebar component. Stores can be selected and deselected individually - users will also be able to select / deselect all stores.

I believe the implementation for this feature is finished but I can think of two possible improvements (let me know if you want them implemented before merging this pull request):

  1. Initially, it's arguable that all the store checkboxes should be selected. This is not reflective of the implementation at the moment.
  2. The "select all" checkbox could have a bit more intelligence - it could automatically check itself once all the stores are selected and deselect itself once any store is unchecked.

Added all the stores in a sidebar panel. The panel is mobile friendly
and will either take 25% of the width of the page or 100% for devices
with smaller screens.

TODO:
- Hook selection / deselection to the form;
- Hook buttons to the form;
- Remove the previous implementation;
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

Successfully merging this pull request may close these issues.

1 participant