Toolkit learn page #71
Conversation
gurtatiLND
requested review from
maxitect and
jackcasstlesjones
as code owners
✅ Deploy Preview for thingswedo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Thanks Tanya, good work getting the content in!
The style of the info page is not consistent with the one created for moods (particularly around the header component and heading sizes), see screenshots below
Perhaps use the moods learn more page styling classes as a guide so we are more consistent
src/app/home/page.tsx
Outdated
There was a problem hiding this comment.
I don't think this file is required for this PR, and maybe even for the project? Pending decision to start the app on toolkit, home is just the "/" route and so it is the page.tsx directly under app/
There was a problem hiding this comment.
Oh, that was a mistake, thanks for noticing!! deleted it.
thank you for noticing! changed Co-authored-by: Maxime Downe <[email protected]>
What type of PR is this? (check all applicable)
Description
Screenshots
UI accessibility checklist
If your PR includes UI changes, please utilize this checklist:
CriticalandSeriousissues?Added/updated tests?
Please aim to keep the code coverage percentage at 80% and above.
have not been included
Delete branch after merge?
What gif best describes this PR or how it makes you feel?