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

Date picker for time range selection #76

Merged
merged 13 commits into from Mar 28, 2017
Merged

Date picker for time range selection #76

merged 13 commits into from Mar 28, 2017

Conversation

@Treora
Copy link
Collaborator

@Treora Treora commented Mar 24, 2017

This builds further on @RajPratim21's work of #74, to resolve #26. Main changes:

  • Make search independent of the store again (pass date range as arguments).
  • Clean up code and css.
  • Style UI a bit.
  • Refresh search results when date range changes.
  • Also apply the date filter when no search query is given.

Comments are welcome, otherwise I will merge it as is in a few days.

@RajPratim21
Copy link
Contributor

@RajPratim21 RajPratim21 commented Mar 24, 2017

@Treora , Thank you for your help, Once you merge it I will make the required changes in the PR #75 for natural language date parser, as they are working in same style, to maintain the integrity.

@Treora Treora merged commit 7d2fd51 into master Mar 28, 2017
@Treora Treora deleted the datepicker branch Mar 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants