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

Refactor of Saved Search Accordion #543

Closed
2 of 5 tasks
hawkishpolicy opened this issue Aug 15, 2024 · 0 comments · Fixed by #561
Closed
2 of 5 tasks

Refactor of Saved Search Accordion #543

hawkishpolicy opened this issue Aug 15, 2024 · 0 comments · Fixed by #561
Assignees
Labels

Comments

@hawkishpolicy
Copy link
Collaborator

hawkishpolicy commented Aug 15, 2024

💡 Summary

  • The creation of the new Side Drawer with its use of Text Fields and toggle-able Checkboxes makes the current Data Grid based component feel out of place.
  • Plus there needs to be a way to quickly toggle a Saved Search off and on, prevent duplicate names and very general searches.
  • Given this and the nature of the data displayed, a conversion of the table into a Checkbox/Radio controlled list would be a good step forward.

Motivation and context

  • Resolve the aforementioned bugs.
  • Consistent User Experience.
  • Reduced footprint.

Implementation notes

Screenshot 2024-08-15 at 10 38 25 AM

Screenshot 2024-08-15 at 10 52 07 AM

Acceptance criteria

How do we know when this work is done?

  • New Checkbox controlled list is implemented.
  • It displays a list of user created Saved Searches.
  • Checked entries indicates that the selected Search is being used.
  • Unchecked entries indicates the item is not in use.
  • An icon to delete each Saved Search.
@hawkishpolicy hawkishpolicy self-assigned this Aug 15, 2024
@hawkishpolicy hawkishpolicy linked a pull request Aug 24, 2024 that will close this issue
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant