-
Notifications
You must be signed in to change notification settings - Fork 13
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
[Issue #1469] Add Funding Opp component w/ stubbed markup #1470
[Issue #1469] Add Funding Opp component w/ stubbed markup #1470
Conversation
name={option.id} | ||
label={option.label} | ||
// onChange={(e) => handleCheck(option.value, e.target.checked)} | ||
// disabled={!mounted} // Required to be disabled until hydrated so query params are updated properly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
<div className="grid-col-fill text-right"> | ||
<button className="usa-button usa-button--unstyled font-sans-xs"> | ||
Clear All | ||
</button> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Were you planning to handle the select and clear ina separate pr?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah. I wasn't handling any functionality here. Just getting the scaffolding in there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's approve and keep iterating with other PRs
Summary
Fixes #1469
Time to review: 5 mins
Changes proposed
SearchFundingOpportunity
componentContext for reviewers
This just stubs in the markup. Still needs to be hooked up so it functions
Additional information