642/create new ask from contributions #775
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why
Resolves issue 642
Pre-Merge Checklist
Does this need to be added to the CHANGELOG.md? If so, please let me know which version to add it to.
What
How
Seeing as the main purpose of this page is to view existing Asks/Offers, I've placed these actions at the bottom of the page. I also chose not to wrap them in a "card" class, as I wanted them to be easy to distinguish between all the other Ask/Offer cards. If you prefer this to look differently or be placed at the top of the page, let me know and I'll update the code.
Testing
Added test to confirm that the new section and buttons have been rendered on the page.
Next Steps
Accessibility: Across the entire app, the focus on the primary buttons has been either removed or overwritten. I tabbed through the page and couldn't really tell which buttons were focused. Consider either improving the current styles or removing them altogether (while they're usually not pretty, default focus styles are more accessible).
Outstanding Questions, Concerns and Other Notes
Accessibility
Buttons are consistent with what's been used on the page so far. Users can access and trigger the newly added Ask/Offer buttons by keyboard. See 'Next steps' for more.