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

[Task]: Disable select all when all selected. Disable clear all when none selected. #1491

Closed
rylew1 opened this issue Mar 18, 2024 · 2 comments · Fixed by #1495
Closed

[Task]: Disable select all when all selected. Disable clear all when none selected. #1491

rylew1 opened this issue Mar 18, 2024 · 2 comments · Fixed by #1495
Assignees
Labels
project: grants.gov Grants.gov Modernization tickets

Comments

@rylew1
Copy link
Contributor

rylew1 commented Mar 18, 2024

Summary

Disable select all when all selected. Disable clear all when none selected. Can likely set the disabled prop (or attribute) here.

Acceptance criteria

  • You can't Select All twice in succession
  • You can't Clear All twice in succession
@rylew1 rylew1 added the project: grants.gov Grants.gov Modernization tickets label Mar 18, 2024
@rylew1
Copy link
Contributor Author

rylew1 commented Mar 18, 2024

cc @crystabelrangel @andycochran

@rylew1 rylew1 self-assigned this Mar 18, 2024
@andycochran
Copy link
Collaborator

You should be able to just add the disabled attribute (prop?) to the button component.

rylew1 added a commit that referenced this issue Mar 19, 2024
…r accordion (#1495)

## Summary
Fixes #1491 

- add states for tracking clear all / select all
- move `searchfetcher` classes to `/services/search/searchfetcher`
folder
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
project: grants.gov Grants.gov Modernization tickets
Projects
2 participants