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

adding filter for number of examples #268

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

savrajsi
Copy link
Collaborator

@savrajsi savrajsi commented Jun 9, 2024

No description provided.

@savrajsi
Copy link
Collaborator Author

Screenshot 2024-06-19 at 8 17 06 PM

@samlhuillier
Copy link
Collaborator

Thanks for making this. I have a couple improvement ideas:

  • Allow for 0 notes for context
  • Filters would be slightly nicer as a slider than a dropdown
  • It should update the state (and electron store) automatically on change - the update filters button is hidden and non-obvious
  • Date component should be slightly more intuitive. I see it as being able to hit like "last hour", "last day", "last week", "last month" etc. options and then being able to hit something like an "advanced" toggle to reveal the full date picker (that is hidden initially)

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.

None yet

3 participants